[E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Morten Nilsen
this patch makes two very small, but good changes.. - don't create .spec in dirs without autogen.sh - use a shared config.cache -- Cheers, Morten :wq Index: autogen.sh === RCS file: /var/cvs/e/e_modules/autogen.sh,v retrieving

Re: [E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Michael Jennings
On Wednesday, 24 May 2006, at 09:55:05 (+0200), Morten Nilsen wrote: #!/bin/sh +cache=`mktemp` + for d in * ; do test -d $d || continue - sed s,%MODNAME%,$d,g e_modules-TEMPLATE.spec.in $d/e_modules-$d.spec.in test -x ${d}/autogen.sh || continue + sed

Re: [E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Morten Nilsen
Michael Jennings wrote: Morten Nilsen wrote: +cache=`mktemp` You realize this will create a separate shared cache file every time it's run and never remove any of them I'd say that's a bad thing. clearly, there should be an rm towards the end there.. the point is not to share it for

Re: [E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Michael Jennings
On Wednesday, 24 May 2006, at 16:57:25 (+0200), Morten Nilsen wrote: clearly, there should be an rm towards the end there.. the point is not to share it for eternity, but to cut down a bit on runtime, as the same things will be tested a lot of times otherwise.. The problem with that, of

Re: [E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Morten Nilsen
Michael Jennings wrote: Morten Nilsen wrote: clearly, there should be an rm towards the end there.. The problem with that, of course, is that each run becomes fresh. Normally, after you've run ./configure once, the cache persists until it is invalidated somehow. If you remove it each time,

Re: [E-devel] teeny patch for e_modules autogen.sh

2006-05-24 Thread Michael Jennings
On Wednesday, 24 May 2006, at 17:35:35 (+0200), Morten Nilsen wrote: How about replacing `mktemp` with `pwd`/config.cache then? That sounds like a winner to me. Good idea. :-) Michael -- Michael Jennings (a.k.a. KainX) http://www.kainx.org/ [EMAIL PROTECTED] n + 1, Inc.,