Issue2121

classification
Title: Jython jar on Maven central embedd other third party libraries
Type: Severity: normal
Components: Versions: Jython 2.7
Milestone:
process
Status: open Resolution: accepted
Dependencies: Superseder:
Assigned To: Nosy List: rjn, tuska, zyasoft
Priority: low Keywords: patch

Created on 2014-04-07.15:24:20 by tuska, last changed 2017-01-16.09:44:51 by rjn.

Files
File name Uploaded Description Edit Remove
issue2121.patch rjn, 2017-01-15.23:55:08
Messages
msg8270 (view) Author: Thomas Mortagne (tuska) Date: 2014-04-07.15:24:19
I can understand it for the jython-standalone one but there is no reason to emmbed all that in the jython jar. It generates tons of duplicated classes.

There is also something that looks like empty (except for the package-info.class files) guava embedded.
msg8271 (view) Author: Thomas Mortagne (tuska) Date: 2014-04-07.15:25:20
I'm talking about the 2.7b1 release which is not listed in the "Versions" list so I indicated 2.7.
msg8272 (view) Author: Thomas Mortagne (tuska) Date: 2014-04-07.15:31:29
The main one is xml-apis:xml-apis (probably 1.4.01).
msg8273 (view) Author: Thomas Mortagne (tuska) Date: 2014-04-07.15:48:58
Yes that's the 1.4.01 according to org.apache.xmlcommons.Version class.
msg8274 (view) Author: Jim Baker (zyasoft) Date: 2014-04-07.16:36:27
We would like to make this change, however this requires rethinking our current build process. This will also help in terms of being compatible in the context of Debian packaging.
msg9080 (view) Author: Jim Baker (zyasoft) Date: 2014-10-05.17:12:48
Defer to 2.7.1
msg11022 (view) Author: Rafal Nagrodzki (rjn) Date: 2017-01-15.23:55:08
I prepared a very simple solution which has an obvious downside of dependencies being specified in two places.

I'm happy to hear your comments and give it another try, if you give me some guidance on how you want to manage dependencies ideally.
msg11023 (view) Author: Rafal Nagrodzki (rjn) Date: 2017-01-16.09:44:50
I just realized I did not tackle package name remappings so my patch is incomplete.
History
Date User Action Args
2017-01-16 09:44:51rjnsetmessages: + msg11023
2017-01-15 23:55:10rjnsetfiles: + issue2121.patch
keywords: + patch
messages: + msg11022
nosy: + rjn
2014-10-05 17:12:48zyasoftsetpriority: low
messages: + msg9080
2014-04-07 16:36:27zyasoftsetresolution: accepted
messages: + msg8274
nosy: + zyasoft
2014-04-07 15:48:58tuskasetmessages: + msg8273
2014-04-07 15:31:29tuskasetmessages: + msg8272
2014-04-07 15:25:20tuskasetmessages: + msg8271
2014-04-07 15:24:20tuskacreate