-
Notifications
You must be signed in to change notification settings - Fork 38
Update Travis to include 0.5, don't override default script #143
Conversation
@Scidom Well that was definitely the fasted label application I've ever seen! 😄 |
Hi @ararslan, thanks for opening this PR; the label was added automatically, I didn't intervene. I will merge once I manage to ensure that Lora works with Julia v0.5. For now, I tried to install Lora with Julia v0.5 and I got a build error in Rmath.jl (actually BinDeps.jl), so I have to wait till this is fixed:
Furthermore, once the above error is sorted out, I have to ensure that this JuliaLang/julia#15068 issue has been resolved before I start using Lora with Julia v0.5. I mentioned this issue nearly 4 months ago, but not sure if it is going to be taken care of anytime soon, so Lora may be stack with Julia v0.4 for a while. |
That's weird. What version of BinDeps are you using? Can you |
I think the Rmath issue is specific to your machine, that works well for me. Isn't #15068 closed by #17487? If not, you should really complain over at #15068. In any case, enabling travis builds on julia 0.5 seems the right thing to do at this moment so that we can start to fix any remaining problems, right? |
Ok, I will merge this PR, and will follow up with Rmath. |
I forgot to mention @ararslan that the building of Rmath after Pkg.update() didn't work, nor I expected it to work, because I tried it yest in a fresh installation anyhow. |
Yeah, make sure to follow up with Rmath. That is now one of the packages that really needs to work on people's machine because without it the whole stats stack is broken (they moved stuff out of Base into Rmath). So really high priority that the Rmath maintainers receive reports of any bugs there. |
Thanks @davidanthoff, I will follow up with this. I am immersed into a research project, hoping to get to the bottom of it in the next week or so and then resume a more normal pace with Julia matters. |
@davidanthoff apologies for the slowness, I can now confirm that Rmath builds just fine on my mac (Julia v0.5). |
Julia 0.5 is now a release. Since the REQUIRE doesn't have an upper bound on the required Julia version for installation, we should be running CI on the 0.5 release and on the nightlies. It's also better not to override the default test script unless there's a particular reason to do so, so I commented that part out.