INDEX patch

Oliver Eikemeier eikemeier at
Wed Jun 2 06:50:53 PDT 2004

Kris Kennaway wrote:

> On Wed, Jun 02, 2004 at 03:40:06PM +0200, Oliver Eikemeier wrote:
>>Kris Kennaway wrote:
>>>On Wed, Jun 02, 2004 at 02:47:07PM +0200, Oliver Eikemeier wrote:
>>>>Kris Kennaway wrote:
>>>>>A previous commit to broke the reporting of missing
>>>>>dependencies from 'make index'.  This patch restores it (and makes
>>>>>these errors fatal).  I plan to commit it soon.
>>>>May I suggest the following patch:
>>>>- check for existence of perl (cf. my RFC on ports-developers)
>>>>- enables the use of `env PATH= /usr/bin/make describe' (nice for testing)
>>>Nice idea, but running this test 11000 times is suboptimal :-)
>>Because test is a sh-builtin, and the file system should cache this 
>>I don't expect to see any difference. Especially since spawning perl 11000 
>>is more than suboptimal ;) You didn't mean this serious, did you?
> I did, because I care about small pessimizations of the INDEX build.
> Sometimes they're not small.  Did you benchmark it?

No. Did you? Honestly, when you really want to improve INDEX building times, it
should be worth thinking about not spawning perl every time, but instead do the
substitutions in batch. I my experience microoptimization on the expense of
clarity and prudence almost always result in highly optimized suboptimal
algorithms that together run far slower that they have to.


More information about the freebsd-ports mailing list