Re: allow convenient library in subdirectory

[Tim Van Holder]

On 19 Jul 2001 19:16:47 -0600, Tom Tromey wrote:
> >>>>> "Tim" == Tim Van Holder <address@hidden> writes:
> 
> Tim> One nit with the patch: it hardcodes '.deps' as the depdir
> Tim> instead of using $(DEPDIR).
> 
> Just to be clear, this is a longstanding bug -- not Alexandre's fault.

I wasn't suggesting it was; I merely noticed it in the patch he sent in.

> Tim> +                push (@dep_list, &require_build_directory 
> ("$(DEPDIR)/$directory"))=0A=
> 
> Don't we want `\$(DEPDIR)'?

Yep - my bad (probably due to the single-quoted occurrence a few lines
lower).
Which do you prefer:
        "\$blah/$bleh"
or
        '$blah/' . $bleh
?

> Tim> +rmdir .deps 2>/dev/null=0A=
> 
> Shouldn't this be `rmdir $DEPDIR'?

Nope - the commands only ever try creating '.deps'; '_deps' only gets
chosen as DEPDIR if this fails.

--
Tim Van Holder - Anubex N.V.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
This message was posted using plain text. I do not endorse any
products or services that may be hyperlinked to this message.