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

doc: Update OpenBSD build instructions for 6.2 (cont'd) #11984

Merged
merged 1 commit into from Dec 30, 2017

Conversation

laanwj
Copy link
Member

@laanwj laanwj commented Dec 22, 2017

(this continues #11442)
There is no more need to install a new compiler. This simplifies instructions a lot.

From discussion with @fanquake on IRC I first wanted to add a new section for 6.2, but that made the document a complex mess. I think it's good enough (and more maintainable too) to only support the most recent release.

Includes #11976.

I moved the "resource limits" section to the end as I didn't seem to need it with clang, but this may vary based on source changes and the phase of the moon so it's good to keep it as optional extra information.

@laanwj laanwj changed the title doc: Update OpenBSD build instructions for 6.2 doc: Update OpenBSD build instructions for 6.2 (cont'd) Dec 22, 2017
@laanwj laanwj force-pushed the 2017_12_openbsd_build_update branch from 2d6fe9f to 4f3a871 Compare December 22, 2017 09:01
There is no more need to install a compiler. This simplifies
instructions a lot.
@laanwj laanwj force-pushed the 2017_12_openbsd_build_update branch from 4f3a871 to 6915f93 Compare December 22, 2017 09:09
@fanquake
Copy link
Member

tested ACK 6915f93 with VirtualBox on OS X.
The phase of the moon has no affect down under.

@janstary
Copy link

Ah, I have my own branch wiht updates for the OpenBSD build.
Should have checked this first. I will put up my PR and link it here.

@laanwj laanwj merged commit 6915f93 into bitcoin:master Dec 30, 2017
laanwj added a commit that referenced this pull request Dec 30, 2017
6915f93 doc: Update OpenBSD build instructions for 6.2 (Wladimir J. van der Laan)

Pull request description:

  (this continues #11442)
  There is no more need to install a new compiler. This simplifies instructions a lot.

  From discussion with @fanquake on IRC I first wanted to add a new section for 6.2, but that made the document a complex mess. I think it's good enough (and more maintainable too) to only support the most recent release.

  Includes #11976.

  I moved the "resource limits" section to the end as I didn't seem to need it with clang, but this may vary based on source changes and the phase of the moon so it's good to keep it as optional extra information.

Tree-SHA512: 15794afec6d682323d0aa13c7616d009acb7fce8b0ef5d2106261f2ebd86b7b2fe66040c04860d9bf2f0c1934fbdc2b594b8c09a98accfaac04f3daf9a6cadf3
@laanwj
Copy link
Member Author

laanwj commented Dec 30, 2017

It's not really great that everyone is opening their own PR instead of helping test existing ones. As @fanquake tested here will merge this one so the others can be rebased on top.

@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.

None yet

3 participants