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

Travis failing on package tests with 0.3.8 release #11132

Closed
dpsanders opened this issue May 5, 2015 · 6 comments · Fixed by travis-ci/travis-build#445
Closed

Travis failing on package tests with 0.3.8 release #11132

dpsanders opened this issue May 5, 2015 · 6 comments · Fixed by travis-ci/travis-build#445

Comments

@dpsanders
Copy link
Contributor

Tests [EDIT: of packages] are passing on 0.4 but failing on 0.3.8, apparently due to the --check-bounds=yes option in the default test script. E.g.
https://travis-ci.org/dpsanders/ValidatedNumerics.jl/jobs/61230482:

Executing the default test script
0.19s$ git fetch --unshallow
remote: Counting objects: 486, done.
remote: Compressing objects: 100% (208/208), done.
remote: Total 486 (delta 279), reused 464 (delta 258), pack-reused 0
Receiving objects: 100% (486/486), 60.00 KiB | 0 bytes/s, done.
Resolving deltas: 100% (279/279), completed with 24 local objects.
10.66s$ julia --color=yes -e 'Pkg.clone(pwd())'
INFO: Initializing package repository /home/travis/.julia/v0.3
INFO: Cloning METADATA from git:https://github.com/JuliaLang/METADATA.jl
INFO: Cloning ValidatedNumerics from /home/travis/build/dpsanders/ValidatedNumerics.jl
INFO: Computing changes...
INFO: Cloning cache of Compat from git:https://github.com/JuliaLang/Compat.jl.git
INFO: Cloning cache of Docile from git:https://github.com/MichaelHatherly/Docile.jl.git
INFO: Installing Compat v0.4.2
INFO: Installing Docile v0.4.11
INFO: Package database updated
3.40s$ julia --color=yes -e "Pkg.build(\"${JL_PKG}\")"
0.73s$ julia --color=yes --check-bounds=yes -e "Pkg.test(\"${JL_PKG}\", coverage=true)"
ERROR: unknown option: --check-bounds=yes
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
The command "julia --color=yes --check-bounds=yes -e "Pkg.test(\"${JL_PKG}\", coverage=true)"" exited with 1.
Done. Your build exited with 1.
@dpsanders dpsanders changed the title Travis failing with 0.3.8 release Travis failing on package tests with 0.3.8 release May 5, 2015
@tkelman
Copy link
Contributor

tkelman commented May 5, 2015

Actually it's a complete coincidence that 0.3.8 was recently released. The reason this is happening is because travis-ci/travis-build#411 was merged recently, and I failed to test sufficiently to remember that command-line arguments are completely hosed on 0.3. Sorry, we should be able to fix it by changing the order of --color=yes and --check-bounds=yes via a PR to travis-build.

tkelman@ygdesk:~/Julia/julia$ julia --color=yes --check-bounds=yes -e "Pkg.test(\"JSON\", coverage=true)"
ERROR: unknown option: --check-bounds=yes
 in process_options at ./client.jl:285
 in _start at ./client.jl:354

tkelman@ygdesk:~/Julia/julia$ julia --check-bounds=yes --color=yes -e "Pkg.test(\"JSON\", coverage=true)"
INFO: Computing test dependencies for JSON...
INFO: No packages to install, update or remove
INFO: Testing JSON
INFO: JSON tests passed
INFO: No packages to install, update or remove

@dpsanders
Copy link
Contributor Author

OK, thanks!

tkelman added a commit to tkelman/travis-build that referenced this issue May 5, 2015
Fixes JuliaLang/julia#11132, regression introduced by merging travis-ci#411

Apparently the order of command line arguments can make a big difference. At least that's been fixed on master.
@tkelman
Copy link
Contributor

tkelman commented May 5, 2015

It's unfortunate that there aren't small pieces of #9482 that could be backported to fix #9384 on release-0.3, since that was a pretty major restructuring.

@tkelman
Copy link
Contributor

tkelman commented May 5, 2015

The fix was already merged. Not sure how long it'll take to deploy, but give this another try, should be working again before long.

@tkelman
Copy link
Contributor

tkelman commented May 5, 2015

Looks like it should be good now. https://travis-ci.org/JuliaLang/ZMQ.jl/jobs/61376643

@tkelman tkelman closed this as completed May 5, 2015
@dpsanders
Copy link
Contributor Author

Yes, back to normal now, thanks!

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 a pull request may close this issue.

2 participants