Subject: Re: pkg/34337: [update] sysutils/file
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: Joerg Sonnenberger <joerg@britannica.bec.de>
List: pkgsrc-bugs
Date: 08/29/2006 15:15:18
The following reply was made to PR pkg/34337; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg@britannica.bec.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/34337: [update] sysutils/file
Date: Tue, 29 Aug 2006 17:12:42 +0200

 On Tue, Aug 29, 2006 at 02:55:07PM +0000, Martin Wilke wrote:
 > Index: buildlink3.mk
 > ===================================================================
 > RCS file: /home/pcvs/pkgsrc/sysutils/file/buildlink3.mk,v
 > retrieving revision 1.6
 > diff -u -r1.6 buildlink3.mk
 > --- buildlink3.mk	8 Jul 2006 23:11:08 -0000	1.6
 > +++ buildlink3.mk	29 Aug 2006 07:06:21 -0000
 > @@ -13,8 +13,8 @@
 >  BUILDLINK_ORDER:=	${BUILDLINK_ORDER} ${BUILDLINK_DEPTH}file
 >  
 >  .if !empty(FILE_BUILDLINK3_MK:M+)
 > -BUILDLINK_API_DEPENDS.file+=	file>=4.09
 > -BUILDLINK_ABI_DEPENDS.file+=	file>=4.09nb1
 > +BUILDLINK_API_DEPENDS.file+=	file>=4.17
 > +BUILDLINK_ABI_DEPENDS.file+=	file>=4.17nb1
 >  BUILDLINK_PKGSRCDIR.file?=	../../sysutils/file
 >  .endif	# FILE_BUILDLINK3_MK
 >  
 
 Are you sure that the API *and* ABI changed? To clarify the difference:
 API_DEPENDS means that the programming interface was changed and most
 packages are expected to use the new API. When it is only extended in a
 backward compatible way, it should be left as is.
 
 ABI_DEPENDS bumps should only occur when e.g. a shared library version
 changed and means that all depending packages need a bump as well.
 
 I don't remember either of that required.
 
 Joerg
 
 > Index: patches/patch-aa
 > ===================================================================
 > RCS file: /home/pcvs/pkgsrc/sysutils/file/patches/patch-aa,v
 > retrieving revision 1.2
 > diff -u -r1.2 patch-aa
 > --- patches/patch-aa	18 Feb 2005 19:39:27 -0000	1.2
 > +++ patches/patch-aa	29 Aug 2006 07:00:53 -0000
 > @@ -1,21 +1,16 @@
 > -$NetBSD: patch-aa,v 1.2 2005/02/18 19:39:27 reed Exp $
 > -
 > ---- src/Makefile.in.orig	2005-01-07 11:17:26.000000000 -0800
 > -+++ src/Makefile.in
 > -@@ -84,14 +84,14 @@ am__include = @am__include@
 > - am__quote = @am__quote@
 > - fsect = @fsect@
 > - install_sh = @install_sh@
 > +--- src/Makefile.in.orig	2006-03-02 23:09:48.000000000 +0100
 > ++++ src/Makefile.in	2006-08-29 08:59:39.000000000 +0200
 > +@@ -192,11 +192,11 @@
 > + sharedstatedir = @sharedstatedir@
 > + sysconfdir = @sysconfdir@
 > + target_alias = @target_alias@
 >  -MAGIC = $(pkgdatadir)/magic
 >  +MAGIC = @sysconfdir@/magic
 >   lib_LTLIBRARIES = libmagic.la
 >   include_HEADERS = magic.h
 >   EXTRA_DIST = test.c
 > - 
 > - bin_PROGRAMS = file
 > - 
 >  -AM_CPPFLAGS = -DMAGIC='"$(MAGIC)"'
 >  +AM_CPPFLAGS = -DMAGIC='"$(MAGIC):$(pkgdatadir)/magic"'
 > - 
 >   libmagic_la_SOURCES = magic.c apprentice.c softmagic.c ascmagic.c \
 >   	compress.c is_tar.c readelf.c print.c fsmagic.c \
 > + 	funcs.c file.h names.h patchlevel.h readelf.h tar.h apptype.c
 
 Patches should be generated e.g. with pkgtools/pkgdiff.
 
 Joerg