Skip to content

ldapvi: switch to openssl@3#134550

Merged
BrewTestBot merged 2 commits into
Homebrew:openssl-migration-stagingfrom
chenrui333:ldapvi-openssl3
Jun 22, 2023
Merged

ldapvi: switch to openssl@3#134550
BrewTestBot merged 2 commits into
Homebrew:openssl-migration-stagingfrom
chenrui333:ldapvi-openssl3

Conversation

@chenrui333

Copy link
Copy Markdown
Member

See #134251.

@chenrui333 chenrui333 added openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch. labels Jun 22, 2023
Signed-off-by: Rui Chen <rui@chenrui.dev>
Comment thread Formula/ldapvi.rb

def install
# Workaround for Xcode 14.3.
ENV.append_to_cflags "-Wno-implicit-function-declaration"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
ENV.append_to_cflags "-Wno-implicit-function-declaration"
ENV.append_to_cflags "-Wno-implicit-function-declaration" if DevelopmentTools.clang_build_version == 1403

This should technically be reported and fixed upstream, so we should do something like this so it nudges us to deal with it properly later.

@github-actions

Copy link
Copy Markdown
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions Bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jun 22, 2023
@BrewTestBot BrewTestBot enabled auto-merge June 22, 2023 11:28
@BrewTestBot BrewTestBot merged commit be1de13 into Homebrew:openssl-migration-staging Jun 22, 2023
@chenrui333 chenrui333 deleted the ldapvi-openssl3 branch January 22, 2024 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants