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

truffle modular #2476

Closed
wants to merge 6 commits into from
Closed

truffle modular #2476

wants to merge 6 commits into from

Conversation

mkristian
Copy link
Member

this is a follow up of #2466 to propose a slightly different setup. it basically introduce a jruby-trufle artifact which can be easily exceluded using the jruby amven artifact. jruby-jars gem takes advantage of it as well, i.e. jruby-core jar there is already truffle free.

@headius
Copy link
Member

headius commented Jan 19, 2015

+1 looks good to me. I'd say we also remove it from jruby-complete, since anyone using jruby-complete can just add jruby-truffle to deps and it will be there.

@headius
Copy link
Member

headius commented Jan 21, 2015

This can merge now.

@mkristian
Copy link
Member Author

yes. beside this some cleanup is needed - readline, ripper, back to
SNAPSHOT version, etc

but this can go in as is.

On Wed, Jan 21, 2015 at 8:41 PM, Charles Oliver Nutter <
notifications@github.com> wrote:

This can merge now.


Reply to this email directly or view it on GitHub
#2476 (comment).

mkristian and others added 2 commits January 21, 2015 17:33
the source is split into core/src and truffle/src. some little
code changes were needed to get it compliled.

the lib/jruby.jar remains as is, dito jruby-complete.jar - just
the way it gets build is slightly different.

jruby-jars.gem has now three jars one is jruby-truffle.jar, so
people want to use truffle need to require this jar as well.

the jruby artifacts (org.jruby:jruby, org.jruby:jruby-noasm) can
easily exclude truffle via :exclusions => ['org.jruby:jruby-truffle']
declaration.
* Removed truffle references from core/pom.
* Moved over some files added after modularization PR.
@headius
Copy link
Member

headius commented Jan 21, 2015

I updated the patch and cleaned up some dependencies and force pushed to the PR branch.

We need to fix -X+T at command line before we merge this, so truffle remains functional at command line. We can do that either in the launcher (requires jruby-launcher work in C/++) or shade it into lib/jruby.jar.

@mkristian
Copy link
Member Author

shade it the truffle into lib/jruby.jar

@mkristian
Copy link
Member Author

merged manually

@mkristian mkristian closed this Jan 22, 2015
@mkristian mkristian deleted the test-truffle-modular branch January 22, 2015 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants