Skip to content

fix enzyme library used for build #150

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
Aug 2, 2024
Merged

fix enzyme library used for build #150

merged 5 commits into from
Aug 2, 2024

Conversation

tgymnich
Copy link
Member

@tgymnich tgymnich commented Aug 2, 2024

Use shared lib instead of module which only works on linux.

@tgymnich tgymnich requested a review from ZuseZ4 August 2, 2024 01:01
@ZuseZ4
Copy link
Collaborator

ZuseZ4 commented Aug 2, 2024

It fails Rust linux CI, but I guess this name is only to be used on mac, right? @marcpabst can you confirm this config works for you too?

@tgymnich
Copy link
Member Author

tgymnich commented Aug 2, 2024

The name should also work just fine on linux

@tgymnich
Copy link
Member Author

tgymnich commented Aug 2, 2024

@ZuseZ4 found some more occurrences of the wrong lib name. We are failing at the test stage now.

@tgymnich tgymnich marked this pull request as ready for review August 2, 2024 16:45
@ZuseZ4 ZuseZ4 merged commit 512c484 into master Aug 2, 2024
8 of 12 checks passed
@ZuseZ4 ZuseZ4 deleted the fix-lib branch August 2, 2024 17:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants