Re: [Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-06 Thread Ian Lance Taylor
On Wed, Nov 6, 2013 at 7:37 AM, Tom Tromey wrote: >> "Tobias" == Tobias Burnus writes: > > Tobias> + cpp_warning (pfile, CPP_W_DATE_TIME, "Macro \"%s\" might > prevent " > Tobias> + "reproduce builds", NODE_NAME (node)); > > Tobias> + cpp_warning (pfile, CP

Re: [Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-06 Thread Tom Tromey
> "Tobias" == Tobias Burnus writes: Tobias> Updated version attached – after bootstrapping and regtesting on Tobias> x86-64-gnu-linux Tobias> OK? Sorry, I didn't notice this until today. Tobias> @@ -925,7 +928,8 @@ enum { Tobias>CPP_W_NORMALIZE, Tobias>CPP_W_INVALID_PCH, Tobias>

Re: [Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-05 Thread Steve Kargl
On Mon, Nov 04, 2013 at 09:58:30PM +0100, Tobias Burnus wrote: > Tobias Burnus wrote: > > Gerald Pfeifer wrote: > >> To make it easier to reproduce builds of software and entire GNU/Linux > >> distributions, RMS had the idea of adding a warning to GCC that warns > >> about the use of __DATE__ and _

Re: [Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-04 Thread Joseph S. Myers
The warning should say "macro" not "Macro" and I think "reproducing" not "reproduce". The c-family and libcpp changes are OK with that fixed. -- Joseph S. Myers jos...@codesourcery.com

[Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-04 Thread Tobias Burnus
Tobias Burnus wrote: Gerald Pfeifer wrote: To make it easier to reproduce builds of software and entire GNU/Linux distributions, RMS had the idea of adding a warning to GCC that warns about the use of __DATE__ and __TIME__. I assume that he also likes to have a warning for __TIMESTAMP__. I w