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

[PkgEval] ICU may have a testing issue on Julia 0.4 (2014-09-26) #20

Closed
IainNZ opened this issue Sep 26, 2014 · 4 comments · Fixed by #21
Closed

[PkgEval] ICU may have a testing issue on Julia 0.4 (2014-09-26) #20

IainNZ opened this issue Sep 26, 2014 · 4 comments · Fixed by #21

Comments

@IainNZ
Copy link

IainNZ commented Sep 26, 2014

PackageEvaluator.jl is a script that runs nightly. It attempts to load all Julia packages and run their tests (if available) on both the stable version of Julia (0.3) and the nightly build of the unstable version (0.4). The results of this script are used to generate a package listing enhanced with testing results.

On Julia 0.4

  • On 2014-09-25 the testing status was No tests, but package loads.
  • On 2014-09-26 the testing status changed to Package doesn't load.

No tests, but package loads. means that PackageEvaluator did not find tests for your package. However, trying to load your package with using worked.

Package doesn't load. means that PackageEvaluator did not find tests for your package. Additionally, trying to load your package with using failed.

This issue was filed because your testing status became worse. No additional issues will be filed if your package remains in this state, and no issue will be filed if it improves. If you'd like to opt-out of these status-change messages, reply to this message saying you'd like to and @IainNZ will add an exception. If you'd like to discuss PackageEvaluator.jl please file an issue at the repository. For example, your package may be untestable on the test machine due to a dependency - an exception can be added.

Test log:

>>> 'Pkg.add("ICU")' log
INFO: Installing BinDeps v0.3.5
INFO: Installing ICU v0.4.2
INFO: Installing SHA v0.0.3
INFO: Installing URIParser v0.0.3
INFO: Building ICU
INFO: Package database updated

>>> 'using ICU' log
ERROR: `set_locale` has no method matching set_locale(::Ptr{Void})
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
 in reload_path at loading.jl:152
 in _require at loading.jl:67
 in require at loading.jl:51
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
 in _start_3B_3605 at /home/idunning/julia04/usr/bin/../lib/julia/sys.so
while loading /home/idunning/pkgtest/.julia/v0.4/ICU/src/ICU.jl, in expression starting on line 638
while loading /home/idunning/pkgtest/.julia/v0.4/ICU/testusing.jl, in expression starting on line 2
Julia Version 0.4.0-dev+789
Commit 98c32a3 (2014-09-26 06:09 UTC)
Platform Info:
  System: Linux (x86_64-unknown-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2650 0 @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3


>>> test log
no tests to run
>>> end of log
@JeffBezanson
Copy link
Member

Should now use Ptr{Void} instead of Ptr{None}.

@IainNZ
Copy link
Author

IainNZ commented Sep 26, 2014

Ah of course!

@IainNZ
Copy link
Author

IainNZ commented Sep 26, 2014

Whats the right way to fix this, something like

if julia0.4
  const None = Void
end

?

@JeffBezanson
Copy link
Member

Just using Ptr{Void} everywhere should work in both versions.

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