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

remove PkgConfig from runtime requires #82

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@bjakubski
Contributor

bjakubski commented Feb 28, 2015

I barely understand interactions that happen between dist, meta files and tools that use them so I might not know something, but PkgConfig should only be in 'configure_requires' and not in 'requires', as it is not a runtime dependency

@ambs

This comment has been minimized.

Show comment
Hide comment
@ambs

ambs Feb 28, 2015

Collaborator

Indeed. Thanks for the PR. will merge later today, hopefully. Feel free to poke if I do not.

Collaborator

ambs commented Feb 28, 2015

Indeed. Thanks for the PR. will merge later today, hopefully. Feel free to poke if I do not.

@bjakubski

This comment has been minimized.

Show comment
Hide comment
@bjakubski

bjakubski Mar 4, 2015

Contributor

Will recreate PR with correct username in commits

Contributor

bjakubski commented Mar 4, 2015

Will recreate PR with correct username in commits

@bjakubski bjakubski closed this Mar 4, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment