Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Akim Demaille
Hi all, Here is an updated proposal, that was taking into account several comments made last week. 0001-bootstrap-let-warn-be-like-tests-init.sh-s-warn_.patch Description: Binary data

Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Jim Meyering
Akim Demaille wrote: > Here is an updated proposal, that was taking into account > several comments made last week. Looks fine, modulo a question and test artifacts: Thanks! ... > -warn() > +# warnf_ FORMAT-STRING ARG1... > +warnf_ () > { > - for i > - do > -echo "$i" > - done | sed -e "s

Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Akim Demaille
Le 10 juil. 2012 à 10:14, Jim Meyering a écrit : >> + case $* in >> +*$nl*) me_=$(printf "$me"|tr "$nl|" '??') > > Is it worth testing for both $nl and '|' ? I tried to keep what you did about $nl in $me (it would break the sed command). I used printf, since the \n from echo gave a spurio

Re: add --match argument to git-version-gen

2012-07-10 Thread Andy Wingo
On Sat 07 Jul 2012 22:42, Jim Meyering writes: >> - --prefix prefix of git tags (default 'v') >> + --prefix prefix of git tags to strip from version (default 'v') >> + --match pattern for git tags to match > > Please mention that it's a "glob" pattern, > so people don't t

Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Jim Meyering
Akim Demaille wrote: > Le 10 juil. 2012 à 10:14, Jim Meyering a écrit : > >>> + case $* in >>> +*$nl*) me_=$(printf "$me"|tr "$nl|" '??') >> >> Is it worth testing for both $nl and '|' ? > > I tried to keep what you did about $nl in $me (it would break > the sed command). I used printf, sinc

Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Akim Demaille
Le 10 juil. 2012 à 12:05, Jim Meyering a écrit : > Akim Demaille wrote: > >> Le 10 juil. 2012 à 10:14, Jim Meyering a écrit : >> + case $* in +*$nl*) me_=$(printf "$me"|tr "$nl|" '??') >>> >>> Is it worth testing for both $nl and '|' ? >> >> I tried to keep what you did about $n

Re: [PATCH] bootstrap: use a more consistent error reporting scheme.

2012-07-10 Thread Jim Meyering
Akim Demaille wrote: > Le 10 juil. 2012 à 12:05, Jim Meyering a écrit : > >> Akim Demaille wrote: >> >>> Le 10 juil. 2012 à 10:14, Jim Meyering a écrit : >>> > + case $* in > +*$nl*) me_=$(printf "$me"|tr "$nl|" '??') Is it worth testing for both $nl and '|' ? >>> >>> I trie

request to relax license of 'stat-time'

2012-07-10 Thread Eric Blake
Libvirt would like to use lib/stat-time.h to portably get at nanosecond timestamps from an LGPLv2+ license; but this file is currently licensed LGPLv3+. Any objections to relaxing the license, particularly since POSIX 2008 guarantees that stat() should be returning this information and the stat mo

Re: request to relax license of 'stat-time'

2012-07-10 Thread Jim Meyering
Eric Blake wrote: > Libvirt would like to use lib/stat-time.h to portably get at nanosecond > timestamps from an LGPLv2+ license; but this file is currently licensed > LGPLv3+. Any objections to relaxing the license, particularly since > POSIX 2008 guarantees that stat() should be returning this i

Re: request to relax license of 'stat-time'

2012-07-10 Thread Paul Eggert
On 07/10/2012 06:29 AM, Jim Meyering wrote: > That is fine with me. > (Bruno and Paul should also sign off) Fine with me too.

Re: request to relax license of 'stat-time'

2012-07-10 Thread Bruno Haible
Eric Blake wrote: > > Libvirt would like to use lib/stat-time.h to portably get at nanosecond > > timestamps from an LGPLv2+ license; but this file is currently licensed > > LGPLv3+. Any objections to relaxing the license, particularly since > > POSIX 2008 guarantees that stat() should be returnin

Re: bug#11905: Unnecessary '#define unix' in src/s/aix4-2.h

2012-07-10 Thread Paul Eggert
On 07/10/2012 04:25 PM, Glenn Morris wrote: > I suspect that Gnulib already has the AIX > case covered (via "__unix"?), so that src/s/aix4-2.h no longer needs to > define "unix". Yes, that sounds right. I have started to fix this by installing the following patch into gnulib, and will propagate t