Skip to content

Project/package fixes #4

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Project/package fixes #4

wants to merge 2 commits into from

Conversation

@goetzf
Copy link

@goetzf goetzf commented May 15, 2025

  • Fixed some file references in the project.
  • Fixed the header search path in the package.
  • Added the linker flag -fprofile-instr-generate to ensure the Swift package builds.

@goetzf goetzf force-pushed the bugfix/project-fixes branch from 5b73c2a to 83fe1ba Compare May 16, 2025 06:25
@macguru
Copy link
Member

macguru commented May 16, 2025

-fprofile-instr-generate should not be necessary, it only relates to code coverage.

@goetzf goetzf force-pushed the bugfix/project-fixes branch from 83fe1ba to 5012500 Compare May 16, 2025 13:55
@goetzf
Copy link
Author

goetzf commented May 16, 2025

-fprofile-instr-generate should not be necessary, it only relates to code coverage.

I’ve removed this change again, since it’s not necessary anymore.

@goetzf goetzf self-assigned this May 16, 2025
@goetzf goetzf requested a review from mangerlahn May 16, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mangerlahn mangerlahn Awaiting requested review from mangerlahn

Assignees

@goetzf goetzf

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@goetzf @macguru