> On March 5, 2014, 3:03 p.m., Dmitro Lisnichenko wrote: > > pom.xml, line 84 > > <https://reviews.apache.org/r/18732/diff/1/?file=509526#file509526line84> > > > > Lets consider switching to unix-maven-plugin > > http://mojo.codehaus.org/unix/unix-handbook/handbook.html#about-the-plugin > > . It's said to support deb, pkg and rpm formats. > > Using different plugins for rpm and deb (with different configurations > > and different shell scripts) may be highly error-prone and hard to maintain.
unix-maven-plugin (no.arktekk.unix,unix-maven-plugin,1.0-alpha-6) http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22unix-maven-plugin%22) are tested on small project and are failed during processing a filterFiles section: "[ERROR] Failed to execute goal no.arktekk.unix:unix-maven-plugin:1.0-alpha-6:package-rpm (default-cli) on project ambari: Unable to parse configuration of mojo no.arktekk.unix:unix-maven-plugin:1.0-alpha-6:package-rpm for parameter filterFiles: Abstract class or interface 'org.codehaus.mojo.unix.maven.plugin.AssemblyOp' cannot be instantiated -> [Help 1]" According to that, there are some problems in the plugin itself, due to version 1.0-alpha-5 have same problems. We can try to use only copy functionality of the plugin and filter resources by standard maven plugin, but there are possibility that plugin can fail (according to previous error) and seems that plugin are unsupported. As result, jdeb looks like one possible and more stable solution to build deb packages at the moment. - Vitaliy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18732/#review36225 ----------------------------------------------------------- On March 5, 2014, 1:21 p.m., Vitaliy Semenyk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18732/ > ----------------------------------------------------------- > > (Updated March 5, 2014, 1:21 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Sid Wagle. > > > Bugs: AMBARI-4934 > https://issues.apache.org/jira/browse/AMBARI-4934 > > > Repository: ambari > > > Description > ------- > > Patch added: > pom.xml: > - new package goal "jdeb", which allows to build deb packages > - "deb.*" properties > - maven-resources-plugin for /usr/sbin/ambari-[agent|server] and version > files (jdeb supports only control files filtering) > general: > - debian control files (src/main/package/deb/control/*) and updated for > Debian platform post-installation, pre-installation scripts > > Build command example: mvn package jdeb:jdeb > > > Diffs > ----- > > .gitignore 246e53e > ambari-agent/pom.xml 600d17d > ambari-agent/src/main/package/deb/control/control PRE-CREATION > ambari-agent/src/main/package/deb/control/postinst PRE-CREATION > ambari-agent/src/main/package/deb/control/postrm PRE-CREATION > ambari-agent/src/main/package/deb/control/posttrm PRE-CREATION > ambari-agent/src/main/package/deb/control/preinst PRE-CREATION > ambari-agent/src/main/package/deb/control/prerm PRE-CREATION > ambari-client/pom.xml 1bd4c84 > ambari-client/src/main/package/deb/control/control PRE-CREATION > ambari-client/src/main/package/deb/control/postinst PRE-CREATION > ambari-client/src/main/package/deb/control/postrm PRE-CREATION > ambari-client/src/main/package/deb/control/posttrm PRE-CREATION > ambari-client/src/main/package/deb/control/preinst PRE-CREATION > ambari-client/src/main/package/deb/control/prerm PRE-CREATION > ambari-project/pom.xml 426a074 > ambari-project/src/main/package/deb/control/control PRE-CREATION > ambari-project/src/main/package/deb/control/postinst PRE-CREATION > ambari-project/src/main/package/deb/control/postrm PRE-CREATION > ambari-project/src/main/package/deb/control/posttrm PRE-CREATION > ambari-project/src/main/package/deb/control/preinst PRE-CREATION > ambari-project/src/main/package/deb/control/prerm PRE-CREATION > ambari-server/pom.xml 0da58d1 > ambari-server/src/main/package/deb/control/control PRE-CREATION > ambari-server/src/main/package/deb/control/postinst PRE-CREATION > ambari-server/src/main/package/deb/control/postrm PRE-CREATION > ambari-server/src/main/package/deb/control/posttrm PRE-CREATION > ambari-server/src/main/package/deb/control/preinst PRE-CREATION > ambari-server/src/main/package/deb/control/prerm PRE-CREATION > ambari-views/pom.xml 5adc775 > ambari-views/src/main/package/deb/control/control PRE-CREATION > ambari-views/src/main/package/deb/control/postinst PRE-CREATION > ambari-views/src/main/package/deb/control/postrm PRE-CREATION > ambari-views/src/main/package/deb/control/posttrm PRE-CREATION > ambari-views/src/main/package/deb/control/preinst PRE-CREATION > ambari-views/src/main/package/deb/control/prerm PRE-CREATION > ambari-web/pom.xml c01d558 > ambari-web/src/main/assemblies/empty.xml PRE-CREATION > ambari-web/src/main/package/deb/control/control PRE-CREATION > ambari-web/src/main/package/deb/control/postinst PRE-CREATION > ambari-web/src/main/package/deb/control/postrm PRE-CREATION > ambari-web/src/main/package/deb/control/posttrm PRE-CREATION > ambari-web/src/main/package/deb/control/preinst PRE-CREATION > ambari-web/src/main/package/deb/control/prerm PRE-CREATION > pom.xml 33948b2 > src/main/assemblies/empty.xml PRE-CREATION > src/main/package/deb/control/control PRE-CREATION > src/main/package/deb/control/postinst PRE-CREATION > src/main/package/deb/control/postrm PRE-CREATION > src/main/package/deb/control/posttrm PRE-CREATION > src/main/package/deb/control/preinst PRE-CREATION > src/main/package/deb/control/prerm PRE-CREATION > > Diff: https://reviews.apache.org/r/18732/diff/ > > > Testing > ------- > > was changed pom.xml files and bash scripts so unit-test are not required > > > Thanks, > > Vitaliy Semenyk > >
