Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clojure doesn't build for me #327

Closed
jmhofer opened this issue Aug 31, 2013 · 6 comments
Closed

Clojure doesn't build for me #327

jmhofer opened this issue Aug 31, 2013 · 6 comments

Comments

@jmhofer
Copy link
Contributor

jmhofer commented Aug 31, 2013

For 0.11.1, ./gradlew clean build leads to

:language-adaptors:rxjava-clojure:clojureTest
Exception in thread "main" java.lang.ClassNotFoundException: rx.lang.clojure.interop.DummyObservable

The class is there, however the build doesn't seem to pick it up.
@benjchristensen
Copy link
Member

That is odd considering it made it through our build and release cycle.

@daveray any ideas?

@mtnygard mtnygard mentioned this issue Aug 31, 2013
Closed
@mtnygard
Copy link

I was able to reproduce this, but not consistently. I also found non-deterministic unit test failures during build. See #329 for the full report & link to a gist with transcripts.

@daveray
Copy link
Contributor

daveray commented Aug 31, 2013

I wrestled with DummyObservable a bit, but thought I had beaten it. Clearly not. I'll take a look. Sorry about that.

daveray added a commit to daveray/RxJava that referenced this issue Aug 31, 2013
@daveray
Make DummyObservable visible to clojureTest classpath.
a76e752 
RxJava issue ReactiveX#327. Moved DummyObservable into main so that it's
compiled and available when the clojureTest task executes. Perhaps
someone with stronger Gradle skills can figure out how to put this Java
class in src/test and get things to work out. Correctly. For the time
being, several clean/build cycles worked fine for me.
@daveray daveray mentioned this issue Aug 31, 2013
Merged
@benjchristensen
Copy link
Member

@jmhofer Does it work for you now that #330 is merged?

@jmhofer
Copy link
Contributor Author

jmhofer commented Aug 31, 2013

Yes, works for me now, thanks!

@jmhofer jmhofer closed this as completed Aug 31, 2013
@benjchristensen
Copy link
Member

Great. Thanks for confirming.

rickbw pushed a commit to rickbw/RxJava that referenced this issue Jan 9, 2014
@daveray
Make DummyObservable visible to clojureTest classpath.
69c75b6 
RxJava issue ReactiveX#327. Moved DummyObservable into main so that it's
compiled and available when the clojureTest task executes. Perhaps
someone with stronger Gradle skills can figure out how to put this Java
class in src/test and get things to work out. Correctly. For the time
being, several clean/build cycles worked fine for me.
jihoonson pushed a commit to jihoonson/RxJava that referenced this issue Mar 6, 2020
@RobWin
Fixed issues: ReactiveX#273, ReactiveX#273 and ReactiveX#327 (Reactiv…
6e4cb87 
…eX#438)

Issue ReactiveX#273: Added remove method to AbstractRegistry.
Issue ReactiveX#273: Added replace method to AbstractRegistry.
Issue ReactiveX#327: MicroMeter tagged Metric classes are automatically updated when an entry is added, removed or replaced in AbstractRegistry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@daveray @mtnygard @jmhofer @benjchristensen