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

chore: use flags for apple arm64 #3213

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ion-elgreco
Copy link
Collaborator

@ion-elgreco ion-elgreco commented Feb 13, 2025

Description

These flags were missing causing my compilations to not being incremental anymore.

Combined with this I still have to provide explicit flags in the compilation:

RUSTFLAGS="-C link-arg=-undefined -C link-arg=dynamic_lookup" uvx maturin develop

Created an issue for this explicitness during maturin develop here: PyO3/maturin#2479

@ion-elgreco ion-elgreco force-pushed the chore/apple-sillicon-flags branch from 193eec2 to 6fe298f Compare February 13, 2025 12:02
Signed-off-by: Ion Koutsouris <[email protected]>
@ion-elgreco ion-elgreco force-pushed the chore/apple-sillicon-flags branch from 6fe298f to 99e4053 Compare February 13, 2025 12:03
@ion-elgreco ion-elgreco changed the title chore: use arm flags for apple arm64 chore: use flags for apple arm64 Feb 13, 2025
Copy link

codecov bot commented Feb 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.10%. Comparing base (b3efdfc) to head (99e4053).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3213   +/-   ##
=======================================
  Coverage   72.10%   72.10%           
=======================================
  Files         138      138           
  Lines       45320    45320           
  Branches    45320    45320           
=======================================
+ Hits        32678    32680    +2     
+ Misses      10567    10562    -5     
- Partials     2075     2078    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

1 participant