Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Update guava #39

Closed
wants to merge 1 commit into from
Closed

Update guava #39

wants to merge 1 commit into from

Conversation

Yannic
Copy link

@Yannic Yannic commented Apr 11, 2017

Fixes failing build

@kaspnilsson
Copy link

+1

@Yannic
Copy link
Author

Yannic commented Jun 28, 2017

@cgruber Any news on this PR?

@cgruber
Copy link
Contributor

cgruber commented Jun 28, 2017

Sorry, have been overloaded this quarter. I'll try to get this done in the next few days. I need to take this into google's internal repo and then push it out and I'm OOO tomorrow. Might get to it tonight, but if not, later this week.

@cgruber
Copy link
Contributor

cgruber commented Jun 28, 2017

Context question... do you mean 22.0-SNAPSHOT, or do you mean 22? Do you need unreleased functionality, or does 22.0 resolve the cygwin issue?

@Yannic
Copy link
Author

Yannic commented Jun 28, 2017

I don't remember why I used 22.0-SNAPSHOT, but I'm sure there has been a reason. I will find out again why I didn't use 22.0 and let you know.

@cgruber
Copy link
Contributor

cgruber commented Jun 28, 2017 via email

@Yannic
Copy link
Author

Yannic commented Jun 30, 2017

The reason I uses 22.0-SNAPSHOT was that 22.0 has not been released in April. I updated this PR to use 22.0

@mkobit
Copy link

mkobit commented Jun 30, 2017

Other snapshot dependencies were removed/updated in #41 because it led to runtime failures

@umairaziz1
Copy link

Hello

@Yannic Yannic closed this May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants