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

Base.path_absolute_re does not match anything #5175

Closed
nalimilan opened this issue Dec 16, 2013 · 7 comments
Closed

Base.path_absolute_re does not match anything #5175

nalimilan opened this issue Dec 16, 2013 · 7 comments

Comments

@nalimilan
Copy link
Member

isabspath() and isdirpath() always return false in my RPM install of Julia. The git build works fine. This is a problem because JULIA_HOME is not considered as an absolute path though it is set to "/usr/bin", and thus Pkg.dir() is set to /home/user//home/user/.julia.

The problem is that Base.path_absolute_re (and other path regexps) is completely wrong. Could that be due to a problem with pcre on load? Or to a bad locale?

julia> ismatch(Base.path_absolute_re, "/home")
false

julia> Base.path_absolute_re
r"^/"

julia> ismatch(r"^/", "/home")
true

julia> xdump(Base.path_absolute_re)
Regex 
  pattern: ASCIIString 
    data: Array(Uint8,(2,)) [0x5e,0x2f]
  options: Uint32 33564672
  regex: Array(Uint8,(58,)) [0x45,0x52,0x43,0x50,0x3a,0x00,0x00,0x00,0x10,0x28    0x7d,0x00,0x06,0x19,0x1d,0x2f,0x72,0x00,0x06,0x00]

julia> xdump(r"^/")
Regex 
  pattern: ASCIIString 
    data: Array(Uint8,(2,)) [0x5e,0x2f]
  options: Uint32 33564672
  regex: Array(Uint8,(74,)) [0x45,0x52,0x43,0x50,0x4a,0x00,0x00,0x00,0x10,0x28    0x7d,0x00,0x06,0x19,0x1d,0x2f,0x72,0x00,0x06,0x00]

@StefanKarpinski
Copy link
Member

What's your versioninfo()?

@nalimilan
Copy link
Member Author

Well, it's a little... funny.

julia> versioninfo()
ERROR: invalid build identifier: ""
 in error at error.jl:22
 in VersionNumber at version.jl:30
 in print at version.jl:42
 in print_to_string at string.jl:23
 in versioninfo at util.jl:322 (repeats 2 times)

More seriously (that's another bug I have to fix), this is version 0.2.0 64-bit on Fedora 20, built against system libraries.

@StefanKarpinski
Copy link
Member

Ah, that's, um, interesting. Something very weird seems to be going on with this build.

@ivarne
Copy link
Member

ivarne commented Dec 16, 2013

@nalimilan If you can show me or @staticfloat your build process and your base/build_h.jl we might help fix the versioninfo problems. #5095 might make the solution easier in the future.

@nalimilan
Copy link
Member Author

@ivarne Yeah I thought I would wait until #5095 is fixed before possibly filing an issue.

@nalimilan
Copy link
Member Author

Interesting. I rebuilt the package since I've upgraded to Fedora 20, and both issues are gone (paths and version info). Looks like an updated PCRE was not fully ABI-compatible...

@ivarne
Copy link
Member

ivarne commented Dec 16, 2013

That explains. I could not figure out how the versioninfo() could fail in that manner, but the Version code uses regexes to parse the version_string from the VERSION file in the repository

The PCRE compatibility issue might be fixed when/if #5066 gets merged.

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

No branches or pull requests

3 participants