Bug 241122 - [PATCH] java/jakarta-bcel move to Apache and update to version 6.4.1
Summary: [PATCH] java/jakarta-bcel move to Apache and update to version 6.4.1
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-java mailing list
URL:
Keywords: buildisok
Depends on:
Blocks:
 
Reported: 2019-10-07 22:23 UTC by Pedro F. Giffuni
Modified: 2020-01-12 22:59 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (java)


Attachments
update diff (without moving yet the directory) (2.11 KB, patch)
2019-10-07 22:23 UTC, Pedro F. Giffuni
no flags Details | Diff
solve conflict from openjdk6 removal. (2.08 KB, patch)
2019-12-09 16:41 UTC, Pedro F. Giffuni
no flags Details | Diff
svn diff version with the directory move (6.45 KB, patch)
2019-12-21 20:35 UTC, Pedro F. Giffuni
no flags Details | Diff
Update Description (947 bytes, patch)
2020-01-07 18:20 UTC, Pedro F. Giffuni
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pedro F. Giffuni freebsd_committer 2019-10-07 22:23:24 UTC
Created attachment 208159 [details]
update diff (without moving yet the directory)

The ASF retired Jakarta on 2011 and now bcel is part of Apache Commons.

- Update to the latest version.
- Change the prefix from jakarta- to apache-.
  This should involve renaming also the directory.
Comment 1 Michael Osipov 2019-10-08 07:58:33 UTC
What is the purpose of maintaining this port at all? 99% of all users will request it from Maven Central anyway.
Comment 2 Pedro F. Giffuni freebsd_committer 2019-10-08 15:35:04 UTC
(In reply to Michael Osipov from comment #1)

Maven causes problems when building things in the cluster and packaging them.
OTOH, nothing depends on this ports so there is little interest in maintaining it.

In any case since we carry it, I'd rather prefer we carry a recent version rather than the old one.
Comment 3 Michael Osipov 2019-10-08 17:49:57 UTC
(In reply to Pedro F. Giffuni from comment #2)

I did not mean to build it with Maven, but other consumers will reference BCEL through MAven Central. No one will manually handle the JAR.

If no other port depends on it, don't put anymore effort into it, deprecate and remove by the end of the year.
Comment 4 Pedro F. Giffuni freebsd_committer 2019-10-08 17:57:13 UTC
(In reply to Michael Osipov from comment #3)

The decision to deprecate it is not mine to make: java@ is the maintainer. If the maintainer wants to deprecate it, I do request it is updated first to have an updated record in case it needs to be resurrected at a later time.
Comment 5 Automation User 2019-10-23 00:18:10 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/90747923
Comment 6 Pedro F. Giffuni freebsd_committer 2019-12-09 16:41:53 UTC
Created attachment 209807 [details]
solve conflict from openjdk6 removal.
Comment 7 Pedro F. Giffuni freebsd_committer 2019-12-21 20:35:20 UTC
Created attachment 210118 [details]
svn diff version with the directory move

Maintainer timeout?
Comment 8 commit-hook freebsd_committer 2019-12-24 00:37:33 UTC
A commit references this bug:

Author: pfg
Date: Tue Dec 24 00:36:35 UTC 2019
New revision: 520763
URL: https://svnweb.freebsd.org/changeset/ports/520763

Log:
  java/jakarta-bcel move to apache- and update to version 6.4.1

  The ASF retired Jakarta on 2011 and now bcel is part of Apache Commons.

  PR:	241122
  Approved by:	thierry (mentor), Maintainer timeout (2+ months)

Changes:
  head/MOVED
  head/java/Makefile
  head/java/apache-bcel/
  head/java/apache-bcel/Makefile
  head/java/apache-bcel/distinfo
  head/java/apache-bcel/pkg-descr
  head/java/jakarta-bcel/
Comment 9 Pedro F. Giffuni freebsd_committer 2020-01-07 18:20:00 UTC
Created attachment 210509 [details]
Update Description
Comment 10 Pedro F. Giffuni freebsd_committer 2020-01-07 18:21:29 UTC
I now think it should have been renamed apache-commons-bcel instead of apache-bcel.

At least the description should be updated.
Comment 11 Michael Osipov 2020-01-12 22:59:06 UTC
(In reply to Pedro F. Giffuni from comment #10)

Correct.