Subject: no-longer-necessary configure patches
To: None <tech-pkg@netbsd.org>
From: Dan Winship <danw@helixcode.com>
List: tech-pkg
Date: 04/12/2000 11:30:56
There are a bunch of packages with patches to configure to work around
a problem with INSTALL that hasn't existed since tron fixed bsd.pkg.mk
about a year ago:

  revision 1.225
  date: 1999/03/10 18:31:02;  author: tron;  state: Exp;  lines: +2 -2
  If "GNU_CONFIGURE" is set to "yes" supply "${INSTALL}" with full pathname
  again because "configure" will create bogus makefiles otherwise.

The packages involved are:

  archivers/liblzo archivers/lzop audio/libaudiofile audio/xmms
  corba/ORBit devel/bonobo devel/glib devel/gtexinfo devel/guile-gtk
  devel/m4 graphics/xsane mail/balsa math/gnumeric math/octave
  print/gnome-print textproc/libxml x11/gnome-libs x11/gtk-- x11/gtk

I'm a pkgsrc novice, so I wanted to make sure I'm doing this right: I
removed the relevant bits from the patch files, and ran "make
makepatchsum". Now I have a bunch of diffs like:

  Index: archivers/liblzo/files/patch-sum
  ===================================================================
  RCS file: /cvsroot/pkgsrc/archivers/liblzo/files/patch-sum,v
  retrieving revision 1.2
  diff -u -r1.2 patch-sum
  --- patch-sum 1999/12/22 19:46:23	1.2
  +++ patch-sum 2000/04/12 15:03:47
  @@ -1,3 +1,3 @@
   $NetBSD: patch-sum,v 1.2 1999/12/22 19:46:23 rh Exp $
   
  -MD5 (patch-aa) = db9cfdab42dba833d1268cc90224d110
  +MD5 (patch-aa) = ac6ea2fdce6b48c7c3faf0a033ebc93a
  Index: archivers/liblzo/patches/patch-aa
  ===================================================================
  RCS file: /cvsroot/pkgsrc/archivers/liblzo/patches/patch-aa,v
  retrieving revision 1.2
  diff -u -r1.2 patch-aa
  --- patch-aa  1999/12/22 19:46:24	1.2
  +++ patch-aa  2000/04/12 15:03:47
  @@ -18,16 +18,3 @@
    
    # Redirect the config.log output again, so that the ltconfig log is
      not
    # clobbered by the next message.
  -@@ -4050,10 +4052,12 @@
  -     top_srcdir="$ac_dots$ac_given_srcdir" ;;
  -   esac
  - 
  -+if false; then
  -   case "$ac_given_INSTALL" in
  -   [/$]*) INSTALL="$ac_given_INSTALL" ;;
  -   *) INSTALL="$ac_dots$ac_given_INSTALL" ;;
  -   esac
  -+fi
  - 
  -   echo creating "$ac_file"
  -   rm -f "$ac_file"

I tested glib, and it still works fine. I didn't actually test the
others... Anyway, is this ok to commit?

-- Dan