Re: [perl PATCH v2] Switch over to using Module::Build

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

 



On Tue, 2018-05-15 at 18:54 +0100, Daniel P. Berrangé wrote:
[...]
> diff --git a/prepare-release.sh b/prepare-release.sh
> index fea03f4..6850350 100755
> --- a/prepare-release.sh
> +++ b/prepare-release.sh
> @@ -4,52 +4,18 @@ NAME=Sys-Virt
>  
>  set -e
>  
> -test -n "$1" && RESULTS=$1 || RESULTS=results.log
> +rm -rf blib _build Build $NAME-*.tar.gz
>  
> -make -k realclean ||:
> -rm -rf MANIFEST blib pm_to_blib
> +test -z "$AUTOBUILD_INSTALL_ROOT" && AUTOBUILD_INSTALL_ROOT=$HOME/builder

Please don't reintroduce $AUTOBUILD_INSTALL_ROOT: just leave this
line alone and...

>  export TEST_MAINTAINER=1
>  
> -perl Makefile.PL PREFIX=$HOME/builder
> +perl Build.PL install_base=$AUTOBUILD_INSTALL_ROOT

... use install_base=$HOME/builder here.


With the caveat that I know nothing about Perl, the changes look
sane enough and I tested them without running into any issue, so

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

Please coordinate before pushing to make sure the CI environment
has been updated by the time the first Module::Build is triggered.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux