← Back to team overview

yade-dev team mailing list archive

Re: [Branch ~yade-dev/yade/trunk] Rev 2449: 1. Rename InteractionDispatchers to InteractionLoop; backwards-compat for python as usual, typede...

 

Sorry to be annoying, but it does not work:

if not os.path.exists(combinedFiles) or
combinedFilesMd5!=md5.md5(open(combinedFiles).read()).hexdigest():
    print 'Rebuilding combined files, since the md5 has changed.'
    combs=[l.split(':')[0] for l in open(combinedFiles)]
    for c in combs:
        env.AlwaysBuild(c)
        env.Default(c)


if the file does not exist, we can not read it in 3-rd string. May I just
skip it, something like that:

if os.path.exists(combinedFiles)
*and*combinedFilesMd5!=md5.md5(open(combinedFiles).read()).hexdigest():
    print 'Rebuilding combined files, since the md5 has changed.'
    combs=[l.split(':')[0] for l in open(combinedFiles)]
    for c in combs:
        env.AlwaysBuild(c)
        env.Default(c)


Or it will not work?
Thanks

Anton



On Thu, Sep 30, 2010 at 6:18 PM, Vaclav Smilauer <eudoxos@xxxxxxxx> wrote:

>
> > I just do not clearly understand what combinedFiles is for.
> It tracks how are source files combined together, to overcome bugs with
> duplicate definitions etc (it was reported a few months back). The file
> is written at each scons run and contents is compared. You can run
> compilation and look in that file. For debian packages, it is not
> applicable, since the build always starts from scratch.
>
> Cheers, v.
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~yade-dev<https://launchpad.net/%7Eyade-dev>
> Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~yade-dev<https://launchpad.net/%7Eyade-dev>
> More help   : https://help.launchpad.net/ListHelp
>

References