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

Disable arch-native flag when building releases that we distribute (backport #10142) #10144

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jun 24, 2024

We don't want to accidentally create binaries that contain CPU instructions only available in some machines. Hashable has this flag that enables the usage of -march=native for its FFI C code, so we have to disable it explicitly.

The cabal.validate.project file is also concerned because we use it to build the cabal-head binary that we distribute.


This is an automatic backport of pull request #10142 done by Mergify.

…10142)

* Disable arch-native flag when building releases that we distribute

* Use the constraints stanza

* Also disable the flag in cabal.validate.project

(cherry picked from commit 9bb4f58)
@ulysses4ever ulysses4ever added the squash+merge me Tell Mergify Bot to squash-merge label Jun 24, 2024
@geekosaur
Copy link
Collaborator

Uh, why does this want two approvals? It says it's a backport.

@ulysses4ever
Copy link
Collaborator

You mean GitHub? Dunno. Mergify seems to be ok (https://github.com/haskell/cabal/pull/10144/checks?check_run_id=26615884274, the backport/squash rule).

@geekosaur
Copy link
Collaborator

Argh, Windows build glitch.

@mergify mergify bot merged commit 357bbf0 into 3.12 Jun 24, 2024
54 checks passed
@mergify mergify bot deleted the mergify/bp/3.12/pr-10142 branch June 24, 2024 21:07
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days squash+merge me Tell Mergify Bot to squash-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants