-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Build failing in nightly Julia #14883
Comments
it looks like there a type-inference regression on the comprehension at: https://github.com/timholy/Images.jl/blob/15de1ee0fbea02077b0d9c061780ba2b6ec6194c/src/algorithms.jl#L744 |
Maybe related: 6396218#commitcomment-15790959 |
@juliohm Do you have any idea of when this started to fail? |
@nalimilan it must be a very recent change, I pushed to GitHub last weekend and the build was successful. |
This will be fixed by #7258 |
Also see #13412 (comment) |
Thank you @JeffBezanson, I understood that the issue will be solved after you merge those changes and nothing is required from my side as a package developer. |
@JeffBezanson could you please confirm the build error I am having with nightly is not related? https://travis-ci.org/juliohm/ImageQuilting.jl/jobs/147008026 The issue #7258 you mentioned is now closed. |
|
Hi Tony,
I'll double check my package to see where I might be using Requires.jl. I
don't remember using it, just turned my computer off. Tomorrow I will check.
…-Júlio
|
@tkelman I cannot find where in my package the |
LightGraphs used to, but the most recent tag doesn't any more. So try again after a |
Thank you Tony, will do.
…-Júlio
|
This issue was solved somewhere, closing it. |
Can anyone help me track the changes in Julia that caused this build failure?
https://travis-ci.org/juliohm/ImageQuilting.jl
I didn't modify anything relevant in my latest push to GitHub. There are many warnings and an error in nightly Julia.
The text was updated successfully, but these errors were encountered: