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

RFC: updates for changes to Julia #35

Closed
wants to merge 2 commits into from
Closed

RFC: updates for changes to Julia #35

wants to merge 2 commits into from

Conversation

timholy
Copy link
Contributor

@timholy timholy commented Feb 17, 2013

This doesn't work yet, but it goes part of the way towards updating Debug against recent Julia.

I'm a bit stumped by this one, however:

julia> using Debug
ERROR: Ex not defined
 in include_from_node1 at loading.jl:76
 in reload_path at loading.jl:96
 in require at loading.jl:48
 in include_from_node1 at loading.jl:76
 in reload_path at loading.jl:96
 in require at loading.jl:48
at /home/tim/.julia/Debug/src/Analysis.jl:15
at /home/tim/.julia/Debug/src/Debug.jl:7

julia> Meta.Ex
Union(Expr,Node{ExValue})

julia> Ex
ERROR: Ex not defined

julia> using Meta

julia> Ex
Union(Expr,Node{ExValue})

Seems odd given that Analysis is saying using Meta. I'm wondering if it's a Julia bug?

@kmsquire
Copy link

In addition to Meta.jl in the Debug package, there's now a base/meta.jl, which defines module Meta (and both are by Toivoh). I haven't looked closely, and have to run, but I'm betting the conflict is there.

@toivoh toivoh closed this in c3db98a Feb 20, 2013
@toivoh
Copy link
Owner

toivoh commented Feb 20, 2013

Thanks for pointing this out! Should be fixed now.

@toivoh
Copy link
Owner

toivoh commented Feb 20, 2013

@timholy: It does seem like a Julia bug. I've filed an issue: JuliaLang/julia#2375.

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

3 participants