[PATCH] bootstrap: fail when bootstrap_post_import_hook fails

2012-01-21 Thread Jim Meyering
FYI, >From 2f1a477706b06256ddca33739a771e3d97d011f1 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 21 Jan 2012 18:12:30 +0100 Subject: [PATCH] bootstrap: fail when bootstrap_post_import_hook fails Otherwise, it's far too easy to miss diagnostics emitted between gnulib-tool's output and t

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Bruno Haible wrote: > >> I too would prefer to use Paul's patch and thus to remove the >> doc/INSTALL* exemptions. >> >> Bruno, I know it's not your preference, but would that be ok with you. > > My objections were mostly esthetic, hence minor. You can decide to > ignore my esthetic preferences. T

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Bruno Haible
Hi Jim, > I too would prefer to use Paul's patch and thus to remove the > doc/INSTALL* exemptions. > > Bruno, I know it's not your preference, but would that be ok with you. My objections were mostly esthetic, hence minor. You can decide to ignore my esthetic preferences. Bruno

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Jim Meyering wrote: > Eric Blake wrote: >> On 01/17/2012 11:44 AM, Jim Meyering wrote: >> > * doc/INSTALL: Likewise. > * doc/INSTALL.ISO: Likewise. > * doc/INSTALL.UTF-8: Likewise. These files are generated through doc/Makefile. Also, on text files that consist of multip

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Eric Blake wrote: > On 01/17/2012 11:44 AM, Jim Meyering wrote: > * doc/INSTALL: Likewise. * doc/INSTALL.ISO: Likewise. * doc/INSTALL.UTF-8: Likewise. >>> >>> These files are generated through doc/Makefile. Also, on text files >>> that consist of multiple paragraphs, I find it good (

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Eric Blake
On 01/17/2012 11:44 AM, Jim Meyering wrote: >>> * doc/INSTALL: Likewise. >>> * doc/INSTALL.ISO: Likewise. >>> * doc/INSTALL.UTF-8: Likewise. >> >> These files are generated through doc/Makefile. Also, on text files >> that consist of multiple paragraphs, I find it good (esthetic) to have >> a blan

Re: [PATCH 2/7] maint: remove empty lines at EOF

2012-01-21 Thread Jim Meyering
Eric Blake wrote: > On 01/21/2012 04:14 AM, Jim Meyering wrote: >> From: Jim Meyering >> >> * man/autoconf.x: Remove empty line at EOF. >> * man/autoheader.x: Likewise. >> * man/autoscan.x: Likewise. >> * man/autoupdate.x: Likewise. >> * man/ifnames.x: Likewise. >> * tests/compile.at: Likewise. >

Re: [PATCH 2/7] maint: remove empty lines at EOF

2012-01-21 Thread Eric Blake
On 01/21/2012 04:14 AM, Jim Meyering wrote: > From: Jim Meyering > > * man/autoconf.x: Remove empty line at EOF. > * man/autoheader.x: Likewise. > * man/autoscan.x: Likewise. > * man/autoupdate.x: Likewise. > * man/ifnames.x: Likewise. > * tests/compile.at: Likewise. > * doc/fdl.texi: Likewise. >

locale-fr.m4 and Bionic

2012-01-21 Thread Bruno Haible
Hi, Cross-compiling a testdir, I see these problems during the configuration: checking whether mbrtowc handles a NULL pwc argument... ../configure: line 22091: test: !=: unary operator expected guessing yes checking whether mbrtowc handles a NULL string argument... ../configure: line 22168: tes

Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-21 Thread Bruno Haible
Hi Simon, Thanks for the report. > I think it helps, but it wasn't complete. Below is a new > backtrace. It seems the system wchar.h includes the system time.h, > which triggers the same problem... that may be harder to fix? > > arm-linux-androideabi-gcc > --sysroot /home/jas/android-ndk-r7/p

Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-21 Thread Bruno Haible
Paul Eggert wrote: > --- a/lib/stdint.in.h > +++ b/lib/stdint.in.h > @@ -521,9 +521,11 @@ typedef int _verify_intmax_size[sizeof (intmax_t) == > sizeof (uintmax_t) > #if @HAVE_WCHAR_H@ && ! (defined WCHAR_MIN && defined WCHAR_MAX) >/* BSD/OS 4.0.1 has a bug: , and must be > included b

Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-21 Thread Simon Josefsson
fre 2012-01-20 klockan 17:17 -0800 skrev Paul Eggert: > Here's a proposed patch for that; it's pretty simple and if your > analysis is correct it should do the trick. Thanks. I think it helps, but it wasn't complete. Below is a new backtrace. It seems the system wchar.h includes the system time