Skip to content

Conversation

laanwj
Copy link
Member

@laanwj laanwj commented Jun 9, 2016

Putting the build date in the executable is a practice that has no place in these days, now that deterministic building is increasingly common.

Continues #7732 which did this for the GUI.

@paveljanik
Copy link
Contributor

utACK f6e201d

@maflcko
Copy link
Member

maflcko commented Jun 9, 2016

Concept ACK f6e201d

Putting the build date in the executable is a practice that has no place
in these days, now that deterministic building is increasingly common.

Continues bitcoin#7732 which did this for the GUI.
@laanwj laanwj force-pushed the 2016_06_bye_client_date branch from f6e201d to d096d22 Compare June 9, 2016 11:34
@laanwj
Copy link
Member Author

laanwj commented Jun 9, 2016

Updated the commit to also remove CLIENT_DATE logic from genbuild.sh (thanks @MarcoFalke)

@maflcko
Copy link
Member

maflcko commented Jun 9, 2016

utACK d096d22

@paveljanik
Copy link
Contributor

ACK d096d22

@sipa
Copy link
Member

sipa commented Jun 9, 2016

utACK d096d22

@theuni
Copy link
Member

theuni commented Jun 9, 2016

ut ACK d096d22

@laanwj laanwj merged commit d096d22 into bitcoin:master Jun 10, 2016
laanwj added a commit that referenced this pull request Jun 10, 2016
d096d22 build: Get rid of `CLIENT_DATE` (Wladimir J. van der Laan)
codablock pushed a commit to codablock/dash that referenced this pull request Dec 22, 2017
d096d22 build: Get rid of `CLIENT_DATE` (Wladimir J. van der Laan)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
d096d22 build: Get rid of `CLIENT_DATE` (Wladimir J. van der Laan)
Fuzzbawls added a commit to PIVX-Project/PIVX that referenced this pull request Sep 2, 2021
55ec1ee build: Get rid of `CLIENT_DATE` (Wladimir J. van der Laan)
b22dc99 [Qt] Debug window: remove "Build date". (furszy)
4a18d50 build: Remove src/obj directory from repository (Wladimir J. van der Laan)

Pull request description:

  >Putting the build date in the executable is a practice that has no place in these days, now that deterministic building is increasingly common.

  Back porting:
  * bitcoin#7732.
  * bitcoin#8181.

ACKs for top commit:
  random-zebra:
    utACK 55ec1ee
  Fuzzbawls:
    ACK 55ec1ee

Tree-SHA512: 21f1ca282fa55777e611644235f1ec8e5621dc7785c58a53a79470445dc458a19a460a4c52c9dae78f0c7cf16d4095bcc908c34e7dd305b8a1c56881d83e5d67
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants