Re: Current binutils snapshots archives missing sequence number?

[Mike Frysinger]

On Friday 22 March 2013 01:25:44 Mike Frysinger wrote:
> On Friday 22 March 2013 00:12:38 Ian Lance Taylor wrote:
> > "Lavrentiev, Anton (NIH/NLM/NCBI) [C]" writes:
> > > There used to be something like a snapshot sequence number (52, 90)
> > > like in these:
> > > 
> > > binutils-2.22.52.tar.bz2  21249 KB        7/27/2012       12:00:00 AM
> > > binutils-2.22.90.tar.bz2  20362 KB        7/27/2012       12:00:00 AM
> > > 
> > > but seems no longer to be the case for more recent snapshot(s):
> > > 
> > > binutils-2.23.0.tar.bz2   20948 KB        11/6/2012       9:05:00 AM
> > > binutils.tar.bz2  22594 KB        3/18/2013       5:43:00 AM
> > > binutils-.tar.bz2         22594 KB        3/18/2013       5:43:00 AM
> > 
> > This may have been broken since this change:
> > 
> > 2012-07-27  Mike Frysinger  <address@hidden>
> > 
> >     * configure.in (AC_INIT): Call with the args bfd and 2.22.52.
> >     (AM_INIT_AUTOMAKE): Remove args.
> > 
> > See the way that VER is set in the top-level src-release script.
> 
> i have no idea how src-release gets invoked.  `git grep src-release`
> doesn't show anything useful,
> 
> maybe this patch will help:
> --- a/src-release
> +++ b/src-release
> @@ -67,7 +67,7 @@ SUPPORT_FILES =
> list-of-support-files-for-tool-in-question # NOTE: No double quotes in the
> below.  It is used within shell script # as VER="$(VER)"
>  VER = `      if grep 'AM_INIT_AUTOMAKE.*BFD_VERSION' $(TOOL)/configure.in
> >/dev/null 2>&1; then \ -       sed < bfd/configure.in -n
> 's/AM_INIT_AUTOMAKE[^,]*, *\([^)]*\))/\1/p'; \ +        bfd/configure 
> --version
> | sed -n -e '1s,.* ,,p'; \
>       elif grep AM_INIT_AUTOMAKE $(TOOL)/configure.in >/dev/null 2>&1; then \
>         sed < $(TOOL)/configure.in -n 's/AM_INIT_AUTOMAKE[^,]*,
> *\([^)]*\))/\1/p'; \ elif test -f $(TOOL)/version.in; then \

well, i can't make the situation worse right ? :)  the existing code will 
clearly fail, so i committed that snippet above:
2013-03-22  Mike Frysinger  <address@hidden>

    * src-release (VER): Change bfd/configure.in sed to use the new 
    `bfd/configure --version` output.
-mike

signature.asc
Description: This is a digitally signed message part.