Skip to content

Use maven-jar-plugin to attach the custom manifest file#124

Merged
WiseEagleOwl merged 2 commits into
LGoodDatePicker:masterfrom
WiseEagleOwl:include-custom-manifest-via-maven-jar-plugin
Feb 22, 2021
Merged

Use maven-jar-plugin to attach the custom manifest file#124
WiseEagleOwl merged 2 commits into
LGoodDatePicker:masterfrom
WiseEagleOwl:include-custom-manifest-via-maven-jar-plugin

Conversation

@WiseEagleOwl

@WiseEagleOwl WiseEagleOwl commented Feb 22, 2021

Copy link
Copy Markdown
Collaborator

This PR provides

previously the maven-shade-plugin was used for this purpose which causes
the issue that the manifest file is not at the beginning of the JAR stream
which is required by the JarInputStream implementation.
@WiseEagleOwl

Copy link
Copy Markdown
Collaborator Author

@bjorndarri Can you please verify that this resolves your issue?
A temporary test version can be found in this staging repository with version 11.2.1-test. You can add this test repository to your Gradle build like this:

repositories {
   //all your default repositories...
  maven {
        url "https://oss.sonatype.org/content/repositories/comgithublgooddatepicker-1030"
    }
}

@WiseEagleOwl WiseEagleOwl changed the title Resolve #122 Use maven-jar-plugin to attach the custom manifest file Use maven-jar-plugin to attach the custom manifest file Feb 22, 2021
@bjorndarri

Copy link
Copy Markdown
Contributor

At first glance it looks like this solves the problem, I'll do some further testing in a properly modular project tonight, I'll report back.

@bjorndarri

Copy link
Copy Markdown
Contributor

Couldn't wait, finished testing this and I can confirm that this does indeed solve the problem.

ps. sorry about that weird solution proposed yesterday, I was so deep down the debugging rabbit hole, having spent hours searching for a solution that I just jumped on the first thing that looked like it fixed the problem, even though it was so obviously wrong :).

@WiseEagleOwl

Copy link
Copy Markdown
Collaborator Author

No worries, tunnel vision on the bug hunt happens to everyone 🙂. You left enough bread crumbs for me to find the root cause, that is what matters 👍

Thank you for taking the time to verify my solution. FYI the release process for LGoodDatePicker v11.2.1 containing this fix will probably begin next week. Your special test version will vanish once a preview release for v11.2.1 is published.

@WiseEagleOwl WiseEagleOwl merged commit 92f6385 into LGoodDatePicker:master Feb 22, 2021
@WiseEagleOwl WiseEagleOwl deleted the include-custom-manifest-via-maven-jar-plugin branch February 24, 2021 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JarInputStream fails to locate the manifest file in LGoodDatePicker.jar

2 participants