public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/47494] New: .MOD files: Consider removing the time stamp
@ 2011-01-27 15:56 burnus at gcc dot gnu.org
  2011-01-27 15:58 ` [Bug fortran/47494] " burnus at gcc dot gnu.org
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: burnus at gcc dot gnu.org @ 2011-01-27 15:56 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47494

           Summary: .MOD files: Consider removing the time stamp
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: burnus@gcc.gnu.org


At http://gcc.gnu.org/ml/gcc-patches/2011-01/msg02012.html Simon Baldwin wrote:

Strict build environments do binary comparisons of files across gcc builds.
Writing the current timestamp into a Fortran .mod file leads to false
positives from such environments.  Omitting the timestamp removes these
false positives.

That change was in Google's internal GCC - and is now pushed to the Google
branch in SVN.

One should consider to apply the patch also to the trunk


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug fortran/47494] .MOD files: Consider removing the time stamp
  2011-01-27 15:56 [Bug fortran/47494] New: .MOD files: Consider removing the time stamp burnus at gcc dot gnu.org
@ 2011-01-27 15:58 ` burnus at gcc dot gnu.org
  2011-02-20 11:11 ` jb at gcc dot gnu.org
  2011-11-23 15:52 ` jb at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: burnus at gcc dot gnu.org @ 2011-01-27 15:58 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47494

--- Comment #1 from Tobias Burnus <burnus at gcc dot gnu.org> 2011-01-27 15:50:57 UTC ---
Cf. also http://gcc.gnu.org/ml/fortran/2011-01/msg00250.html


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug fortran/47494] .MOD files: Consider removing the time stamp
  2011-01-27 15:56 [Bug fortran/47494] New: .MOD files: Consider removing the time stamp burnus at gcc dot gnu.org
  2011-01-27 15:58 ` [Bug fortran/47494] " burnus at gcc dot gnu.org
@ 2011-02-20 11:11 ` jb at gcc dot gnu.org
  2011-11-23 15:52 ` jb at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: jb at gcc dot gnu.org @ 2011-02-20 11:11 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47494

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.02.20 09:34:06
                 CC|                            |jb at gcc dot gnu.org
     Ever Confirmed|0                           |1

--- Comment #2 from Janne Blomqvist <jb at gcc dot gnu.org> 2011-02-20 09:34:06 UTC ---
Confirmed.

While AFAICS this is not a problem wrt removing the timestamp, one should make
sure to not break the avoid-recompilation-cascade feature, see e.g. discussion
in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47495


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug fortran/47494] .MOD files: Consider removing the time stamp
  2011-01-27 15:56 [Bug fortran/47494] New: .MOD files: Consider removing the time stamp burnus at gcc dot gnu.org
  2011-01-27 15:58 ` [Bug fortran/47494] " burnus at gcc dot gnu.org
  2011-02-20 11:11 ` jb at gcc dot gnu.org
@ 2011-11-23 15:52 ` jb at gcc dot gnu.org
  2 siblings, 0 replies; 4+ messages in thread
From: jb at gcc dot gnu.org @ 2011-11-23 15:52 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47494

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Janne Blomqvist <jb at gcc dot gnu.org> 2011-11-23 14:42:16 UTC ---
Patch was committed to trunk on 2011-09-20.

According to git annotate, the timestamp has been there since the original
commit on 2004-05-13, so I don't think this is a regression, so no need to
backport to other branches.

Therefore, closing.


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-11-23 14:42 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-27 15:56 [Bug fortran/47494] New: .MOD files: Consider removing the time stamp burnus at gcc dot gnu.org
2011-01-27 15:58 ` [Bug fortran/47494] " burnus at gcc dot gnu.org
2011-02-20 11:11 ` jb at gcc dot gnu.org
2011-11-23 15:52 ` jb at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).