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

Add Julia 1.0 compatibility changes. #158

Merged
merged 5 commits into from
Aug 28, 2018

Conversation

hessammehr
Copy link
Contributor

Basically change warn to @warn and migrate iterators to the new format.

Basically change `warn` to `@warn`, `info` to `@info` and migrate iterators to the
new format.
@musm
Copy link
Contributor

musm commented Aug 22, 2018

LGTM, someone should also release a tagged version after this PR is merged.

@davidanthoff
Copy link
Collaborator

Can someone merge this and tag?

@musm
Copy link
Contributor

musm commented Aug 26, 2018

cc @ararslan @nalimilan @tkelman

@musm
Copy link
Contributor

musm commented Aug 26, 2018

basically any packages that depend on Winrpm, like hdf5, do not build on Julia 1.0

Copy link
Member

@nalimilan nalimilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but can you confirm you tested it locally and it works?

@hessammehr
Copy link
Contributor Author

hessammehr commented Aug 26, 2018

@nalimilan WinRPM.update and WinRPM.install work fine on my computer. I will try and open a PR for a test suite soon, but in the mean time this will allow Julia 1.0 users to start using packages like HDF5.jl on Windows.

@musm
Copy link
Contributor

musm commented Aug 27, 2018

(v1.0) pkg> build HDF5
  Building LibCURL ─ `C:\Users\Mus\.julia\packages\LibCURL\Qs5og\deps\build.log`
  Building WinRPM ── `C:\Users\Mus\.julia\dev\WinRPM\deps\build.log`
 Resolving package versions...
┌ Error: Error building `WinRPM`:
│ ERROR: LoadError: InitError: "ErrorException(\"Error parsing document : 0\"), syntax error, 1, 1, 1"
│ Stacktrace:
│  [1] error(::String) at .\error.jl:33
│  [2] xp_parse(::String) at C:\Users\Mus\.julia\packages\LibExpat\6jLDP\src\LibExpat.jl:240
│  [3] update(::Bool, ::Bool) at C:\Users\Mus\.julia\dev\WinRPM\src\WinRPM.jl:167
│  [4] __init__() at C:\Users\Mus\.julia\dev\WinRPM\src\WinRPM.jl:46
│  [5] _include_from_serialized(::String, ::Array{Any,1}) at .\loading.jl:627
│  [6] _require_from_serialized(::String) at .\loading.jl:678
│  [7] _require(::Base.PkgId) at .\logging.jl:317
│  [8] require(::Base.PkgId) at .\loading.jl:852
│  [9] macro expansion at .\logging.jl:311 [inlined]
│  [10] require(::Module, ::Symbol) at .\loading.jl:834
│  [11] top-level scope at C:\Users\Mus\.julia\dev\WinRPM\deps\build.jl:2
│  [12] include at .\boot.jl:317 [inlined]
│  [13] include_relative(::Module, ::String) at .\loading.jl:1038
│  [14] include(::Module, ::String) at .\sysimg.jl:29
│  [15] include(::String) at .\client.jl:388
│  [16] top-level scope at none:0
│ during initialization of module WinRPM
│ in expression starting at C:\Users\Mus\.julia\dev\WinRPM\deps\build.jl:1
└ @ Pkg.Operations C:\cygwin\home\Administrator\buildbot\worker\package_win64\build\usr\share\julia\stdlib\v1.0\Pkg\src\Operations.jl:1068

testing this branch locally

@hessammehr have ideas with LibExpat fails here

@musm
Copy link
Contributor

musm commented Aug 27, 2018

It looks like I had to manually delete the WinRPM cache to get things to work.

I think this PR is good to be merged.

@andreasnoack
Copy link
Contributor

Indeed. I've also just been able to install HDF5.jl without problems with this PR so @nalimilan it would be great if you could merge and make a new release.

@nalimilan nalimilan merged commit cb7a2fa into JuliaPackaging:master Aug 28, 2018
@nalimilan
Copy link
Member

See JuliaLang/METADATA.jl#17428. Hopefully it won't break anything. It would be really nice to have a test suite.

@andreasnoack
Copy link
Contributor

Thanks

It would be really nice to have a test suite.

Indeed. It is quite reckless that this package doesn't have tests.

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 this pull request may close these issues.

None yet

5 participants