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

=dev-java/gradle-3.1: do not work with network-sandbox #38

Open
tharvik opened this issue Oct 11, 2016 · 5 comments
Open

=dev-java/gradle-3.1: do not work with network-sandbox #38

tharvik opened this issue Oct 11, 2016 · 5 comments

Comments

@tharvik
Copy link
Contributor

tharvik commented Oct 11, 2016

=dev-java/gradle-3.1 do not works with FEATURES=network-sandbox.

It might be as easy as modifing subprojects/docs/src/docs/dsl/org.gradle.api.tasks.wrapper.Wrapper.xml.

@tharvik tharvik changed the title dev-java/gradle: do not work with network-sandbox =dev-java/gradle-3.1: do not work with network-sandbox Oct 11, 2016
@chewi
Copy link
Member

chewi commented Oct 11, 2016

@Flowdalic, @msva, please investigate!

@msva
Copy link
Contributor

msva commented Oct 11, 2016

Yes, it is. Just as any other package with "bootsrapping" (where they fetching some crappy dependencies during build time and ignores system-wide installed ones (even if they exists in tree/overlay))

By the way, AFAIK, gradle was work in that way since eternity. Just as maven and some other bootstrap-systems.

Usually, fixing such things requires either heavy patching in ebuild or heavy patching of dist-sources and repackaging.

So, I'll try to take a look, but if you already know a solution, it will be very good if you'll apply a pull-request.

@chewi
Copy link
Member

chewi commented Oct 11, 2016

Okay, brain wasn't switched on earlier. I am aware that it does this and I don't like it but I was prepared to allow it in java-overlay if fixing it isn't trivial. I wouldn't allow it in the main tree.

@tharvik
Copy link
Contributor Author

tharvik commented Oct 11, 2016

#40 should fix it then.

@Flowdalic
Copy link
Member

What @msva said :)

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 a pull request may close this issue.

4 participants