Skip to content

arangodb: switch to openssl@3#134350

Closed
p-linnane wants to merge 1 commit into
Homebrew:masterfrom
p-linnane:arangodb-openssl3
Closed

arangodb: switch to openssl@3#134350
p-linnane wants to merge 1 commit into
Homebrew:masterfrom
p-linnane:arangodb-openssl3

Conversation

@p-linnane

Copy link
Copy Markdown
Member

See #134251.

@p-linnane p-linnane added openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch. labels Jun 21, 2023
@github-actions github-actions Bot added go Go use is a significant feature of the PR or issue long build Set a long timeout for formula testing labels Jun 21, 2023
@p-linnane p-linnane added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Jun 21, 2023
@github-actions github-actions Bot removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Jun 21, 2023
@Bo98

Bo98 commented Jun 21, 2023

Copy link
Copy Markdown
Member

cc1: error: bad value ('merom') for '-march=' switch

Our -march filtering is probably messing it up - the CMake script is supposed to check if -march=merom is available and falls back to core2 if not. We're also building an old version here so I'm guessing this issue has impacted updating the formula.

3.11.x totally redos the architecture stuff so if we update to that will require some care.

@p-linnane

Copy link
Copy Markdown
Member Author

Yeah, I was unable to get us bumped to 3.11.1 a few days ago: #134010

@cho-m

cho-m commented Jun 23, 2023

Copy link
Copy Markdown
Member

Could try core instead though we will lose some optimization (SSSE3).

Part of reason we haven't been able to update ArangoDB is that they only support specific Clang/GCC versions, which is why we aren't able to use default compilers.

Looks like new LLVM/Clang (16 ?) behavior is causing build failure for update to 3.11.x.

@p-linnane p-linnane changed the base branch from openssl-migration-staging to master June 26, 2023 23:59
@p-linnane p-linnane removed the staging-branch-pr This pull request targets a *-staging branch. label Jun 26, 2023
@github-actions github-actions Bot added automerge-skip `brew pr-automerge` will skip this pull request autosquash Automatically squash pull request commits according to Homebrew style. labels Jun 26, 2023
@p-linnane p-linnane removed automerge-skip `brew pr-automerge` will skip this pull request autosquash Automatically squash pull request commits according to Homebrew style. labels Jun 27, 2023
@carlocab

carlocab commented Jul 1, 2023

Copy link
Copy Markdown
Member

Superseded by #135487.

@carlocab carlocab closed this Jul 1, 2023
@carlocab carlocab added the superseded PR was replaced by another PR label Jul 1, 2023
@p-linnane p-linnane deleted the arangodb-openssl3 branch July 1, 2023 16:16
@github-actions github-actions Bot added the outdated PR was locked due to age label Aug 1, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

go Go use is a significant feature of the PR or issue long build Set a long timeout for formula testing openssl-3-migration Related to switching to an OpenSSL 3 dependency outdated PR was locked due to age superseded PR was replaced by another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants