Closed olamy closed 4 months ago
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA). The following contributors of this PR have not signed the OCA:
To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application.
When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated.
If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public.
no idea what to do to sign the OCA.... I have an account with the email used to sign this commit. I can connect to my oracle but now when I want to sign the OCA, I get this
Please contact but there is no link provided.
Hi @olamy, thanks for the PR. Maybe @alina-yur can help us find out what's going on with the OCA portal. In the meantime, could you maybe extend the description of the PR a bit and make it clear why this is needed and what problem this solves? Thank you! :)
hi @olamy! Thank you for reporting, I passed your message to the OCA team. In the meantime, can you please try again (and ideally from a different browser), maybe it was a one-time glitch?
@alina-yur @fniephaus all good. I tried again and it works. Thanks
Thank you for signing the OCA.
@olamy your OCA is now approved. happy contributing :)
Why do you need yet another skip parameter? Is
skipNativeBuild
not enough?
Using that in a sub-module would also make the sub-sub-modules skip native, which is not what we want.
Skipping by packaging type is more appropriate with maven than a global skip option like skipNativeBuild
.
Btw, just so you know, @olamy is a committer on the Apache Maven project itself, and is involved in countless maven plugin projects around the maven ecosystem. I trust his experience when it comes to proper maven plugin behavior and maintainability of both the maven plugin and the projects that use the maven plugin.
Using that in a sub-module would also make the sub-sub-modules skip native, which is not what we want. Skipping by packaging type is more appropriate with maven than a global skip option like
skipNativeBuild
.
Right, in that case I think the mojo should just simply skip execution for pom projects, without needing an extra parameter.
@olamy I understand your intention was to not introduce breaking changes, which I appreciate, but we can probably regard this issue as a bug, in which case it's fine to do the change I proposed.
Btw, just so you know, @olamy is a committer on the Apache Maven project itself, and is involved in countless maven plugin projects around the maven ecosystem. I trust his experience when it comes to proper maven plugin behavior and maintainability of both the maven plugin and the projects that use the maven plugin.
Thanks for the heads-up. I didn't mean to question his skills at all. Mine was a pure question that, in retrospect, could have been written differently. Sorry if it sounded too harsh, not my intention.
Using that in a sub-module would also make the sub-sub-modules skip native, which is not what we want. Skipping by packaging type is more appropriate with maven than a global skip option like
skipNativeBuild
.Right, in that case I think the mojo should just simply skip execution for pom projects, without needing an extra parameter.
There are many maven plugins that simply hardcode skipping of a project if it has no source, the easiest way is to skip pom
packaging projects, others take a holistic approach and ask the Project if any sources have been attached (eg: extracted from another dependency via the maven-dependency-plugin and then attached as source for compilation into non-default artifacts,, even though the project is pom
packaging).
This second approach also helps with nonstandard project packaging types (eg: bundle
, resource
, or any of a hundred other packaging types seen in the wild)
@alvarosanchez it looks @joakime explained perfectly why I choose this way of skipping :)
Thanks a lot for contributing and making clear why this is the right approach! :)
When using in a multi module Apache Maven build, the plugin can cause some issues for
pom
type project. SuchSee some details here https://github.com/jetty/jetty.project/issues/11683