Upgrade unrelated: remove default (super POM) settings
------------------------------------------------------

                 Key: MIFOS-4524
                 URL: http://mifosforge.jira.com/browse/MIFOS-4524
             Project: mifos
          Issue Type: Dev Task
            Reporter: Michael Vorburger


The Mifos pom.xml sometimes contain "too much" - they repeat things which in 
normal Maven usage are inherited from the parent pom.xml (in our case that 
mifos-parent) and usually not repeated..

A specific problem I ran into while working on MIFOS-4099 was that the 
<build><outputDirectory> was fixed to target/classes in several pom.xml - there 
is no need to repeat such a default in every pom.xml (on my branch in 
mifos-parent it's overriden, but because the default was repeated it didn't 
work; background as to why it's overridden is on 
http://mifosforge.jira.com/wiki/display/MIFOS/WorkspaceEclipseMavenSettings#WorkspaceEclipseMavenSettings-SeparateEclipseandMavenbuildfolders)

It looks almost as if somebody thought it would be useful to repeat all of 
Maven's "super POM" default configurations in every Mifos pom.xml - that's an 
anti-pattern IMHO, and should be cleared out.

It would also make for shorter and more readable pom.xml files.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://mifosforge.jira.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
February 28th, so secure your free ArcSight Logger TODAY! 
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
Mifos-issues mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mifos-issues

Reply via email to