libtool-patches
[Top][All Lists]
Advanced

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

RE: [FYI] update NEWS on 1.5.1 in one go, because I'm lazy


From: Boehne, Robert
Subject: RE: [FYI] update NEWS on 1.5.1 in one go, because I'm lazy
Date: Sat, 24 Jan 2004 12:29:26 -0600

Bob,

I would agree that it should go in, I've had to work around this myself.

Robert

-----Original Message-----
From: Bob Friesenhahn [mailto:address@hidden
Sent: Saturday, January 24, 2004 12:10 PM
To: address@hidden
Subject: Re: [FYI] update NEWS on 1.5.1 in one go, because I'm lazy


I find this patch/feature very useful.  However, since it adds a new
feature and this is a bug-fix branch, I would like to hear agreement
from other libtool maintainers before applying this patch.

Bob

On Sat, 24 Jan 2004, Albert Chin wrote:

> On Fri, Jan 23, 2004 at 11:22:18PM +0000, Scott James Remnant wrote:
> > On Fri, 2004-01-23 at 22:31, Albert Chin wrote:
> >
> > > On Fri, Jan 23, 2004 at 07:48:40PM +0000, Scott James Remnant wrote:
> > > > On Fri, 2004-01-23 at 08:30, Gary V.Vaughan wrote:
> > > >
> > > > > Please commit to HEAD as well: It's nice to have all the NEWS in one
> > > > > file when you are browsing a HEAD release.  And it helps us check that
> > > > > all sensible branch bug-fixes are ported to HEAD before we roll a new
> > > > > point-oh.
> > > > >
> > > > Will wait to see whether Peter wants anything else backported into
> > > > branch-1-5 before releasing.  Then I'll copy the NEWS block into HEAD
> > > > and adjust the "1.6" one accordingly.
> > >
> > > How about AC_LIBTOOL_TAGS in branch-1-5?
> >
> > This is pretty reliant on nearly all of Gary's tidying where tags and
> > tag variables are concerned, porting this to 1.5.x would basically mean
> > just copying HEAD onto branch-1-5 which I don't think we should do.
>
> No. We submitted the initial patch against 1.5. I've synced with the
> latest 1.5 branch and am repositing it. We've used it extensively here
> and have not run into any problems. So, I'm for including it. I'll
> backport the documentation from 1.6 if the powers that be agree it
> should go in.
>
> --
> albert chin (address@hidden)
>
> -- snip snip
> Index: libtool.m4
> ===================================================================
> RCS file: /cvsroot/libtool/libtool/Attic/libtool.m4,v
> retrieving revision 1.314.2.25
> diff -u -3 -p -r1.314.2.25 libtool.m4
> --- libtool.m4        23 Jan 2004 06:06:50 -0000      1.314.2.25
> +++ libtool.m4        24 Jan 2004 07:31:26 -0000
> @@ -39,35 +39,7 @@ m4_ifdef([AC_PROVIDE_IFELSE],
>  # ---------------
>  AC_DEFUN([AC_PROG_LIBTOOL],
>  [AC_REQUIRE([_AC_PROG_LIBTOOL])dnl
> -dnl If AC_PROG_CXX has already been expanded, run AC_LIBTOOL_CXX
> -dnl immediately, otherwise, hook it in at the end of AC_PROG_CXX.
> -  AC_PROVIDE_IFELSE([AC_PROG_CXX],
> -    [AC_LIBTOOL_CXX],
> -    [define([AC_PROG_CXX], defn([AC_PROG_CXX])[AC_LIBTOOL_CXX
> -  ])])
> -dnl And a similar setup for Fortran 77 support
> -  AC_PROVIDE_IFELSE([AC_PROG_F77],
> -    [AC_LIBTOOL_F77],
> -    [define([AC_PROG_F77], defn([AC_PROG_F77])[AC_LIBTOOL_F77
> -])])
> -
> -dnl Quote A][M_PROG_GCJ so that aclocal doesn't bring it in needlessly.
> -dnl If either AC_PROG_GCJ or A][M_PROG_GCJ have already been expanded, run
> -dnl AC_LIBTOOL_GCJ immediately, otherwise, hook it in at the end of both.
> -  AC_PROVIDE_IFELSE([AC_PROG_GCJ],
> -    [AC_LIBTOOL_GCJ],
> -    [AC_PROVIDE_IFELSE([A][M_PROG_GCJ],
> -      [AC_LIBTOOL_GCJ],
> -      [AC_PROVIDE_IFELSE([LT_AC_PROG_GCJ],
> -     [AC_LIBTOOL_GCJ],
> -      [ifdef([AC_PROG_GCJ],
> -          [define([AC_PROG_GCJ], defn([AC_PROG_GCJ])[AC_LIBTOOL_GCJ])])
> -       ifdef([A][M_PROG_GCJ],
> -          [define([A][M_PROG_GCJ], defn([A][M_PROG_GCJ])[AC_LIBTOOL_GCJ])])
> -       ifdef([LT_AC_PROG_GCJ],
> -          [define([LT_AC_PROG_GCJ],
> -             defn([LT_AC_PROG_GCJ])[AC_LIBTOOL_GCJ])])])])
> -])])# AC_PROG_LIBTOOL
> +]) # AC_PROG_LIBTOOL
>
>
>  # _AC_PROG_LIBTOOL
> @@ -226,9 +199,8 @@ AC_ARG_WITH([pic],
>  test -z "$pic_mode" && pic_mode=default
>
>  # Use C for the default configuration in the libtool script
> -tagname=
>  AC_LIBTOOL_LANG_C_CONFIG
> -_LT_AC_TAGCONFIG
> +_LT_AC_TAG_CONFIG
>  ])# AC_LIBTOOL_SETUP
>
>
> @@ -1625,15 +1604,29 @@ test "$dynamic_linker" = no && can_build
>  ])# AC_LIBTOOL_SYS_DYNAMIC_LINKER
>
>
> -# _LT_AC_TAGCONFIG
> +# AC_LIBTOOL_TAGS
> +# ---------------
> +# tags to enable
> +AC_DEFUN([AC_LIBTOOL_TAGS],
> +[m4_define([_LT_TAGS],[$1])
> +]) # AC_LIBTOOL_TAGS
> +
> +# _LT_AC_TAG_CHECK
>  # ----------------
> -AC_DEFUN([_LT_AC_TAGCONFIG],
> -[AC_ARG_WITH([tags],
> -    [AC_HELP_STRING([--with-tags@<:@=TAGS@:>@],
> -        [include additional configurations @<:@automatic@:>@])],
> -    [tagnames="$withval"])
> +AC_DEFUN([_LT_AC_TAG_CHECK],
> +[m4_ifdef([_LT_TAG_]$1,
> +  [m4_errprintn(m4_location[: error: duplicate tag: ]"$1")
> +  m4_exit(1)],
> +  [m4_define([_LT_TAG_]$1, [])])
> +]) # _LT_AC_TAG_CHECK
> +
> +# _LT_AC_TAG_CONFIG
> +# -----------------
> +AC_DEFUN([_LT_AC_TAG_CONFIG],
> +[AC_PROVIDE_IFELSE([AC_LIBTOOL_TAGS],,
> +  [AC_LIBTOOL_TAGS([CXX F77 GCJ RC])])
>
> -if test -f "$ltmain" && test -n "$tagnames"; then
> +if test -f "$ltmain"; then
>    if test ! -f "${ofile}"; then
>      AC_MSG_WARN([output file `$ofile' does not exist])
>    fi
> @@ -1651,66 +1644,33 @@ if test -f "$ltmain" && test -n "$tagnam
>    # Note that this assumes the entire list is on one line.
>    available_tags=`grep "^available_tags=" "${ofile}" | $SED -e 
> 's/available_tags=\(.*$\)/\1/' -e 's/\"//g'`
>
> -  lt_save_ifs="$IFS"; IFS="${IFS}$PATH_SEPARATOR,"
> -  for tagname in $tagnames; do
> -    IFS="$lt_save_ifs"
> -    # Check whether tagname contains only valid characters
> -    case `$echo "X$tagname" | $Xsed -e 
> 's:[[-_ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890,/]]::g'`
>  in
> -    "") ;;
> -    *)  AC_MSG_ERROR([invalid tag name: $tagname])
> -     ;;
> -    esac
> -
> -    if grep "^# ### BEGIN LIBTOOL TAG CONFIG: $tagname$" < "${ofile}" > 
> /dev/null
> -    then
> -      AC_MSG_ERROR([tag name \"$tagname\" already exists])
> -    fi
> -
> -    # Update the list of available tags.
> -    if test -n "$tagname"; then
> -      echo appending configuration tag \"$tagname\" to $ofile
> -
> -      case $tagname in
> -      CXX)
> -     if test -n "$CXX" && test "X$CXX" != "Xno"; then
> -       AC_LIBTOOL_LANG_CXX_CONFIG
> -     else
> -       tagname=""
> -     fi
> -     ;;
> -
> -      F77)
> -     if test -n "$F77" && test "X$F77" != "Xno"; then
> -       AC_LIBTOOL_LANG_F77_CONFIG
> -     else
> -       tagname=""
> -     fi
> -     ;;
> -
> -      GCJ)
> -     if test -n "$GCJ" && test "X$GCJ" != "Xno"; then
> -       AC_LIBTOOL_LANG_GCJ_CONFIG
> -     else
> -       tagname=""
> -     fi
> -     ;;
> -
> -      RC)
> -     AC_LIBTOOL_LANG_RC_CONFIG
> -     ;;
> -
> -      *)
> -     AC_MSG_ERROR([Unsupported tag name: $tagname])
> -     ;;
> -      esac
> -
> -      # Append the new tag name to the list of available tags.
> -      if test -n "$tagname" ; then
> -      available_tags="$available_tags $tagname"
> -    fi
> -    fi
> -  done
> -  IFS="$lt_save_ifs"
> +  AC_FOREACH([_LT_TAG], _LT_TAGS,
> +      [m4_case(_LT_TAG,
> +      [CXX], [_LT_AC_TAG_CHECK([CXX])
> +  echo appending configuration tag \"CXX\" to $ofile
> +  if test -n "$CXX" && test "X$CXX" != "Xno"; then
> +      AC_LIBTOOL_LANG_CXX_CONFIG
> +      available_tags="$available_tags _LT_TAG"
> +  fi],
> +      [F77], [_LT_AC_TAG_CHECK(_LT_TAG)
> +  echo appending configuration tag \"_LT_TAG\" to $ofile
> +  if test -n "$F77" && test "X$F77" != "Xno"; then
> +      AC_LIBTOOL_LANG_F77_CONFIG
> +      available_tags="$available_tags _LT_TAG"
> +  fi],
> +      [GCJ], [_LT_AC_TAG_CHECK(_LT_TAG)
> +  echo appending configuration tag \"_LT_TAG\" to $ofile
> +  if test -n "$GCJ" && test "X$GCJ" != "Xno"; then
> +      AC_LIBTOOL_LANG_GCJ_CONFIG
> +      available_tags="$available_tags _LT_TAG"
> +  fi],
> +      [RC], [_LT_AC_TAG_CHECK(_LT_TAG)
> +  echo appending configuration tag \"_LT_TAG\" to $ofile
> +  AC_LIBTOOL_LANG_RC_CONFIG
> +  available_tags="$available_tags _LT_TAG"],
> +      [m4_errprintn(m4_location[: error: invalid tag name: ]"_LT_TAG")
> +      m4_exit(1)])
> +  ])
>
>    # Now substitute the updated list of available tags.
>    if eval "sed -e 
> 's/^available_tags=.*\$/available_tags=\"$available_tags\"/' \"$ofile\" > 
> \"${ofile}T\""; then
> @@ -1721,7 +1681,7 @@ if test -f "$ltmain" && test -n "$tagnam
>      AC_MSG_ERROR([unable to update list of available tagged configurations.])
>    fi
>  fi
> -])# _LT_AC_TAGCONFIG
> +])# _LT_AC_TAG_CONFIG
>
>
>  # AC_LIBTOOL_DLOPEN
> @@ -3976,7 +3944,7 @@ if test "X\${CDPATH+set}" = Xset; then C
>  available_tags=
>
>  # ### BEGIN LIBTOOL CONFIG],
> -[# ### BEGIN LIBTOOL TAG CONFIG: $tagname])
> +[# ### BEGIN LIBTOOL TAG CONFIG: $1])
>
>  # Libtool was configured on host `(hostname || uname -n) 2>/dev/null | sed 
> 1q`:
>
> @@ -4272,7 +4245,7 @@ include_expsyms=$lt_[]_LT_AC_TAGVAR(incl
>
>  ifelse([$1],[],
>  [# ### END LIBTOOL CONFIG],
> -[# ### END LIBTOOL TAG CONFIG: $tagname])
> +[# ### END LIBTOOL TAG CONFIG: $1])
>
>  __EOF__
>
>
>
> _______________________________________________
> Libtool-patches mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/libtool-patches
>

======================================
Bob Friesenhahn
address@hidden
http://www.simplesystems.org/users/bfriesen



_______________________________________________
Libtool-patches mailing list
address@hidden
http://mail.gnu.org/mailman/listinfo/libtool-patches


- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
- - - - - - - - -  - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
- - -
This e-mail and any files transmitted with it are confidential and intended 
solely for the use of the individual or entity to whom they are addressed.  If 
you have received this email in error please notify the sender immediately and 
delete this e-mail from your system.  Please note that any views or opinions 
presented in this email are solely those of the author and do not necessarily 
represent those of Ricardo (save for reports and other documentation formally 
approved and signed for release to the intended recipient).  Only Directors or 
Duly Authorised Officers are authorised to enter into legally binding 
obligations on behalf of Ricardo unless the obligation is contained within a 
Ricardo Purchase Order.  Ricardo may monitor outgoing and incoming e-mails and 
other telecommunications on its e-mail and telecommunications systems.  
By replying to this e-mail you give your consent to such monitoring.  The 
recipient should check this email and any attachments for the presence of 
viruses.  Ricardo accepts no liability for any damage caused by any virus 
transmitted by this email.  'Ricardo' means Ricardo Plc and its subsidiary 
companies.

Ricardo plc is a public limited company registered in England with registered 
number 00222915. The registered office of Ricardo plc is Bridge Works, 
Shoreham-by-Sea, West Sussex, BN43 5FG.
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
- - - - - - - - -- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
- - - 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]