Skip to content

use correct joda-time#3827

Merged
drcrallen merged 1 commit intoapache:masterfrom
metamx:fix_joda_time_version
Jan 7, 2017
Merged

use correct joda-time#3827
drcrallen merged 1 commit intoapache:masterfrom
metamx:fix_joda_time_version

Conversation

@clintropolis
Copy link
Copy Markdown
Member

Removed the version on joda-time in java-util, was causing an issue with building extensions

@drcrallen
Copy link
Copy Markdown
Contributor

👍

@drcrallen drcrallen added the Bug label Jan 6, 2017
@drcrallen drcrallen added this to the 0.10.0 milestone Jan 6, 2017
@gianm
Copy link
Copy Markdown
Contributor

gianm commented Jan 7, 2017

👍 and also, really, nothing in java-util/pom.xml should have versions. That should all be handled by dependencyManagement in the master pom.

@drcrallen drcrallen merged commit 97ce006 into apache:master Jan 7, 2017
@drcrallen drcrallen deleted the fix_joda_time_version branch January 7, 2017 00:31
@gianm
Copy link
Copy Markdown
Contributor

gianm commented Jan 7, 2017

Well, that was weird. This change was already made a month ago in #3764 but github showed a diff in this PR anyway. And now there's an empty commit in the history: 97ce006

A bit weird but no harm done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants