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

minizip.pc: Ensure variables point to an absolute path. #590

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

besser82
Copy link
Contributor

With commit 503bfd5 the CMake _IMPORT_PREFIX has been changed to relative paths. This breaks the paths reflected by the auto-generated pkgconfig file, as pkgconfig expects those paths to be absolute.


Closes #584.

@nmoinvaz nmoinvaz changed the base branch from master to dev August 31, 2021 20:09
@nmoinvaz
Copy link
Member

Idk why but it says it needs rebased. I tried to fix it but it didn't work. If you can squash and rebase everything thanks. Sorry I didn't get to your PR first.

@besser82 besser82 changed the base branch from dev to master September 1, 2021 07:35
@besser82 besser82 changed the base branch from master to dev September 1, 2021 07:35
With commit 503bfd5 the CMake _IMPORT_PREFIX has been
changed to relative paths.  This breaks the paths reflected
by the auto-generated pkgconfig file, as pkgconfig expects
those paths to be absolute.

Signed-off-by: Björn Esser <[email protected]>
@besser82
Copy link
Contributor Author

besser82 commented Sep 1, 2021

Idk why but it says it needs rebased. I tried to fix it but it didn't work. If you can squash and rebase everything thanks. Sorry I didn't get to your PR first.

No problem! I've rebased my commit onto the dev branch, and it looks like the previously merged PR has not covered the needed fix toincludedir. You can merge now. =)

@nmoinvaz nmoinvaz merged commit 6390942 into zlib-ng:dev Sep 1, 2021
@besser82 besser82 deleted the topic/besser82/fix_pkgconfig branch September 1, 2021 18:32
@besser82
Copy link
Contributor Author

besser82 commented Sep 1, 2021

Thank you!

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.

2 participants