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

Update Perl dependencies #561

Closed
wants to merge 1 commit into from
Closed

Update Perl dependencies #561

wants to merge 1 commit into from

Conversation

lopsided98
Copy link
Contributor

Add missing Perl dependencies. This change is copied from NixOS/nixpkgs#39314

Add missing Perl dependencies
@dtzWill
Copy link
Member

dtzWill commented May 21, 2018

👍 thanks!

Adding these shouldn't cause problems when building against "older" nixpkgs right, since it's a matter of explicitly listing existing dependencies already used (instead of assuming they're pulled in implicitly by other things)? (similar to include-what-you-use re:headers and such)

@lopsided98
Copy link
Contributor Author

lopsided98 commented May 21, 2018

Both CatalystXRoleApplicator and JSONAny are direct dependencies of Hydra and were added to nixpkgs in 2013. I assume something else used to depend on them and pull them in implicitly, but no longer does.

@lopsided98
Copy link
Contributor Author

Already done by 3ef253c.

@lopsided98 lopsided98 closed this Apr 1, 2019
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.

None yet

2 participants