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

Re: AVR32 port - config.{sub,guess} bug filing



On Fri, Apr 24, 2009 at 01:37:01PM +0800, Paul Wise wrote:

> cur_v=`echo "$timestamp" | sed s/-//g`
> 
> for path in \
>   "$HOME/.config/automake" \
>   /usr/local/share/automake \
>   /usr/local/share/misc \
>   /usr/share/automake \
>   /usr/share/misc \
>     ; do
> 
>     if test -x "$path/config.sub" ; then
>       v=`"$path/config.sub" --time-stamp | sed s/-//g`
>       if test "$v" -gt "$cur_v" ; then
>         "$path/config.sub" $*
>         exit $?
>       fi
>     fi
> done

Hmm, wouldn't it be better to look for the newest version instead of
"first one that's newer"? The above would work on a buildd where there is
nothing in $HOME or under /usr/local, but could break on user's machines
if they have a script in say /usr/local/share/automake that's newer than
what the package has, but still older than what would be needed to build
correctly.

Gabor

-- 
     ---------------------------------------------------------
     MTA SZTAKI Computer and Automation Research Institute
                Hungarian Academy of Sciences
     ---------------------------------------------------------


Reply to: