Skip to content

Add rxjava3 to third_party#12523

Closed
coeuvre wants to merge 1 commit intobazelbuild:masterfrom
coeuvre:update-third-party
Closed

Add rxjava3 to third_party#12523
coeuvre wants to merge 1 commit intobazelbuild:masterfrom
coeuvre:update-third-party

Conversation

@coeuvre
Copy link
Copy Markdown
Member

@coeuvre coeuvre commented Nov 19, 2020

We are going to refactor remote module with RxJava. This PR add rxjava3 to third_party so we can use it later.

@google-cla google-cla Bot added the cla: yes label Nov 19, 2020
@coeuvre coeuvre closed this Nov 19, 2020
@meteorcloudy
Copy link
Copy Markdown
Member

/cc @olekw
Hey Olek, we are planning to use rxjava3 to improve Bazel's remote execution code. It's not yet packaged for Debian, but it has almost no additional dependencies. Can you please verify if it's ok to package this for future Bazel version in Debian?
https://github.com/ReactiveX/RxJava
https://mvnrepository.com/artifact/io.reactivex.rxjava3/rxjava/3.0.7

@olekw
Copy link
Copy Markdown
Contributor

olekw commented Dec 27, 2020

Can you please verify if it's ok to package this for future Bazel version in Debian?

Ugh, a Gradle build... But, from a quick look, it does look like it only has one real dependency (already in Debian) and I should be able to just delete most of the extraneous Gradle build stuff. I'll try to work on that in the next couple days, @meteorcloudy. :)

@olekw
Copy link
Copy Markdown
Contributor

olekw commented Dec 29, 2020

Ok, packaged and in the NEW queue.

@philwo
Copy link
Copy Markdown
Member

philwo commented Jan 5, 2021

Thank you @olekw!! 😊

@meteorcloudy
Copy link
Copy Markdown
Member

Amazing, thank you, Olek!

@olekw
Copy link
Copy Markdown
Contributor

olekw commented Jan 5, 2021

Looks like a nice piece of software, glad I could help bring it to Debian. :)

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.

4 participants