Menu

#391 C++ Include checks are not recursive

open
nobody
None
5
2005-04-27
2005-04-27
Robert Blum
No

Example: Derived.cpp includes Derived.h. Derived.h
includes Base.h. Touch Base.h, and nant will happily
ignore the change.

(Also, once implemented, the recursive search should
terminate at the precompiled header, if there is any.
And at least for larger project, that dependency tree
should be cached...)

Discussion

  • Gert Driesen

    Gert Driesen - 2005-05-03

    Logged In: YES
    user_id=707851

    I don't have time to work on this right now. You don't happen
    to volunteer for this ?

     
  • Robert Blum

    Robert Blum - 2005-05-04

    Logged In: YES
    user_id=25786

    Ugh. I was afraid you'd ask.

    In fact, reading through the mailing list and the bug DB I
    get the feeling I'm the one of the very few who seriously
    uses Nant & C++ ;)

    But OK - as soon as I come up for air from my current tasks,
    I'll look into this. Have you guys given any thought and/or
    code to recursive dependency tracking at all? (I.e. caching
    of intermediates, updating dependencies through the compiler
    or by hand, etc...)

    Basically, anything to get me started ;)

     
  • Gert Driesen

    Gert Driesen - 2005-05-04

    Logged In: YES
    user_id=707851

    I've attached a fix for this issue to this bug report, but I'm not
    committing it in its current form as it sucks both performance
    and implementation wise ...

     
  • Robert Blum

    Robert Blum - 2005-05-04

    Logged In: YES
    user_id=25786

    Not seeing it. Are you sure you attached it?

    I've got a couple of ideas how to make this work decently
    and will take a look this weekend, if you want me to.

     
  • Gert Driesen

    Gert Driesen - 2005-05-05

    quick and (very) dirty fix

     
  • Bill Arnette

    Bill Arnette - 2006-12-21

    Logged In: YES
    user_id=1097659
    Originator: NO

    Robert, you are not the only poor soul still stuck in C++. So when is this going to make it into the release?

     

Log in to post a comment.