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

Under Gentoo, "impl"-type netbeans module dependencies prevent plugins from installation #174

Open
pvgoran opened this issue Dec 7, 2016 · 3 comments

Comments

@pvgoran
Copy link

pvgoran commented Dec 7, 2016

In Gentoo, NetBeans is compiled from sources, so its modules (at least some of them, including org.netbeans.modules.extexecution.base) get OpenIDE-Module-Implementation-Version values (in MANIFEST.MF) that depend on the build time, like this: 20161207-unknown-revn. This prevents "impl"-type modules dependencies that are used in scala.console and scala.sbt from matching, because they want the implementation version of org.netbeans.modules.extexecution.base from the official NetBeans modules (which is something like 201609300101).

So I can't install the modules in NetBeans 8.2 under Gentoo. I tried akochnev's build, and I tried to compile nbscala from git.

@pvgoran
Copy link
Author

pvgoran commented Dec 7, 2016

After investigation, it looks like the "impl"-type dependencies are not needed. I replaced usages of org.netbeans.modules.extexecution.base.ExternalProcessBuilder with org.netbeans.api.extexecution.ExternalProcessBuilder in these two modules, after that the "impl"-type dependencies could be safely removed. This way, I managed to compile and install the modules.

Is it possible to attach a patch here? Or the only way is to fork and create a pull request?

@pvgoran
Copy link
Author

pvgoran commented Dec 7, 2016

Ok, here is the patch. (I had to rename to .txt before GitHub allowed it.)

remove-impl-module-dependencies.patch.txt

@akochnev
Copy link
Contributor

@pvgoran - nice work on tracking down the replacement for the external execution module that became an API! I'm a bit surprised that the rest of the plugin worked when you removed the other implementation dependencies, but it certainly is possible that they're out of date. I'll try out your patch and create a PR for @dcaoyuan to merge in

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

No branches or pull requests

2 participants