-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 poppunk to 2.1.1 #23350
Update poppunk to 2.1.1 #23350
Conversation
@bioconda/core My build succeeds, but then I am getting the pasted error in the run/test stage. It seems like the installation of gtk3 is erroring, which I guess is pulled in my matplotlib-base. Have you seen this before, or have clues as to why this might be happening?
|
Key line I think is: |
@bioconda/core To get this to pass I needed to add all of the xorg recipes to get |
(busy atm. Will comment later, sorry, in the meantime relevant references below) |
I guess we can't have a |
Hi, I am sorry haven't explained in detail/at all. |
Hello from the conda-forge Probably no one here knows, but how does bioconda deal with packages that depend on Qt? The Qt package has been around a lot longer uses CDTs for X11, so I'd have thought that this problem would have come up in the test phase of some Qt-depending package. Maybe there are none or they are not run in the test phase. For this specific issue, assuming that a user's system will have X11 installed, you can probably make this work in the test phase by adding all of those |
I've dealt with the "something needs X11" issue with SINA by breaking the What's the story with matplot-base? If it functions without X11, it should be split to break dependency tree. Not sure how many packages use this. |
Hi, wasn't able to get to this on the weekend. A special case then was Now, what is new with the change in the build container image, is that the builds may actually fail because either the build itself or the test in that container encounter the missing libraries. To fix that, we'd have to re-introduce those libraries in the container. In sum: To really fix the issue and make things reasonably comfortable for the packager, we have to adjust Bioconda's build system and look into a |
We have
|
So, this means this uses libraries that are not even available in the |
Thanks for all the detailed comments from everyone. One thing I figured out, and hope I didn't confuse by, is that the error here was caused by adding the I did get this to pass in 33c2202 by adding the xorg dependencies. Shall I re-add these, or do you have a better solution (I'm sorry, I wasn't able to follow all of the discussion above) |
@johnlees, I didn't check which Xorg packages were needed before. Hence I assumed those were just the once that are provided by
But since we need other |
@BiocondaBot please merge |
I will attempt to upload artifacts and merge this PR. This may take some time, please have patience. |
Update
poppunk
: 2.0.2 → 2.1.1recipes/poppunk
(click to view/edit other files)This pull request was automatically generated (see docs).