Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

push maven version to 3.6.3 #743

Closed
wants to merge 1 commit into from

Conversation

eduardkareletc
Copy link

An updated .info file to update maven to the most recent version (3.6.3).

@nieder nieder added the new upstream Package has an updated upstream version label Feb 15, 2021
@nieder
Copy link
Member

nieder commented Feb 15, 2021

Thanks for the submission.

Validating the .deb fails with this error:

Error: package contains the shared library
          /sw/share/maven/lib/jansi-native/osx/libjansi.jnilib
       but the corresponding install_name and compatibility_version
          /Users/gnodet/work/git/jansi-native/target/native-build/target/lib/libjansi-1.8.jnilib 0.0.0
       are not listed in the Shlibs field.  See the packaging manual.

The prebuilt jnilib needs to 1) have it's install_name fixed to the actual install location, and 2) be listed as a private library in the Shlibs: field. This would be fixed in InstallScript.

Also, searching through the installed files, bin/mvn and share/maven/bin/mvn reference the location /etc/mavenrc. This should be edited to point to %p/etc/mavenrc. See youtube-dl.info and youtube-dl.patch for a common way that the Fink prefix is added into files.

Also, feel free to change the Maintainer field to yourself. The previous maintainer has said it's OK to do that for his packages.

nieder added a commit that referenced this pull request Dec 5, 2021
borrowed heavily from #743
@nieder
Copy link
Member

nieder commented Dec 5, 2021

Closed/outdated by dd5f7b1 (and later updated to v3.8.4).
dd5f7b1 borrowed heavily from here

@nieder nieder closed this Dec 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new upstream Package has an updated upstream version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants