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

Update guava#39

Closed
Yannic wants to merge 1 commit into
google:masterfrom
Yannic:master
Closed

Update guava#39
Yannic wants to merge 1 commit into
google:masterfrom
Yannic:master

Conversation

@Yannic

@Yannic Yannic commented Apr 11, 2017

Copy link
Copy Markdown

Fixes failing build

@kaspnilsson

Copy link
Copy Markdown

+1

@Yannic

Yannic commented Jun 28, 2017

Copy link
Copy Markdown
Author

@cgruber Any news on this PR?

@cgruber

cgruber commented Jun 28, 2017

Copy link
Copy Markdown
Contributor

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

cgruber commented Jun 28, 2017

Copy link
Copy Markdown
Contributor

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

Yannic commented Jun 28, 2017

Copy link
Copy Markdown
Author

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

cgruber commented Jun 28, 2017 via email

Copy link
Copy Markdown
Contributor

@Yannic

Yannic commented Jun 30, 2017

Copy link
Copy Markdown
Author

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

mkobit commented Jun 30, 2017

Copy link
Copy Markdown

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

@umairaziz1

Copy link
Copy Markdown

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