You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by "Peter Donald (JIRA)" <ji...@apache.org> on 2012/09/24 02:44:07 UTC

[jira] [Closed] (BUILDR-615) Support non-numeric characters in VersionRequirement (e.g. version = "r09")

     [ https://issues.apache.org/jira/browse/BUILDR-615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Peter Donald closed BUILDR-615.
-------------------------------

    
> Support non-numeric characters in VersionRequirement (e.g. version = "r09")
> ---------------------------------------------------------------------------
>
>                 Key: BUILDR-615
>                 URL: https://issues.apache.org/jira/browse/BUILDR-615
>             Project: Buildr
>          Issue Type: Bug
>          Components: Dependency management
>    Affects Versions: 1.4.6
>            Reporter: Alex Boisvert
>            Assignee: Alex Boisvert
>            Priority: Minor
>             Fix For: 1.4.8
>
>
> From this thread: http://buildr.markmail.org/thread/kkpbbzmxyb4ojgbs
> Consider the following namespace code:
>   REQUIRES = ArtifactNamespace.for(self) do |ns|
>     ns.guava! "com.google.guava:guava:jar:r09"
>     ns.javassist! "javassist:javassist:jar:3.8.0.GA"
>     ns.reflections! "org.reflections:reflections:jar:0.9.5-RC1"
>   end
> The guava spec here causes the following error to be raised
>   Invalid version: "r09"
> This is a result of VersionRequirement.version? which checks the version
> string against the regex
>   /^\s*\d[#{VER_CHARS}]*\s*$/
> This regex is checked here
>   raise "Invalid version: #{version.inspect}" unless
> self.class.version?(version)
> So it seems that maybe this regex could be expanded to include versions
> like 'r09'.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira