Skip to content

Commit

Permalink
Fix appveyor
Browse files Browse the repository at this point in the history
  • Loading branch information
blegat committed Oct 6, 2016
1 parent a8b7250 commit 5c51e8c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions appveyor.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
environment:
matrix:
- JULIAVERSION: "julialang/bin/winnt/x86/0.4/julia-0.4-latest-win32.exe"
- JULIAVERSION: "julialang/bin/winnt/x64/0.4/julia-0.4-latest-win64.exe"
- JULIAVERSION: "julialang/bin/winnt/x86/0.5/julia-0.5-latest-win32.exe"
- JULIAVERSION: "julialang/bin/winnt/x64/0.5/julia-0.5-latest-win64.exe"
- JULIAVERSION: "julianightlies/bin/winnt/x86/julia-latest-win32.exe"
- JULIAVERSION: "julianightlies/bin/winnt/x64/julia-latest-win64.exe"

Expand All @@ -28,7 +28,7 @@ build_script:
# Need to convert from shallow to complete for Pkg.clone to work
- IF EXIST .git\shallow (git fetch --unshallow)
- C:\projects\julia\bin\julia -e "versioninfo();
Pkg.clone(pwd(), \"Polyhedra\"); Pkg.clone(pwd(), \"CDDLib\"); Pkg.build(\"CDDLib\")"
Pkg.clone(\"https://github.com/blegat/Polyhedra.jl\"); Pkg.clone(pwd(), \"CDDLib\"); Pkg.build(\"CDDLib\")"

This comment has been minimized.

Copy link
@tkelman

tkelman Oct 8, 2016

Hoping this is temporary? Does this mean the most recent version of CDDLib will only work with the most recent version of Polyhedra? If so, best to declare minimum version requirements in REQUIRE (and possibly also maximum version requirements on past tagged versions, when you release something new that would break the older releases).

This comment has been minimized.

Copy link
@blegat

blegat Oct 9, 2016

Author Member

Yes this was temporary I should have tagged Polyhedra first and then removed this before tagging CDDLib. I will fix the REQUIRE file for versions of CDDLib


test_script:
- C:\projects\julia\bin\julia --check-bounds=yes -e "Pkg.test(\"CDDLib\")"

0 comments on commit 5c51e8c

Please sign in to comment.