~qeef/damn-dev

damn-plugin: Add Plugin-Minimum-Java-Version to manifest v1 PROPOSED

Taylor Smock <smocktaylor@gmail.com>
2 months ago 
Taylor Smock: 1
 Add Plugin-Minimum-Java-Version to manifest

 1 files changed, 5 insertions(+), 2 deletions(-)
#1294151 running .build.yml
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git: 

curl -s https://lists.sr.ht/~qeef/damn-dev/patches/54286/mbox | git am -3
Learn more about email & git
Reply to thread View this thread in the archives

[PATCH damn-plugin] Add Plugin-Minimum-Java-Version to manifest Export this patch

Taylor Smock <smocktaylor@gmail.com>
2 months ago 
This is needed since JOSM currently has a minimum Java version of 11,
while this plugin now requires Java 17+.

Signed-off-by: Taylor Smock <tsmock@meta.com>
---
 pom.xml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index 03cd7a4..ecf0d24 100644
--- a/pom.xml
+++ b/pom.xml
@@ -6,6 +6,7 @@
    <version>1.0-SNAPSHOT</version>
    <properties>
        <plugin.main.version>19067</plugin.main.version>
        <plugin.minimum.java.version>17</plugin.minimum.java.version>
    </properties>
    <repositories>
        <repository>
@@ -60,6 +61,8 @@
                            <Author>qeef</Author>
                            <Plugin-Link>https://damn-project.org</Plugin-Link>
                            <Plugin-Icon>images/dialogs/damn.svg</Plugin-Icon>
                            <!-- JOSM r19067 has a minimum Java version of 11. We have a minimum Java version of 17, so we should specify that we need a newer version of Java -->
                            <Plugin-Minimum-Java-Version>${plugin.minimum.java.version}</Plugin-Minimum-Java-Version>
                        </manifestEntries>
                    </archive>
                </configuration>
@@ -69,7 +72,7 @@
                <artifactId>maven-compiler-plugin</artifactId>
                <version>3.13.0</version>
                <configuration>
                    <release>17</release>
                    <release>${plugin.minimum.java.version}</release>
                </configuration>
            </plugin>
            <plugin>
@@ -88,7 +91,7 @@
                                    <version>3.6.3</version>
                                </requireMavenVersion>
                                <requireJavaVersion>
                                    <version>17</version>
                                    <version>${plugin.minimum.java.version}</version>
                                </requireJavaVersion>
                            </rules>
                        </configuration>
-- 
2.46.0
builds.sr.ht <builds@sr.ht>
2 months ago 
damn-plugin/patches/.build.yml: TIMEOUT in 5h0m0s

[Add Plugin-Minimum-Java-Version to manifest][0] from [Taylor Smock][1]

[0]: https://lists.sr.ht/~qeef/damn-dev/patches/54286
[1]: mailto:smocktaylor@gmail.com

✗ #1294151 TIMEOUT damn-plugin/patches/.build.yml https://builds.sr.ht/~qeef/job/1294151
Jiri Vlasak <jiri.hubacek@gmail.com>
a month ago 
Thank you for the patches, all have been merged.