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

Re: [Ovirt-devel] [PATCH] [ovirt-node] autobuild.sh



On Wed, Sep 10, 2008 at 02:30:47PM -0400, Mohammed Morsi wrote:
> ---
>  autobuild.sh |   43 +++++++++++++++++++++++++++++++++++++++++++
>  autogen.sh   |    1 +
>  2 files changed, 44 insertions(+), 0 deletions(-)
>  create mode 100755 autobuild.sh

> 
> diff --git a/autobuild.sh b/autobuild.sh
> new file mode 100755
> index 0000000..a1a1fd0
> --- /dev/null
> +++ b/autobuild.sh
> @@ -0,0 +1,43 @@
> +#!/bin/sh
> +#oVirt node autobuild script
> +#
> +# Copyright (C) 2008 Red Hat, Inc.
> +#
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; version 2 of the License.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA  02110-1301, USA.  A copy of the GNU General Public License is
> +# also available at http://www.gnu.org/copyleft/gpl.html.
> +
> +echo "Running oVirt Node Autobuild"
> +
> +set -e
> +set -v
> +
> +test -f Makefile && make -k distclean || :
> +
> +./autogen.sh --prefix=$AUTOBUILD_INSTALL_ROOT
> +make
> +make install
> +
> +rm -f *.tar.gz
> +make dist
> +
> +if [ -f /usr/bin/rpmbuild ]; then
> +  if [ -n "$AUTOBUILD_COUNTER" ]; then
> +    EXTRA_RELEASE=".auto$AUTOBUILD_COUNTER"
> +  else
> +    NOW=`date +"%s"`
> +    EXTRA_RELEASE=".$USER$NOW"
> +  fi 
> +  rpmbuild --nodeps --define "extra_release $EXTRA_RELEASE" -ta --clean *.tar.gz

You're missing the corresponding change in the specfile to actually
do something with the extra_release, eg change

  Release:        %(echo `awk '{ print $2 }' %{SOURCE1}`)%{?dist}

to

  Release:        %(echo `awk '{ print $2 }' %{SOURCE1}`)%{?dist}%{?extra_release}

ACK to all these autobuild.sh files though.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|


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