Re: pkg/36191: new package -- devel/xdelta3

[B K <mtbakerguy%yahoo.com@localhost>]

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

From: B K <mtbakerguy%yahoo.com@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/36191: new package -- devel/xdelta3
Date: Sun, 22 Apr 2007 21:24:01 -0700 (PDT)

 You're right that the BUILD_TARGET setting worked to
 do most of what I wanted.  Since there's no install
 target, we'll still need a minimal patch-aa to do the
 following though:
 
 +install:
 +       ${INSTALL}  -c -s -m 555 xdelta3 ${PREFIX}/bin
 
 AFAICT, XD3_USE_LARGEFILE64 is always enabled.
 
 --Brad
 
 --- Tobias Nygren <tnn%NetBSD.org@localhost> wrote:
 
 > The following reply was made to PR pkg/36191; it has
 > been noted by GNATS.
 > 
 > From: Tobias Nygren <tnn%NetBSD.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: mrbakerguy%yahoo.com@localhost
 > Subject: Re: pkg/36191: new package -- devel/xdelta3
 > Date: Sun, 22 Apr 2007 17:06:03 +0200
 > 
 >  I had a look at this and it looks good for the most
 > part.
 >  But I don't understand the big Makefile changes in
 > patch-aa.
 >  Could you try to remove patch-aa and set
 > BUILD_TARGET= xdelta3
 >  in the pkgsrc Makefile instead? I think that's what
 > you're
 >  trying to do? :)
 >  Also I think you should always enable use of
 > LARGEFILE64.
 >  
 >  -Tobias
 >  
 > 
 
 
 __________________________________________________
 Do You Yahoo!?
 Tired of spam?  Yahoo! Mail has the best spam protection around 
 http://mail.yahoo.com