Skip to content

Replace removed Guava classes and methods by updated versions#17

Merged
anjackson merged 1 commit into
iipc:masterfrom
gerhardgossen:master
Jun 4, 2014
Merged

Replace removed Guava classes and methods by updated versions#17
anjackson merged 1 commit into
iipc:masterfrom
gerhardgossen:master

Conversation

@gerhardgossen

Copy link
Copy Markdown
Contributor

Several classes and methods have been deprecated and removed since Guava 14.0.1 (e.g. NullOutputStream, LimitInputStream and MapMaker.makeComputingMap). This prevents clients from updating to a newer version of Guava.

This change replaces all uses of removed Guava APIs by their documented alternatives and updates Guava to the current version (17.0).

@anjackson

Copy link
Copy Markdown
Member

This looks good, although there may be consequences for down-stream codebases like Heritrix and Open/Wayback. Could @PsypherPunk and/or @kris-sigur have a quick look before we go ahead?

@PsypherPunk

Copy link
Copy Markdown

Looking at the changes, none should affect any API calls. I'd suggest we keep OpenWayback dependent on a current version of webarchive-commons (i.e. still using 14.0.1—1.1.1?) until a post-2.0.0 release though.

@gerhardgossen

Copy link
Copy Markdown
Contributor Author

If you want to avoid a big jump in the Guava dependency version, it should also be fine to bump Guava just to 15.0 (all tests pass). Version 14.0.1 does not work unfortunately (InternetDomainName.name() was merged with .toString() in 15.0 and removed later, but the return value of .toString() in 14.0.1 is incompatible).

@kris-sigur

Copy link
Copy Markdown
Member

Agree with @PsypherPunk. It all looks fine from an API standpoint and the changes in code seem to be the minimum required due to changes in Guava.

If we're upgrading to a new version of Guava, we might as well go to the latest

@anjackson

Copy link
Copy Markdown
Member

Yes, keeping our dependencies up to date is good Maven hygiene. Let's go ahead.

anjackson added a commit that referenced this pull request Jun 4, 2014
Replace removed Guava classes and methods by updated versions
@anjackson anjackson merged commit dc98c69 into iipc:master Jun 4, 2014
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.

4 participants