Re: [PATCH] Add a "make rpmball" target

[Don Slutz <dslutz@verizon.com>]

On 03/03/14 11:07, George Dunlap wrote:
> On 03/03/2014 02:59 PM, George Dunlap wrote:
>> Build a simplistic dummy package, similar to "make debball", for
>> developers on rpm-based systems.
>>
>> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>
> I meant to add, I've tested this running as root, but not as a normal user -- I'd appreciate it if someone could give it a try...
>

As a user I get:
...
sh ./tools/misc/mkrpm /home/don/xen $(make -C xen xenversion | grep -v :)
error: line 3: Illegal char '-' in: Version: 4.5-unstable
make: *** [rpmball] Error 1

    -Don Slutz


>  -George
>
>> ---
>> CC: Ian Jackson <ian.jackson@citrix.com>
>> CC: Ian Campbell <ian.campbell@citrix.com>
>> CC: Dario Faggioli <dario.faggioli@citrix.com>
>> ---
>>   Makefile         |    7 +++++
>>   tools/misc/mkrpm |   82 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   2 files changed, 89 insertions(+)
>>   create mode 100644 tools/misc/mkrpm
>>
>> diff --git a/Makefile b/Makefile
>> index 4e48457..75e845b 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -152,6 +152,13 @@ world:
>>   debball: dist
>>       fakeroot sh ./tools/misc/mkdeb $(XEN_ROOT) $$($(MAKE) -C xen xenversion | grep -v :)
>>   +# Package a build in an rpmball file, that is inside a .rpm format
>> +# container to allow for easy and clean removal. This is not intended
>> +# to be a full featured policy compliant .rpm package.
>> +.PHONY: rpmball
>> +rpmball: dist
>> +    sh ./tools/misc/mkrpm $(XEN_ROOT) $$($(MAKE) -C xen xenversion | grep -v :)
>> +
>>   # clean doesn't do a kclean
>>   .PHONY: clean
>>   clean::
>> diff --git a/tools/misc/mkrpm b/tools/misc/mkrpm
>> new file mode 100644
>> index 0000000..7d20788
>> --- /dev/null
>> +++ b/tools/misc/mkrpm
>> @@ -0,0 +1,82 @@
>> +#!/bin/sh
>> +#
>> +# mkrpm: package the dist/install output of a Xen build in an .rpm
>> +#
>> +# Takes 2 arguments, the path to the dist directory and the version
>> +
>> +set -e
>> +
>> +if test -z "$1" -o -z "$2" ; then
>> +  echo "usage: $0 path-to-XEN_ROOT xen-version"
>> +  exit 1
>> +fi
>> +
>> +xenroot="$1"
>> +
>> +cd $xenroot
>> +version=$2
>> +
>> +# Prepare the directory to package
>> +cd dist
>> +rm -rf rpm
>> +
>> +# Fill in the rpm boilerplate
>> +mkdir -p rpm/SPEC
>> +cat >rpm/SPEC/xen.spec <<EOF
>> +Summary: Xen development build, version $version
>> +Name: xen
>> +Version: $version
>> +Release: unstable
>> +License: GPL
>> +Group:   System/Hypervisor
>> +URL: http://xenbits.xenproject.org/xen.git
>> +
>> +BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root
>> +%define __spec_install_post /usr/lib/rpm/brp-compress || :
>> +%define debug_package %{nil}
>> +
>> +%description
>> +This package contains the Xen hypervisor and associated tools, built
>> +from a source tree.  It is not a fully packaged and supported Xen, just
>> +the output of a xen "make dist" wrapped in an .rpm to make it easy to
>> +uninstall.
>> +
>> +%build
>> +
>> +%install
>> +rm -rf \$RPM_BUILD_ROOT
>> +mkdir -p \$RPM_BUILD_ROOT
>> +cd %{_xenroot}
>> +dist/install.sh \$RPM_BUILD_ROOT/
>> +
>> +cd \$RPM_BUILD_ROOT
>> +
>> +# Don't include xen-syms
>> +rm -f boot/xen-syms*
>> +# Remove all the "linked" xen-*.gz files
>> +rm -f \$(find boot/ -type l)
>> +# Rename the real xen.gz binary, if necessary.
>> +if [[ ! -e boot/%{name}-%{version}-%{release}.gz ]] ; then
>> +    mv boot/xen-*.gz boot/%{name}-%{version}-%{release}.gz
>> +fi
>> +
>> +
>> +%clean
>> +rm -rf \$RPM_BUILD_ROOT
>> +
>> +%files
>> +%defattr(-,root,root,-)
>> +/*
>> +
>> +%post
>> +# Point xen.gz to the real binary.
>> +cd /boot
>> +ln -sf %{name}-%{version}-%{release}.gz xen.gz
>> +EOF
>> +
>> +# Package it up
>> +rpmbuild --define "_xenroot $xenroot" --define "_topdir $PWD/rpm" -bb rpm/SPEC/xen.spec
>> +
>> +# Tidy up after ourselves
>> +mv rpm/RPMS/*/*.rpm .
>> +rm -rf rpm
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel