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

sci-*/*: remove dead java-ant-2 eclass #1294

Closed
wants to merge 3 commits into from

Conversation

APN-Pucky
Copy link
Contributor

@APN-Pucky APN-Pucky commented Nov 7, 2024

  • Test if the packages still compile
  • Use eant instead of ant
  • Patch java source/target version

@APN-Pucky APN-Pucky changed the title */*: sci-*/* remove dead java-ant-2 eclass sci-*/* remove dead java-ant-2 eclass Nov 7, 2024
@APN-Pucky APN-Pucky changed the title sci-*/* remove dead java-ant-2 eclass sci-*/*: remove dead java-ant-2 eclass Nov 7, 2024
@APN-Pucky APN-Pucky marked this pull request as ready for review November 7, 2024 15:11
@APN-Pucky
Copy link
Contributor Author

APN-Pucky commented Nov 7, 2024

Still Failing:
sci-biology/fastqc: source/target options 6 is no longer supported Use 7 or later
sci-biology/trimmomatic: source/target options 5 is no longer supported Use 7 or later
sci-biology/imagej: source/target options 6 is no longer supported Use 7 or later

even with explicit version args, so probably the ant file needs to be patched...

Copy link
Member

@Nowa-Ammerlaan Nowa-Ammerlaan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know much about java packaging, but note that some of these packages are already masked because the dependencies no longer resolve: https://github.com/gentoo/sci/blob/master/profiles/package.mask

@Nowa-Ammerlaan
Copy link
Member

even with explicit version args, so probably the ant file needs to be patched...

If you want to you could check if there is a newer version available that supports newer java.

Otherwise I'd just drop these packages, java 5/6 is very old. In theory you maybe could still compile them by eselecting dev-java/openjdk:8, but if upstream is gone then it is probably not worth keeping.

@APN-Pucky
Copy link
Contributor Author

APN-Pucky commented Nov 7, 2024

imagej is not only biology and generally usefull, also that version is of September this year, so that can be kept. I think they all work with more recent java versions (usually backwards compatible), but they need patches to the ant script. I think upstream just treats source/target as minimum java versions.

@APN-Pucky
Copy link
Contributor Author

APN-Pucky commented Nov 7, 2024

Also I am not sure where ant-core went, but it is still available through some other/renamed package (#1275), so the masking could probably be undone after the changes.

I just do not know how to "really" test the packages (and I frankly do not care too much about sci-biology :))

@Nowa-Ammerlaan
Copy link
Member

Also I am not sure where ant-core went, but it is still available through some other/renamed package (#1275), so the masking could probably be undone after the changes.

According to the package.deprecated entry ant-core was merged into ant itself: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=90d27074d2d374159463e6adebedf96658cbdb50

I just do not know how to "really" test the packages (and I frankly do not care too much about sci-biology :))

Personally, I also don't care about the individual packages. What we really need is some biologist who does care about sci-biology to help maintain all of these packages. I do care about the overall usability of this repository though, and global scope errors and unresolvable dependencies tend to interfere not just with the offending package but with all the other packages in the repository as well. For that reason I usually just remove them.

@APN-Pucky
Copy link
Contributor Author

I bumped them now. All three compile and for imageJ only I checked that the GUI works.

Copy link
Member

@Nowa-Ammerlaan Nowa-Ammerlaan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, Thanks 👍

You can also drop the now obsolete entries from package.mask

@@ -65,6 +65,9 @@ src_prepare() {
if ! use debug ; then
sed -i 's: debug="on">: debug="off">:' "${S}"/build.xml || die
fi
sed -i 's/source="1\.6"/source="1.7"/g' build.xml || die
sed -i 's/target="1\.6"/target="1.7"/g' build.xml || die
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better delete -source / -target attributes completely from alll javac elements of all build.xml files and use the ant.build.javac.source ant.build.javac.target options as done in dev-java/c3p0 or in dev-java/lucene.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Done.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Done.

Sorry, forgot to mention, pls avoid using jre-1.7:* / jdk-1.7:*. It is no longer supported by Java 21.
https://wiki.gentoo.org/wiki/Java_Developer_Guide#JDK.2FJRE
Also, the support for EAPI 7 will be dropped soon on java-pkg-simple.eclass.

Signed-off-by: Alexander Puck Neuwirth <alexander@neuwirth-informatik.de>
Signed-off-by: Alexander Puck Neuwirth <alexander@neuwirth-informatik.de>
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.

3 participants