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

Drop --c++11 brew flag in build-osx.md #2755

Merged
merged 2 commits into from
Mar 11, 2019
Merged

Drop --c++11 brew flag in build-osx.md #2755

merged 2 commits into from
Mar 11, 2019

Conversation

charlesrocket
Copy link

No description provided.

@UdjinM6
Copy link

UdjinM6 commented Mar 10, 2019

AFAIK -- is the correct way to specify brew formula build options. Why do you think it's a typo?

@charlesrocket
Copy link
Author

charlesrocket commented Mar 10, 2019

its giving me an unkown option error on current osx. runner brew doc and upgrades, then re attempt with same output. my thought was its one of recent brew changes

@UdjinM6
Copy link

UdjinM6 commented Mar 10, 2019

Hmm... indeed. Looks like this was reworked. I think we don't need c++11 at all now.

@charlesrocket
Copy link
Author

@UdjinM6 switch to -c++14 ?

@UdjinM6
Copy link

UdjinM6 commented Mar 10, 2019

No, I mean just drop it completely.

doc/build-osx.md Outdated Show resolved Hide resolved
@UdjinM6 UdjinM6 changed the title Fix Brew flag typo in build-osx.md Drop --c++11 brew flag in build-osx.md Mar 10, 2019
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me (macos 10.14.3).

ACK

@UdjinM6 UdjinM6 modified the milestones: 12.3, 14.0 Mar 10, 2019
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@UdjinM6 UdjinM6 merged commit 7b24f9b into dashpay:develop Mar 11, 2019
@charlesrocket charlesrocket deleted the fix-build-osx-md branch March 11, 2019 08:17
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.

3 participants