RE: Current binutils snapshots archives missing sequence number?

[Lavrentiev, Anton (NIH/NLM/NCBI) [C]]

Thanks!  It looks like you've nailed it:

binutils-2.23.52.tar.bz2        22705 KB        3/23/2013       5:39:00 AM

Anton Lavrentiev
Contractor NIH/NLM/NCBI

> -----Original Message-----
> From: Mike Frysinger [mailto:address@hidden
> Sent: Friday, March 22, 2013 7:21 PM
> To: address@hidden
> Cc: Ian Lance Taylor; Lavrentiev, Anton (NIH/NLM/NCBI) [C]
> Subject: Re: Current binutils snapshots archives missing sequence
> number?
> 
> 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