-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-3437][BUILD] Support crossbuilding in maven. With new scala-install-plugin. #2357
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
Conversation
|
QA tests have started for PR 2357 at commit
|
|
QA tests have finished for PR 2357 at commit
|
|
This sort of conflicts with parts of #2329. Would it be possible to add a flag to the new plugin so that it can automatically skip projects where a certain property is set? That way you just set "scala.install.skip" instead of copying a whole lot of XML for each module you don't want to install. No idea if that's possible, and I'm aware that's not very maven-approved, but just a suggestion. |
|
It will not conflict with that, it works the same way as maven install plugin. (accidentally deleted my previous post. :/) |
|
Hey @ScrapCodes rather than forking the existing install plug-in, could we write our own plug-in that runs before the install plug-in? Forking seems like it will be difficult of e.g. the maven install plug-in updates. |
|
you are right about that, but we are forking it because we want an install plugin modified. If it was possible to run a plugin just before install and install plugin magically does the job correctly. That would have been nicer. But that isn't possible unless I try to hack things using reflection. Its like maven keeps its copy of objects before letting plugins use it. |
|
I am anyway trying more options to not need to modify maven-install-plugin. |
|
|
Hi @srowen, like I said. We can run a plugin before maven install - no questions about that. But since maven install gets a copy of STATE(somehow via guice). Altering it in another maven plugin does not help much. So as of now, I do not see a way around not modifying maven install plugin. |
|
I see, I thought you mentioned above that running a plugin before It is worth keeping track of alternatives when evaluating just how far to hack this. For example: Scala 2.11 support could be bound up with Spark 2.x support. Or it can be in a branch that is maintained over a few minor versions, which is not such a big deal if the delta is just flipping "2.10" -> "2.11" in many places. |
|
Turns out this is not very useful either. |
|
QA tests have started for PR 2357 at commit
|
|
QA tests have finished for PR 2357 at commit
|
|
Test PASSed. |
|
Can one of the admins verify this patch? |
Since this plugin is not deployed anywhere, for anyone trying this patch has to publish it locally by cloning the following repo https://github.com/ScrapCodes/scala-install-plugin. And then running
mvn install.