groovy-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nickboldt <...@git.apache.org>
Subject [GitHub] groovy pull request #597: GROOVY-8305: Fix default Ivy settings file for loc...
Date Fri, 29 Sep 2017 13:07:15 GMT
Github user nickboldt commented on a diff in the pull request:

    https://github.com/apache/groovy/pull/597#discussion_r141862977
  
    --- Diff: src/resources/groovy/grape/defaultGrapeConfig.xml ---
    @@ -26,7 +26,7 @@
             <ivy pattern="${user.home}/.groovy/grapes/[organisation]/[module]/ivy-[revision].xml"/>
             <artifact pattern="${user.home}/.groovy/grapes/[organisation]/[module]/[type]s/[artifact]-[revision](-[classifier]).[ext]"/>
           </filesystem>
    -      <ibiblio name="localm2" root="file:${user.home}/.m2/repository/" checkmodified="true"
changingPattern=".*" changingMatcher="regexp" m2compatible="true"/>
    +      <ibiblio name="localm2" root="file:/${user.home}/.m2/repository/" checkmodified="true"
changingPattern=".*" changingMatcher="regexp" m2compatible="true"/>
    --- End diff --
    
    This seems logical/obvious since ${user.home} includes a prefix slash, and for a valid
URI you need two slashes. Thus when in doubt, file:/${user.home}/path/to/things. This is true
for p2 repo URIs too.


---

Mime
View raw message