CI: A temporary workaround to pass the macos-latest trufferuby-head case.#652
Closed
junaruga wants to merge 1 commit intoruby:masterfrom
Closed
CI: A temporary workaround to pass the macos-latest trufferuby-head case.#652junaruga wants to merge 1 commit intoruby:masterfrom
junaruga wants to merge 1 commit intoruby:masterfrom
Conversation
This was referenced Jul 21, 2023
…ase. Remove openssl@3 brew package's include directory as a temporary workaround to avoid an error by the -Wmacro-redefined compiler warning in the process of compiling the Ruby OpenSSL binding in the macos-latest trufferuby-head CI case.
junaruga
force-pushed
the
wip/truffleruby-head-warning-macro-redfined
branch
from
c7416ac to
832ad91
Compare
Member
|
Let's use #653 instead. Thank you for reporting the issue. |
Member
Author
|
Sure. I am okay for that. Thanks for your PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a workaround for the #650. It seems that the setting of the
openssl@3's include directory is unintentionally added to the compiler argument-I<include dir>. And that causes the compiler error in the process of the Ruby OpenSSL binding.Remove openssl@3 brew package's include directory as a temporary workaround to avoid an error by the -Wmacro-redefined compiler warning in the process of compiling the Ruby OpenSSL binding in the macos-latest trufferuby-head CI case.