Re: [gentoo-dev] [PATCH] perl-functions.eclass: Replace unnecessary eval with ${!var}

2017-02-25 Thread Kent Fredric
On Fri, 24 Feb 2017 14:11:57 +0100 Michał Górny wrote: > If there is no hurry to the other changes, maybe either of us could > commit them all at once along with all those eval patches (say, in 6 > days)? The eutils change is going to cause almost everything to be >

Re: [gentoo-dev] Removal of CVS headers

2017-02-25 Thread Rich Freeman
On Sat, Feb 25, 2017 at 10:12 AM, Ulrich Mueller wrote: >> On Sat, 25 Feb 2017, Sergei Trofimovich wrote: > >> Typical questions for tree-wide cleanups: > >> - Are new ebuilds forbidden to have '$Id$' or just discouraged? >> - [same as above] Will new version of repoman

Re: [gentoo-dev] Removal of CVS headers

2017-02-25 Thread Ulrich Mueller
> On Sat, 25 Feb 2017, Sergei Trofimovich wrote: > Typical questions for tree-wide cleanups: > - Are new ebuilds forbidden to have '$Id$' or just discouraged? > - [same as above] Will new version of repoman complain about > leftover '$Id$'? Not sure. That would have to be controlled via

Re: [gentoo-dev] Removal of CVS headers

2017-02-25 Thread Sergei Trofimovich
On Sat, 25 Feb 2017 15:05:09 +0100 Ulrich Mueller wrote: > As the council has decided in its 2014-10-14 meeting (and confirmed > again in the 2016-11-13 meeting), CVS headers should be removed after > the migration to Git. Until recently, this was blocked by repoman > still

Re: [gentoo-dev] [PATCH v2] java-ant-2.eclass: Replace unnecessary evals with arrays

2017-02-25 Thread James Le Cuirot
On Wed, 22 Feb 2017 22:54:22 +0100 Michał Górny wrote: > Replace the horrifying use of evals along with quoting to pass multiple > filenames whitespace-safe with much simpler bash arrays. While at it, > also simplify the find-read loop. I've given this a rough test. Some of

Re: [gentoo-dev] Removal of CVS headers

2017-02-25 Thread Rich Freeman
On Sat, Feb 25, 2017 at 9:05 AM, Ulrich Mueller wrote: > > Therefore, I am going to remove the remaining CVS headers throughout > the tree (except for patches, of course) in two days from now. > Speaking from git migration experience, I'd be really careful about how you go about

[gentoo-dev] Removal of CVS headers

2017-02-25 Thread Ulrich Mueller
As the council has decided in its 2014-10-14 meeting (and confirmed again in the 2016-11-13 meeting), CVS headers should be removed after the migration to Git. Until recently, this was blocked by repoman still checking for the $Id$ line. The latter is now fixed in the stable repoman version.