build: Disable UPX by default. (#935)

.github/workflows/release.yml

@@ -87,6 +87,10 @@ jobs:
         run: >
           ./mvnw --no-transfer-progress -Pnative -DskipTests -pl eu.michael-simons.neo4j:neo4j-migrations-cli package -am
 
+      - name: 'Smoke test'
+        run: >
+          ./neo4j-migrations-cli/target/neo4j-migrations -V
+
       - name: 'Upload build artifact'
         uses: actions/upload-artifact@v2
         with:
@@ -128,7 +132,11 @@ jobs:
 
       - name: 'Create distribution'
         run: >
-          ./mvnw --no-transfer-progress -Pnative -DskipCompress -DskipTests -pl eu.michael-simons.neo4j:neo4j-migrations-cli package -am
+          ./mvnw --no-transfer-progress -Pnative -DskipTests -pl eu.michael-simons.neo4j:neo4j-migrations-cli package -am
+
+      - name: 'Smoke test'
+        run: >
+          ./neo4j-migrations-cli/target/neo4j-migrations -V
 
       - name: 'Upload build artifact'
         uses: actions/upload-artifact@v2

CONTRIBUTING.adoc

@@ -42,21 +42,21 @@ Run it with:
 
 [source,console,subs="verbatim,attributes"]
 ----
-./mvnw -Dnative -DskipCompress clean package
+./mvnw -Dnative clean package
 ----
 
 The resulting migration tool can only be used to load Cypher script based migrations in file locations.
 It won't find classes, as those are instantiated dynamically via reflection, which is not supported in full in a native image.
 
-NOTE: The binaries are compressed by default via https://upx.github.io[upx]. While it has some downsides on the RSS memory
+NOTE: You can try to use https://upx.github.io[upx] to compress your binaries. While it has some downsides on the RSS memory
       usage as explained by the folks over at https://quarkus.io/blog/upx/[the Quarkus team], the advantages of having a 2/3
-      smaller binary are bigger. Especially due to the fact that the CLI isn't a permanent memory guest.
+      smaller binary are bigger.
       +
-      If you want to enable the compression in your build, remove the above `-DskipCompress` argument.
+      If you want to enable the compression in your build, add `-DskipCompress=false` to the build.
 
 === Fast build
 
-NOTE: This is useful if you want to just have an install of a snapshot version. No tests are run, no verification is done.
+NOTE: This is useful if you want to just have an installation of a snapshot version. No tests are run, no verification is done.
 
 [source,console,subs="verbatim,attributes"]
 [[build-default-bash]]
@@ -84,7 +84,7 @@ Use the provided script to bump the minor or major version:
 ./bin/bump-version.sh 3.0
 ----
 
-Version can be either a new major or minor version (such as 3, 3.0 etc) or a fully qualified snapshot version (such as 3.0.0-SNAPSHOT).
+Version can be either a new major or minor version (such as 3, 3.0 etc.) or a fully qualified snapshot version (such as 3.0.0-SNAPSHOT).
 
 === Building the documentation
 

neo4j-migrations-cli/pom.xml

@@ -36,7 +36,7 @@
 		<executable-suffix />
 		<java-module-name>ac.simons.neo4j.migrations.cli</java-module-name>
 		<name-of-main-class>ac.simons.neo4j.migrations.cli.MigrationsCli</name-of-main-class>
-		<skipCompress>false</skipCompress>
+		<skipCompress>true</skipCompress>
 		<sonar.coverage.jacoco.xmlReportPaths>${basedir}/../${aggregate.report.dir}</sonar.coverage.jacoco.xmlReportPaths>
 	</properties>