Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Merge #620: Install headers automatically" #626

Conversation

doitian
Copy link
Contributor

@doitian doitian commented May 24, 2019

This reverts commit 91fae3a, reversing
changes made to 5df77a0.

See discussion in #625

After the change, if we enable any module, make install fails because of the
duplicated files in the command line arguments.

Closes #624

This reverts commit 91fae3a, reversing
changes made to 5df77a0.

See discussion in bitcoin-core#625

After the change, if we enable any module, `make install` fails because of the
duplicated files in the command line arguments.
@gmaxwell
Copy link
Contributor

ACK. (following the principle of revert errors eagerly and favor the status quo, if it turns out I'm missing some reason why this wasn't the best solution-- that's fine. We could switch to another solution later).

@gmaxwell gmaxwell merged commit 662918c into bitcoin-core:master May 24, 2019
gmaxwell added a commit that referenced this pull request May 24, 2019
662918c Revert "Merge #620: Install headers automatically" (ian)

Pull request description:

  This reverts commit 91fae3a, reversing
  changes made to 5df77a0.

  See discussion in #625

  After the change, if we enable any module, `make install` fails because of the
  duplicated files in the command line arguments.

  Closes #624

Tree-SHA512: 7769ede527ac307bff094603e5974c56b19e41bc2ef55113173d3dbc8e545d6add7ae044753fa0361595e5e7a746d6c8d641f98caa3381b683aa2b61a1742097
@doitian doitian deleted the fix-installing-duplicated-headers-by-revert branch May 24, 2019 02:06
@doitian doitian restored the fix-installing-duplicated-headers-by-revert branch May 24, 2019 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make install fails
2 participants