Re: support for DESTDIR: security/openssh-portable

2006-08-11 Thread Gábor Kövesdán
Brooks Davis wrote: On Thu, Aug 10, 2006 at 03:25:38PM +0200, G?bor K?vesd?n wrote: Brooks Davis wrote: On Wed, Aug 09, 2006 at 05:59:18PM -0600, John E Hein wrote: John E Hein wrote at 17:43 -0600 on Aug 9, 2006: Well, the part that makes it annoying to duplicate i

Re: support for DESTDIR: security/openssh-portable

2006-08-10 Thread John E Hein
Brooks Davis wrote at 09:05 -0500 on Aug 10, 2006: > My inclination would be something like: > > PKG_INSTALL_TEMP=`mktemp ${DESTDIR}/tmp/pkg_install` && \ > (${CAT} ${PKG_INSTALL} > ${PKG_INSTALL_TEMP}; \ > ${SH} ${PKG_INSTALL_TEMP}; \ > ${RM} ${PKG_INSTALL_

Re: support for DESTDIR: security/openssh-portable

2006-08-10 Thread John E Hein
John E Hein wrote at 11:55 -0600 on Aug 10, 2006: > Brooks Davis wrote at 09:05 -0500 on Aug 10, 2006: > > I think we should ideally introduce a feature to allow ports to > > automatically run pkg-install and stuff the code in bsd.port.mk so > > ports don't have to know about DESTDIR in this

Re: support for DESTDIR: security/openssh-portable

2006-08-10 Thread Brooks Davis
On Thu, Aug 10, 2006 at 03:25:38PM +0200, G?bor K?vesd?n wrote: > Brooks Davis wrote: > >On Wed, Aug 09, 2006 at 05:59:18PM -0600, John E Hein wrote: > > > >>John E Hein wrote at 17:43 -0600 on Aug 9, 2006: > >> > Well, the part that makes it annoying to duplicate in all ports is not > >> > the

Re: support for DESTDIR: security/openssh-portable

2006-08-10 Thread Gábor Kövesdán
Brooks Davis wrote: On Wed, Aug 09, 2006 at 05:59:18PM -0600, John E Hein wrote: John E Hein wrote at 17:43 -0600 on Aug 9, 2006: > Well, the part that makes it annoying to duplicate in all ports is not > the two separate words (CHROOT DESTDIR), but that you have to test > defined(DESTDIR

Re: support for DESTDIR: security/openssh-portable

2006-08-10 Thread Brooks Davis
On Wed, Aug 09, 2006 at 05:59:18PM -0600, John E Hein wrote: > John E Hein wrote at 17:43 -0600 on Aug 9, 2006: > > Well, the part that makes it annoying to duplicate in all ports is not > > the two separate words (CHROOT DESTDIR), but that you have to test > > defined(DESTDIR) && !empty(DESTDI

Re: support for DESTDIR: security/openssh-portable

2006-08-09 Thread John E Hein
Gábor Kövesdán wrote at 01:47 +0200 on Aug 10, 2006: > Ah, you mean defining CHROOTDESTDIR only when DESTDIR is set and leave > it empty when not? It sounds reasonable then. I'll work this out after > some hours of sleeping. :) Yep... that's it. __

Re: support for DESTDIR: security/openssh-portable

2006-08-09 Thread John E Hein
John E Hein wrote at 17:43 -0600 on Aug 9, 2006: > Well, the part that makes it annoying to duplicate in all ports is not > the two separate words (CHROOT DESTDIR), but that you have to test > defined(DESTDIR) && !empty(DESTDIR) before you can figure out whether > to use ${CHROOT} ${DESTDIR} o

Re: support for DESTDIR: security/openssh-portable

2006-08-09 Thread Gábor Kövesdán
John E Hein wrote: Gábor Kövesdán wrote at 01:29 +0200 on Aug 10, 2006: > John E Hein wrote: > > John E Hein wrote at 16:31 -0600 on Aug 9, 2006: > > > Now that ports/Mk does the right thing for DESTDIR (thanks to Gábor), > > > here's a patch that supports DESTDIR properly for > > > secu

Re: support for DESTDIR: security/openssh-portable

2006-08-09 Thread John E Hein
Gábor Kövesdán wrote at 01:29 +0200 on Aug 10, 2006: > John E Hein wrote: > > John E Hein wrote at 16:31 -0600 on Aug 9, 2006: > > > Now that ports/Mk does the right thing for DESTDIR (thanks to Gábor), > > > here's a patch that supports DESTDIR properly for > > > security/openssh-portable

Re: support for DESTDIR: security/openssh-portable

2006-08-09 Thread Gábor Kövesdán
John E Hein wrote: John E Hein wrote at 16:31 -0600 on Aug 9, 2006: > Now that ports/Mk does the right thing for DESTDIR (thanks to Gábor), > here's a patch that supports DESTDIR properly for > security/openssh-portable: > [snip] > @@ -171,29 +171,33 @@ post-extract: > post-patch: >