maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Romain Manni-Bucau (JIRA)" <>
Subject [jira] [Comment Edited] (MCHECKSTYLE-267) maven merge policy breaks checkstyle inline configuration
Date Sat, 17 Mar 2018 16:10:00 GMT


Romain Manni-Bucau edited comment on MCHECKSTYLE-267 at 3/17/18 4:09 PM:

:thumb_up: , thanks a lot

was (Author: romain.manni-bucau):
:thulb_up: , thanks a lot

> maven merge policy breaks checkstyle inline configuration
> ---------------------------------------------------------
>                 Key: MCHECKSTYLE-267
>                 URL:
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>    Affects Versions: 2.13
>            Reporter: Romain Manni-Bucau
>            Priority: Major
>         Attachments: pom.xml, pom.xml
> This is surely a bug in maven itself since it appears in effective pom but please redirect
and link this issue if you think so (it can be worked around in checkstyle plugin and I'm
not sure maven cares)
> Issue is: you have a pom and a child pom. Both configure checkstyle "inline" (ie without
checkstyle.xml but directly in <configuration>). In this case maven merge both configs
(great) but tag by tag. it means if both checkstyle configs dont have the same modules then
configuraiton is broken and the child module config can slip to another module for instance.
> Workaround would be to get raw config of both pom and do the merge in the plugin. Correct
solution IMO is to consider attribute values during maven merge.

This message was sent by Atlassian JIRA

View raw message