Re: Mk/bsd.openssl.mk optimization

2008-08-04 Thread Doug Barton
V.Chukharev wrote: On Mon, 04 Aug 2008 09:20:04 +0300, Doug Barton <[EMAIL PROTECTED]> wrote: Vladimir Chukharev wrote: On Tue, 29 Jul 2008 23:13:44 +0300, Doug Barton <[EMAIL PROTECTED]> wrote: V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch

Re: Mk/bsd.openssl.mk optimization

2008-08-04 Thread V.Chukharev
On Mon, 04 Aug 2008 09:20:04 +0300, Doug Barton <[EMAIL PROTECTED]> wrote: > Vladimir Chukharev wrote: >> On Tue, 29 Jul 2008 23:13:44 +0300, Doug Barton <[EMAIL PROTECTED]> >> wrote: >> >>> V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch f

Re: Mk/bsd.openssl.mk optimization

2008-08-03 Thread Doug Barton
Vladimir Chukharev wrote: On Tue, 29 Jul 2008 23:13:44 +0300, Doug Barton <[EMAIL PROTECTED]> wrote: V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch for bsd.port.subdir.mk. I use: grep -l "@comment ORIGIN:${1}$" $pdb/*/+CONTENTS for similar purp

Re: Mk/bsd.openssl.mk optimization

2008-07-30 Thread Kris Kennaway
Vladimir Chukharev wrote: On Wed, 30 Jul 2008 14:09:16 +0300, Kris Kennaway <[EMAIL PROTECTED]> wrote: V.Chukharev wrote: I have found why there is a difference in INDEX-7 generated with and without caching. Without cache security/p5-openxpki* (6 connected ports) are indexed as depending o

Re: Mk/bsd.openssl.mk optimization

2008-07-30 Thread Vladimir Chukharev
On Wed, 30 Jul 2008 14:09:16 +0300, Kris Kennaway <[EMAIL PROTECTED]> wrote: > V.Chukharev wrote: >> I have found why there is a difference in INDEX-7 generated with and without >> caching. >> Without cache security/p5-openxpki* (6 connected ports) are indexed as >> depending on >> both openssl

Re: Mk/bsd.openssl.mk optimization

2008-07-30 Thread Kris Kennaway
V.Chukharev wrote: On Tue, 29 Jul 2008 18:28:26 +0300, V.Chukharev <[EMAIL PROTECTED]> wrote: On Fri, 25 Jul 2008 20:19:42 +0300, Kris Kennaway <[EMAIL PROTECTED]> wrote: You don't want to .include the file, just use the same make -f trick that I used for e.g. the python and java variable cac

Re: Mk/bsd.openssl.mk optimization

2008-07-30 Thread V.Chukharev
On Tue, 29 Jul 2008 18:28:26 +0300, V.Chukharev <[EMAIL PROTECTED]> wrote: > On Fri, 25 Jul 2008 20:19:42 +0300, Kris Kennaway <[EMAIL PROTECTED]> wrote: > >> You don't want to .include the file, just use the same make -f trick >> that I used for e.g. the python and java variable caching. > > Ah,

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread Vladimir Chukharev
On Tue, 29 Jul 2008 23:13:44 +0300, Doug Barton <[EMAIL PROTECTED]> wrote: > V.Chukharev wrote: >> Another patch, just one line. It can be applied independently from the patch >> for bsd.port.subdir.mk. > > I use: grep -l "@comment ORIGIN:${1}$" $pdb/*/+CONTENTS > for similar purposes in portmaste

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread Sean C. Farley
On Tue, 29 Jul 2008, Freddie Cash wrote: On July 29, 2008 12:52 pm V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch for bsd.port.subdir.mk. = --- /usr/ports/Mk/bsd.openssl.mk.orig 2008-07-23 09:14:29.0 +0300 +++ /usr/ports/Mk

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread Kris Kennaway
Freddie Cash wrote: On July 29, 2008 12:52 pm V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch for bsd.port.subdir.mk. = --- /usr/ports/Mk/bsd.openssl.mk.orig 2008-07-23 09:14:29.0 +0300 +++ /usr/ports/Mk/bsd.openssl.mk 2008-

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread Freddie Cash
On July 29, 2008 12:52 pm V.Chukharev wrote: > Another patch, just one line. It can be applied independently from the > patch for bsd.port.subdir.mk. > > = > --- /usr/ports/Mk/bsd.openssl.mk.orig 2008-07-23 09:14:29.0 > +0300 +++ /usr/ports/Mk/bsd.openssl.mk2008-07-29 20

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread Doug Barton
V.Chukharev wrote: Another patch, just one line. It can be applied independently from the patch for bsd.port.subdir.mk. I use: grep -l "@comment ORIGIN:${1}$" $pdb/*/+CONTENTS for similar purposes in portmaster with no complaints so far. Is there a reason you need to do the complicated thing?

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread V.Chukharev
Another patch, just one line. It can be applied independently from the patch for bsd.port.subdir.mk. = --- /usr/ports/Mk/bsd.openssl.mk.orig 2008-07-23 09:14:29.0 +0300 +++ /usr/ports/Mk/bsd.openssl.mk2008-07-29 20:35:05.0 +0300 @@ -121,7 +121,7 @@ exi

Re: Mk/bsd.openssl.mk optimization

2008-07-29 Thread V.Chukharev
On Fri, 25 Jul 2008 20:19:42 +0300, Kris Kennaway <[EMAIL PROTECTED]> wrote: > V.Chukharev wrote: >> With the patch Mk/bsd.port.sudir.mk .include's Mk/bsd.openssl.mk, hence >> everywhere >> when the former is used the latter is inderectly included. This is because I >> did not >> want to split t

Re: Mk/bsd.openssl.mk optimization

2008-07-27 Thread Kris Kennaway
Dirk Meyer wrote: Kris Kennaway schrieb:, To repeat, why can't you set the openssl variables in bsd.port.subdir.mk and pass them in to the port-level make process instead of recomputing them with every port? Becuase if someone does "make package" in ports/security The value must change durin

Re: Mk/bsd.openssl.mk optimization

2008-07-26 Thread Kris Kennaway
Dirk Meyer wrote: Kris Kennaway schrieb:, openssl does set NOPRECIOUSMAKEVARS=yes, so that part ist skipped in bsd.port.subdir.mk. OK, but why is this necessary? otherwise the build breaks, as the length of the commandline reaches a limit. We're not talking about builds, but other processi

Re: Mk/bsd.openssl.mk optimization

2008-07-26 Thread Kris Kennaway
Dirk Meyer wrote: Kris Kennaway schrieb:, The result can be cached only if no other ports or package is installed. So you are right that make index could be otimized, but it will take much more work. Why can't you cache it in bsd.port.subdir.mk as I suggested? openssl does set NOPRECIOUSMAKE

Re: Mk/bsd.openssl.mk optimization

2008-07-25 Thread Kris Kennaway
V.Chukharev wrote: On Fri, 25 Jul 2008 16:09:10 +0300, V.Chukharev <[EMAIL PROTECTED]> wrote: On Fri, 25 Jul 2008 07:30:18 +0300, Dirk Meyer <[EMAIL PROTECTED]> wrote: So you are right that make index could be otimized, but it will take much more work. Totally agree on this. I tryed to make

Re: Mk/bsd.openssl.mk optimization

2008-07-25 Thread V.Chukharev
On Fri, 25 Jul 2008 16:09:10 +0300, V.Chukharev <[EMAIL PROTECTED]> wrote: > On Fri, 25 Jul 2008 07:30:18 +0300, Dirk Meyer <[EMAIL PROTECTED]> wrote: > >> So you are right that make index could be otimized, >> but it will take much more work. > > Totally agree on this. I tryed to make caching in

Re: Mk/bsd.openssl.mk optimization

2008-07-25 Thread V.Chukharev
On Fri, 25 Jul 2008 07:30:18 +0300, Dirk Meyer <[EMAIL PROTECTED]> wrote: > V.Chukharev schrieb:, > >> I should confess I do not understand how my patch works. I expected to see a >> file /usr/ports/.openssl_installed or /tmp/index_something/.openssl_installed >> or similar, but could not see it..

Re: Mk/bsd.openssl.mk optimization

2008-07-25 Thread Kris Kennaway
Dirk Meyer wrote: V.Chukharev schrieb:, So I looked through some Mk stuff, and tryed to optimize it. Since I do not know any magic of bsd.*.mk, I did just a simple test. And with the below shown patch the time is only 1500 s, while with the original version the time is more than 15000 s. I gues

Re: Mk/bsd.openssl.mk optimization

2008-07-23 Thread Kris Kennaway
V.Chukharev wrote: Dear maintainer, I have noticed that most time of 'make index' on my system takes a grep for libssl. The following is done while 'make index' was running. $ ps axww | grep libssl 23119 p1 IN+0:00,00 sh -c grep -l -r "^lib/libssl.so." "/var/db/pkg" | while read content

Mk/bsd.openssl.mk optimization

2008-07-23 Thread V.Chukharev
Dear maintainer, I have noticed that most time of 'make index' on my system takes a grep for libssl. The following is done while 'make index' was running. $ ps axww | grep libssl 23119 p1 IN+0:00,00 sh -c grep -l -r "^lib/libssl.so." "/var/db/pkg" | while read contents; do sslprefix=`gr