Hi Roy,

Sure you can easily implement such a FileVault validator rule, but that IMHO 
doesn’t require any changes on the Validation API nor any skip logic.
I would just make the rule configurable to check for banned content package ids.
That way it is really quick and doesn’t do harm if it is executed for every sub 
package.
I would appreciate if such a rule could be donated to ASF FileVault so other 
could use it too easily.

Konrad

> On 7. Jul 2023, at 07:31, Roy Teeuwen <r...@teeuwen.be> wrote:
> 
> Hey Konrad,
> 
> Sure, the case is the following:
> 
> I have the following reactor module:
> 
> - all
> - core
> - ui.apps
> - ui.content
> - it.content
> 
> I only want the it.content to be installed to specific environments, namely 
> local and an automated builds environment. To do this, the it.content is 
> defined in the embedded section of the filevault plugin, but I check if an 
> environment variable is available and profile-wise add the it.content 
> dependency. The setting failOnMissingEmbed is set to 'false'.
> 
> I now had the case that another submodule created a dependency on it.content, 
> making it.content available as dependency to the 'all' package and installed 
> on the wrong environment. 
> To fix this, I'd like to see if I can add a FileVault validation rule to the 
> 'all' package to state one of the following (whichever one is doable):
> 
> - it should not contain an embedded package named it.content
> - it should not contain any subpackage that has as a filter 
> /content/${mysite} (preferable, because that way you actually really check 
> what you don't want to happen, even if it would be added on accident to any 
> other package)
> 
> Thanks!
> Roy
> 
> 
>> On 6 Jul 2023, at 07:51, Konrad Windszus <k...@apache.org> wrote:
>> 
>> Hi Roy,
>> Usually one needs to distinguish between sub packages provided from outside 
>> Maven reactor (which should be skipped) and sub packages coming from reactor 
>> modules. As the latter ones are already checked individually in that case 
>> using skipSubPackageValidation is usually sufficient as checking the sub 
>> packages again won’t emit different validation issues.
>> Maybe you can elaborate a bit on your use case...
>> Feel free, though, to open a JIRA issue and (in the best case) provide a PR 
>> for skipping sub packages with specific Maven coordinates.
>> Thanks,
>> Konrad
>> 
>>> On 5. Jul 2023, at 21:13, Roy Teeuwen <r...@teeuwen.be> wrote:
>>> 
>>> Hey all,
>>> 
>>> I'd like to skip the validation for only a certain subpackage (with other 
>>> subpackages contained in that subpackage). Is this possible? I see that 
>>> there is a skipSubPackageValidation, but this skips all sub packages while 
>>> I only want to do one. I also see you can make custom validatorSettings, 
>>> but if I understand this correctly, I'd have to specify every validator to 
>>> isDisabled true, which would become a long list. So i'm wondering if there 
>>> is a shorter way?
>>> 
>>> Greets,
>>> Roy
>> 
> 

Reply via email to