Re: pkg/32961: netpbm-10.32 fails to "make install".

[Neil Hoggarth <neil.hoggarth%physiol.ox.ac.uk@localhost>]

The following reply was made to PR pkg/32961; it has been noted by GNATS.

From: Neil Hoggarth <neil.hoggarth%physiol.ox.ac.uk@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
        pkgsrc-bugs%netbsd.org@localhost
Subject: Re: pkg/32961: netpbm-10.32 fails to "make install".
Date: Thu, 2 Mar 2006 10:26:44 +0000 (GMT)

 On Wed, 1 Mar 2006, joerg%britannica.bec.de@localhost wrote:
 
 > Please check the output of the *build* phase for possible compilation
 > errors.
 
 The build phase completes successfully, and I've not managed to find any 
 errors in the output. There is also no mention of pnmdepth in the build 
 phase output.
 
 <http://netpbm.sourceforge.net/doc/pamdepth.html> says that pnmdepth has 
 been newly replaced in this particular release of NetPBM.
 
 The install phase output contains the following:
 
 gmake[1]: Entering directory 
`/export/scratch/pkgsrc/graphics/netpbm/work/netpbm-10.32/other'
 [...]
 cd  /export/scratch/pkgsrc/graphics/netpbm/work/staging/bin ; \
    rm -f pnmsplit ; \
    ln -s pamsplit pnmsplit
 cd  /export/scratch/pkgsrc/graphics/netpbm/work/staging/bin ; \
    rm -f pnmsdepth ; \
    ln -s pnmsdepth pnmdepth
 gmake[1]: Leaving directory 
`/export/scratch/pkgsrc/graphics/netpbm/work/netpbm-10.32/other'
 
 which looks wrong to me - the final rm/ln combination explicitly creates 
 a dangling symlink. I'd expect the latter couple of lines to be 
 something like:
 
    rm -f pnmdepth ; \
    ln -s pamdepth pnmdepth
 
 The erroneous make instructions appear to come from the NetPBM release
 (but get exposed on pkgsrc because of the "staging" that is part of
 the install process for this package?). I have now managed to get the
 package to install, by manually applying the following change:
 
 --- work/netpbm-10.32/other/Makefile.old       Tue Jan  3 22:47:15 2006
 +++ work/netpbm-10.32/other/Makefile   Thu Mar  2 10:20:21 2006
 @@ -66,7 +66,7 @@
          $(SYMLINK) pamsplit$(EXE) pnmsplit
   # In February 2006, pamdepth replaced pnmdepth
        cd $(PKGDIR)/bin ; \
 -        rm -f pnmsdepth ; \
 -        $(SYMLINK) pnmsdepth$(EXE) pnmdepth
 +        rm -f pnmdepth ; \
 +        $(SYMLINK) pamdepth$(EXE) pnmdepth
 
   FORCE:
 
 
 Regards,
 -- 
 Neil Hoggarth  -------------------  Computing Manager, Sherrington Building
 Department of Physiology, Anatomy and Genetics  -  University of Oxford, UK