Re: Assembly ID error with mvn-rpmbuild

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/18/2013 08:37 AM, Simone Caronni wrote:
> Many thanks!
> 
> On 18 July 2013 05:14, Mikolaj Izdebski <mizdebsk@xxxxxxxxxx> wrote:
> 
>> Patching assembly descriptor should help:
>>
>>   sed -i '/<assembly>/a\  <id>foo</id>' src/main/assembly/static.xml
>>
>> Alternatively add <appendAssemblyId>false</appendAssemblyId> to plugin
>> configuration (recommended in the FAQ, but not by me).
>>
> 
> This patch directly from the developer fixes it:
> [...]

Right.  I made a very similar fix in one of my projects 2 months ago,
but I forgot that appendAssemblyId is really needed indeed.

  https://github.com/mizdebsk/xmvn/commit/96ea6d3

commit 96ea6d3b879e7738fa8c5c496e09e875233e313b
Author: Mikolaj Izdebski <mizdebsk@xxxxxxxxxx>
Date:   Fri May 17 10:35:56 2013 +0200

    Add mandatory assembly ID

    Version of assembly plugin was recently bumped.  In currently used
    version assembly ID is mandatory, see MASSEMBLY-464.

diff --git a/pom.xml b/pom.xml
index 50c1773..decb149 100644
--- a/pom.xml
+++ b/pom.xml
@@ -89,6 +89,7 @@
               <goal>single</goal>
             </goals>
             <configuration>
+              <appendAssemblyId>false</appendAssemblyId>
               <descriptors>
                 <descriptor>src/main/assembly/bin.xml</descriptor>
               </descriptors>
diff --git a/src/main/assembly/bin.xml b/src/main/assembly/bin.xml
index 992c9ed..db104d0 100644
--- a/src/main/assembly/bin.xml
+++ b/src/main/assembly/bin.xml
@@ -15,6 +15,7 @@
  ! limitations under the License.
  `-->
 <assembly>
+  <id>bin</id>
   <formats>
     <format>tar.bz2</format>
   </formats>


-- 
Mikolaj Izdebski
Software Engineer, Red Hat
IRC: mizdebsk
--
java-devel mailing list
java-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/java-devel





[Index of Archives]     [Red Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux