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

gtk+-3: Added poppler as required dep #810

Closed
wants to merge 1 commit into from
Closed

gtk+-3: Added poppler as required dep #810

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 30, 2017

No description provided.

@ghost
Copy link
Author

ghost commented Jan 30, 2017

I've had enough :|

@ghost ghost closed this Jan 30, 2017
@v4hn
Copy link
Member

v4hn commented Jan 31, 2017

@bsfc1685 I see why, you've been fighting with this for a while now..

@Ratler looks like this failed because the server didn't rebuild cairo for the pull-request and the cairo installation used for testing the request does not have pdf support enabled (the default for this dep changed in #810 ). Could you have a look please?

I suppose this was the problem all along, so I don't think we should merge this request.
@bsfc1685 the best way we have to handle this at the moment is something like this.
Feel free to add a new request that does that for the cairo->poppler dependency.
Support for depending on optional dependencies of other packages has been on the list of wanted features for some years actually..

This pull request was closed.
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.

1 participant