Il 11/Mar/2015 18:26 "Jeffrey Johnson" <n3...@me.com> ha scritto:
>
> Is a git remote checkout of config.guess really needed?
>
> Can't config.guess be checked in and copied into place in autogen.sh
instead?

It is another possibility yes but this implementation is more automatic. I
prefer but it is only an opinion.

If worth i can change sure.

Thanks

Best regards

>
> 73 de Jeff
> On Mar 11, 2015, at 1:07 PM, Pinto Elia wrote:
>
> >  RPM Package Manager, CVS Repository
> >  http://rpm5.org/cvs/
> >
____________________________________________________________________________
> >
> >  Server: rpm5.org                         Name:   Pinto Elia
> >  Root:   /v/rpm/cvs                       Email:  devzero2...@rpm5.org
> >  Module: popt                             Date:   11-Mar-2015 18:07:57
> >  Branch: popt_1_17                        Handle: 2015031117075700
> >
> >  Modified files:           (Branch: popt_1_17)
> >    popt                    autogen.sh
> >
> >  Log:
> >    *** empty log message ***
> >
> >  Summary:
> >    Revision    Changes     Path
> >    1.22.2.5    +95 -3      popt/autogen.sh
> >
____________________________________________________________________________
> >
> >  patch -p0 <<'@@ .'
> >  Index: popt/autogen.sh
> >
============================================================================
> >  $ cvs diff -u -r1.22.2.4 -r1.22.2.5 autogen.sh
> >  --- popt/autogen.sh  11 Mar 2015 16:59:09 -0000      1.22.2.4
> >  +++ popt/autogen.sh  11 Mar 2015 17:07:57 -0000      1.22.2.5
> >  @@ -1,6 +1,6 @@
> >   #!/bin/sh
> >   #
> >  -# $Id: autogen.sh,v 1.22.2.4 2015/03/11 16:59:09 devzero2000 Exp $
> >  +# $Id: autogen.sh,v 1.22.2.5 2015/03/11 17:07:57 devzero2000 Exp $
> >   # autogen.sh: autogen.sh script for popt projects
> >   #
> >   # Copyright (c) 2010-2011 Elia Pinto <devzero2...@rpm5.org>
> >  @@ -22,6 +22,13 @@
> >     blu=' [1;34m'; \
> >     std=' [m'; \
> >   }
> >  +#
> >  +# git repository for autoconf config.guess and config.sub
> >  +#
> >  +_CONFIG_URL="http://git.savannah.gnu.org/gitweb/";
> >
+_CONFIG_GUESS_URL="${_CONFIG_URL}?p=config.git;a=blob_plain;f=config.guess;hb=HEAD"
> >
+_CONFIG_SUB_URL="${_CONFIG_URL}?p=config.git;a=blob_plain;f=config.guess;hb=HEAD"
> >  +
> >
> >   Die()    {
> >           color="$red"
> >  @@ -133,6 +140,64 @@
> >     # can't depend on column -t
> >   }
> >
> >  +usage ( ) {
> >  +
> >  +cat <<EOF
> >  +Usage: ${_PROGNAME} [-h|--help]  [-d|--download]
> >  +
> >  +--help      Help on $NAME_OF_AUTOGEN usage
> >  +--download  Download the latest config.guess from gnulib
> >  +
> >  +EOF
> >  +
> >  +    return 0
> >  +}
> >  +
> >  +download_gnulib_config_guess () {
> >  +    config_guess_temp="config.guess.$$.download"
> >  +    config_sub_temp="config.sub.$$.download"
> >  +    ret=1
> >  +    for __cmd in wget curl fetch ; do
> >  +    ${__cmd} --version > /dev/null 2>&1
> >  +    ret=$?
> >  +    if [ ! $ret = 0 ] ; then
> >  +        continue
> >  +        fi
> >  +
> >  +    __cmd_version=`${__cmd} --version | head -n 1 | sed -e
's/^[^0-9]\+//' -e 's/ .*//'`
> >  +
> >  +    opts=""
> >  +    case ${__cmd} in
> >  +        wget)
> >  +            opts="--timeout=5  -O"
> >  +            ;;
> >  +        curl)
> >  +            opts="--max-time=5 -o"
> >  +            ;;
> >  +        fetch)
> >  +            opts="-t 5 -f"
> >  +            ;;
> >  +    esac
> >  +
> >  +    eval "$__cmd \"${_CONFIG_GUESS_URL}\" $opts
\"${config_guess_temp}\"" > /dev/null 2>&1
> >  +    if [ $? = 0 ] ; then
> >  +        mv -f "${config_guess_temp}" ${_aux_dir}/config.guess
> >  +            eval "$__cmd \"${_CONFIG_SUB_URL}\" $opts
\"${config_sub_temp}\"" > /dev/null 2>&1
> >  +        if [ $? = 0 ] ; then
> >  +            mv -f "${config_sub_temp}" ${_aux_dir}/config.sub
> >  +            ret=0
> >  +            break
> >  +        fi
> >  +        fi
> >  +    done
> >  +
> >  +    if [ ! $ret = 0 ] ; then
> >  +    Notice "Warning: config.{guess,sub} download failed from
${_CONFIG_URL}"
> >  +    rm -f "${config_guess_temp}"
> >  +    rm -f "${config_sub_temp}"
> >  +    fi
> >  +}
> >  +
> >   #######################
> >   # Begin  Bootstrapping
> >   #######################
> >  @@ -144,6 +209,24 @@
> >   gettext    0.19
> >   libtool        1.5.22
> >   "
> >  +##################
> >  +# argument check #
> >  +##################
> >  +ARGS="$*"
> >  +#
> >  +for arg in $ARGS ; do
> >  +    case "x$arg" in
> >  +    x--help) usage && exit 0;;
> >  +    x-[dD]) DOWNLOAD=yes ;;
> >  +    x--download) DOWNLOAD=yes ;;
> >  +    *)
> >  +        echo "${_PROGNAME}:Unknown option: $arg"
> >  +        echo
> >  +        usage
> >  +        exit 1
> >  +        ;;
> >  +    esac
> >  +done
> >   echo
> >   Notice "Bootstrapping popt build system..."
> >   echo
> >  @@ -156,8 +239,16 @@
> >     Die "could not find configure.ac or configure.in"
> >     echo
> >   fi
> >  -
> >  -if ! printf "$buildreq" | check_versions; then
> >  +#
> >  +# detect the aux dir
> >  +# for config.{sub,guess}
> >  +aux_dir="`grep AC_CONFIG_AUX_DIR $conffile | grep -v
'.*#.*AC_CONFIG_AUX_DIR' | tail -${TAIL_N}1 | sed 's/^[
]*AC_CONFIG_AUX_DIR(\(.*\)).*/\1/' | sed 's/.*\[\(.*\)\].*/\1/'`"
> >  +if test ! -d "$_aux_dir"
> >  +then
> >  +        _aux_dir=.
> >  +fi
> >  +#
> >  +if ! printf "$buildreq" | check_versions; then test -f README-prereq
&&
> >     test -f README-prereq &&
> >     echo
> >     echo "See README-prereq for notes on obtaining these prerequisite
programs:" >&2
> >  @@ -190,6 +281,7 @@
> >                 sed 's|.*/||; s|\.po$||' > "$po_dir/LINGUAS"
> >
> >   #
> >  +[ ${DOWNLOAD} = "yes" ] && download_gnulib_config_guess
> >   echo
> >   Notice "done.  Now you can run './configure'."
> >   #######################
> >  @@ .
> > ______________________________________________________________________
> > RPM Package Manager                                    http://rpm5.org
> > CVS Sources Repository                                rpm-...@rpm5.org
>
> ______________________________________________________________________
> RPM Package Manager                                    http://rpm5.org
> Developer Communication List                        rpm-devel@rpm5.org

Reply via email to