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.

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

Avoid re-generating meta examples on make #3659

Merged
merged 1 commit into from Feb 27, 2017

Conversation

geektoni
Copy link
Contributor

Fix for issue #3653.

I believe this behaviour appeared after the introduction of the Scala interface. As I read from pull #3597, it was decided to run the java .class file with Scala (without generating the .scala files). However, the meta_examples target expects to find the all the .scala files, but since they are not present it triggers the re-generation of the meta examples.

IMHO this is a ugly workaround, but it solves the problem until a more advanced solution arrives.

@karlnapf
Copy link
Member

There are new errors in the scala build now. Repdoduce locally with -DENABLE_TESTING=ON

See https://travis-ci.org/shogun-toolbox/shogun/jobs/205716515#L4975

@geektoni
Copy link
Contributor Author

Correct the Scala errors. The others Travis failures should be unrelated to my changes.

# FIXME
# Hack to avoid generation of scala meta examples dependencies that will
# not be generated by generate.py script (since there is no scala.json)
LIST(REMOVE_ITEM AVAILABLE_INTERFACES "ScalaModular")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fine with me this hack

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

peeeeeeeeeeeeople DISABLED_INTERFACES

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding the Scala module to DISABLED_INTERFACES will cause to not include examples/meta/scala/CMakeLists.txt. Only the Scala integration tests will be generated (and they will fail) and the flag -DScalaModular will become a bit useless (it will behave like an alias of -DJavaModular=ON).

This hack permits to create all the Scala tests anyway (generated and integration), but as I said before, this is just a workaround to prevent the continuous generation of the meta examples code.

@karlnapf
Copy link
Member

Thanks!

@karlnapf karlnapf merged commit 1b95b89 into shogun-toolbox:develop Feb 27, 2017
karasikov pushed a commit to karasikov/shogun that referenced this pull request Apr 15, 2017
Avoid re-generating meta examples on `make`
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.

None yet

3 participants