rpm plugin does not handle properties for rpm-specification files (parameter: 
defineStatement)
----------------------------------------------------------------------------------------------

                 Key: MOJO-1202
                 URL: http://jira.codehaus.org/browse/MOJO-1202
             Project: Mojo
          Issue Type: Bug
          Components: rpm
         Environment: openSuse 11, rpm-maven-plugin 2.0-beta-1
            Reporter: Philipp
            Priority: Blocker


Class rpm-maven-plugin/src/main/java/org/codehaus/mojo/rpm/RPMMojo.java does 
not handle
parameter defineStatement properly - thus it's not possible to pass parameters 
to RPM specification files.
Therefore we can not generate rpm files, since we do need more than one 
parameter in our pom.

Here's the patch:
rpm-maven-plugin/src/main/java/org/codehaus/mojo/rpm/RPMMojo.java

current version:
827             if ( null != defineStatements )
828             {
829                 Iterator defineIter = defineStatements.iterator();
830                 String defineStatement = (String) defineIter.next();
831                 spec.println( "%define " + defineStatement );
832             }

patched version:
827             if ( null != defineStatements )
828             {
829                 Iterator defineIter = defineStatements.iterator();
830                 while(defineIter.hasNext()) {
831                   String defineStatement = (String) defineIter.next();
832                   spec.println( "%define " + defineStatement );
833                 }
834             } 

Pls release a new version pretty soon.
Thx

P Ottlinger

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

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to