timholy / AffineTransforms.jl

Computational geometry with affine transformations
Other
20 stars 6 forks source link

PkgEval reports v0.0.1 broken on Julia 0.2 #1

Closed IainNZ closed 10 years ago

IainNZ commented 10 years ago

(not sure if it matters much at this point!)

timholy commented 10 years ago

That's odd, because the juliarelease build on Travis worked (https://travis-ci.org/timholy/AffineTransforms.jl/jobs/31544090, although perhaps only I have permissions to see it).

Anyway, I just pushed something that might help.

IainNZ commented 10 years ago

I don't have any explanation...

timholy commented 10 years ago

It almost seems like they must be running different notions of juliarelease.

IainNZ commented 10 years ago

I've been using

[idunning@che julia02]$ ./julia -e 'versioninfo()'
Julia Version 0.2.1+2
Commit 677086a (2014-02-14 06:30 UTC)
Platform Info:
  System: Linux (x86_64-unknown-linux-gnu)
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
  LAPACK: libopenblas
  LIBM: libopenlibm

pretty much since the start of testing

IainNZ commented 10 years ago

https://github.com/JuliaLang/julia/compare/release-0.2 I'm seeing a lot of commits past kmsquires VERSION 0.2.1 commit...

IainNZ commented 10 years ago

In fact, the commit I have there is actually behind even that (which is the commit that is tagged as 0.2.1)

IainNZ commented 10 years ago

I can't see a commit that'd make the difference here though. I'm just gonna pretend this didn't happen and hopefully I'll never have to think about it again in a week or two :D

IainNZ commented 10 years ago

Crushingly @test_throws DimensionMismatch AffineTransforms.tformfwd(tfm1, zeros(nd+1)) doesn't work on 0.2 either, which I can't explain why you didn't see on Travis either

timholy commented 10 years ago

Good catch. Hopefully fixed now.