-
-
Notifications
You must be signed in to change notification settings - Fork 925
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
Double packaging of org/joda/time/format/messages.properties
in jruby-jars
#3861
Comments
There are several files that are duplicated:
|
hey! there weren't any in 9.0.5.0, right? ... there wasn't much packaging related change as far as I know. |
@donv can you please tell me where you got the jruby-stdlib from ? the file from http://repo1.maven.org/maven2/org/jruby/jruby-stdlib/9.1.0.0/ has only files under META-INF directory which look all OK to me. |
Not critical for 9.1.1 release. Punting. |
I got the files from jruby-jars.gem from rubygems.org and from http://ci.jruby.org/snapshots/master/jruby-jars-9.1.3.0.SNAPSHOT.gem |
for the 9.1.2.0 released jruby-jars gem (from rubygems) I do not see those duplicates in the packed :
(no messages*.properties in jruby-stdlib .jar) ... same for the linked SNAPSHOT :
did I miss anything? |
oh najs ... that maven meta-data part seems scary - good that you kept an eye on that one. thanks! |
did remove joda-time-2.3.jar from stdlib dist. not sure how it got there - thus keeping open just in case. |
What more do we need to do for this one, @kares? |
should be good to go ... we can always reopen if its still an issue. |
Environment
Expected Behavior
No duplicate files in
jruby-core
vsjruby-stdlib
Actual Behavior
Found duplicate file: org/joda/time/format/messages.properties
When packaging Ruboto applications, this breaks the packaging. Which file should be included in the package?
The text was updated successfully, but these errors were encountered: