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

Cairo tests fail on 32-bit Appveyor #356

Open
simonbyrne opened this issue Feb 26, 2018 · 6 comments
Open

Cairo tests fail on 32-bit Appveyor #356

simonbyrne opened this issue Feb 26, 2018 · 6 comments

Comments

@simonbyrne
Copy link
Contributor

Apparently have been broken for 9 months https://ci.appveyor.com/project/nalimilan/bindeps-jl/build/1.0.222

The first failure was on #300: that doesn't seem related, so it was probably a change with Cairo.jl (which doesn't have 32-bit tests enabled).

Either we should fix Cairo.jl, or remove it from the tests.

@simonbyrne
Copy link
Contributor Author

Looks like some gcc issue: JuliaGraphics/Cairo.jl#196

The easiest option at this stage is to move Cairo to binaryprovider.jl, and get rid of the test here.

@tkelman
Copy link
Contributor

tkelman commented Feb 27, 2018

The point of the test is to exercise code paths here, so if the upstream package changes to no longer use BinDeps then the test dependency should be upper-bounded.
Cairo can probably be skipped on win32 for now but it should be replaced by a different WinRPM-using package that does work.

@lobingera
Copy link
Contributor

@simonbyrne Do you have experiences to move libraries to binaryprovider?

@simonbyrne
Copy link
Contributor Author

No, unfortunately. Ask on the JuliaLang Slack #bindeps2 channel

@lobingera
Copy link
Contributor

lobingera commented Mar 5, 2018

easiest option

?

@simonbyrne
Copy link
Contributor Author

Well, I don't understand BinDeps.jl either. 😄

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

No branches or pull requests

3 participants