Perdjesk opened a new pull request, #689:
URL: https://github.com/apache/incubator-baremaps/pull/689

   Resolves
   ```
    % baremaps --version
   Baremaps vnull
   ```
   
   For whatever reason the globbing done in the shell script or something else 
was making the JVM picks-up the `version.txt` from baremaps-server which is not 
correctly interpolated with Maven properties.
   ```
   java -cp "$DIR/../lib/*" org.apache.baremaps.cli.Baremaps "$@"
   ```
   
   This proposal scopes the resource to a package to avoid further conflicts 
and ensure that the resources picks-up is the version specified in baremaps-cli 
jar file.
   ```
   /lib/baremaps-cli-0.7.2-SNAPSHOT.jar!/org/apache/baremaps/cli/version.txt
   ```
   instead of classpath order resolution that serves a firstly found 
`version.txt` file at root of a jar.
   ```
   /lib/<???order-based-resolution???>.jar!/version.txt
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to