Menu

#46 FileNotFoundException when analyzing dependencies

open
nobody
cpptasks (35)
5
2006-12-04
2006-11-20
No

Hello,

Here is a build.xml reproducing an issue with the dependencies analysis, simply run ant at least two times to get the issue.

The problem happens when an included file has the same name as a directory, but the directory is found before among the includes.
The consequence is that it messes up the dependency computation and either does not rebuild when needed or always keeps rebuilding even when not needed.
As a side not is happens all the time with boost which has directories named like system include files (algorithm, math, etc..).

The fix is so simple that I truly really hope it gets commited soon.....

MAT.

Discussion

  • Mathieu Champlon

    test case

     
  • Mathieu Champlon

    fix

     
  • Mathieu Champlon

    • labels: --> cpptasks
     
  • Simon Martin

    Simon Martin - 2008-04-24

    Logged In: YES
    user_id=2071270
    Originator: NO

    Hi,

    seems that this fix isn't included in 1.0b5, right? Otherwise the fix doesn't work, at least when using Boost library.

     
  • Mathieu Champlon

    Logged In: YES
    user_id=71590
    Originator: YES

    Hi,

    Does the fix work for fixing the provided test case when you run it ?
    If it does could you modify the test case so that it fails again for your case using boost ?

    I've been using this fix for months now with quite a large number of projects (most of all using boost) and never experienced the FileNotFoundException again, so at least it has to fix something. :)

    MAT.

     
  • Simon Martin

    Simon Martin - 2008-04-25

    Logged In: YES
    user_id=2071270
    Originator: NO

    Okay, now I applied the patch correctly and it works for the provided test case AND my project. Sorry for confusion :)

    Will the patch be applied to the next (beta) release of cpptasks?

    BR

     
  • Thomas Brezinski

    it does not seem to be in 1.0b5. applied this though to 1.0b5 and so far i have not seen the issue. would be nice if we could get this committed.

     

Log in to post a comment.