RFR: 8288001: SOURCE_DATE_EPOCH should not be set by default

[Erik Joelsson]

On Wed, 8 Jun 2022 07:47:24 GMT, KIRIYAMA Takuya <duke at openjdk.java.net> wrote:

> At default configuration, SOURCE_DATE_EPOCH is exported as environment variable in SetupReproducibleBuild. Then, gcc is affected of SOURCE_DATE_EPOCH environment variable. This value is used only to set SOURCE_DATE_ISO_8601 (except below), so I removed "export" for SOURCE_DATE_EPOCH in SetupReproducibleBuild. And, at building ct.sym, SOURCE_DATE_EPOCH environment variable is needed. So I added setting routine if SOURCE_DATE_EPOCH is not set.
> 
> This fix works fine. With default configuration shows -Xinternalversion output same as Windows, and with --with-source-date configuration shows -Xinternalversion output specified timestamp. Would you please review this fix?

Is the problem that on Windows we get -Xinternalversion in local timezone but on Linux we get UTC? That's a difference that may warrant fixing, but I don't think this is the way to do it. 

I don't think it makes a practical difference if the timestamp is taken at the start of the build or when a certain src file is compiled.

-------------

PR: https://git.openjdk.org/jdk/pull/9081