Re: [jamboost] Re: bad dependency generation

From: Vladimir Prus (ghost_at_[hidden])
Date: 2005-05-26 00:34:28

On Wednesday 25 May 2005 19:05, David Abrahams wrote:
> Vladimir Prus <ghost_at_[hidden]> writes:
> > On Wednesday 25 May 2005 18:20, David Abrahams wrote:
> >> Vladimir Prus <ghost_at_[hidden]> writes:
> >> > - DEPENDS $(rsp) : $(sources) ;
> >>
> >> What makes it OK to delete this line?
> >
> > Your own comment! Mentioned in the patch itself:
> >
> > + # Note: we don't need dependecy from response file on sources
> > + # because response file only needs the names of the sources.
>
> Uhm, wait. I did say the response file only needs the names of the
> sources. I did *NOT* draw the conclusion that we don't need a
> dependency on the sources.

Uhm.. then I must have drawn that conclusion myself.

> > That's true, response file will be created before the target, and it only
> > need the name of sources, not their content, and so should not depend on
> > them.
>
> That can never work. If the .rsp doesn't exist to begin with, then
> the target will never built, because it depends on the .rsp and there
> are no instructions for building the .rsp.

Sorry, you're wrong. There are instructions for building .rsp file, they 
are given in common.response-file. The .rsp files has no dependencies,
but that's no problem -- it still will be rebuilt.

Did you try the patch in practice. For me, it does generates .rsp files!

- Volodya

-- 
Vladimir Prus
http://vladimir_prus.blogspot.com
Boost.Build V2: http://boost.org/boost-build2