[Scons-dev] Python 2to3
Tim Jenness
tjenness at lsst.org
Thu Sep 22 19:04:32 EDT 2016
Maybe add a slash after “build”? So “build/"
> On Sep 22, 2016, at 16:03, Bill Deegan <bill at baddogconsulting.com> wrote:
>
> when you run bootstrap.py it creates the build and bootstrap directory trees.
> Those shouldn't be added to source control.
> Perhaps they're not specified specifically enough
>
> -Bill
>
> On Thu, Sep 22, 2016 at 3:30 PM, William Blevins <wblevins001 at gmail.com <mailto:wblevins001 at gmail.com>> wrote:
> In the .hgignore file, the first line "build" is apparently causing "build.py" to be skipped. I assume this is because it exists before the syntax:glob. Do you know the purpose of the first 2 lines. They are probably not behaving as intended.
>
> V/R,
> William
>
> On Thu, Sep 22, 2016 at 6:16 PM, Bill Deegan <bill at baddogconsulting.com <mailto:bill at baddogconsulting.com>> wrote:
> Looks like this was removed:
>
> -
>
> -test.write(['work4', 'build.py'], """\
>
> -import sys
>
> -outfp = open(sys.argv[1], 'wb')
>
> -for f in sys.argv[2:]:
>
> - outfp.write(open(f, 'rb').read())
>
> -outfp.close()
>
> -""")
>
> But no file added to the fixture.
>
> I've fixed and pushed. I'm betting it's in your local repo, but didn't get committed?
>
> -Bill
>
> On Thu, Sep 22, 2016 at 2:57 PM, William Blevins <wblevins001 at gmail.com <mailto:wblevins001 at gmail.com>> wrote:
> Bill,
>
> Do you know if the QMTest fixture commands have been used in anger? That first issue worries me a bit. The later one, not so much.
>
> V/R,
> William
>
> On Thu, Sep 22, 2016 at 5:56 PM, William Blevins <wblevins001 at gmail.com <mailto:wblevins001 at gmail.com>> wrote:
> 1. test/Actions/pre-post.py
>
> I moved the file writes into a fixture. On my machine this is fine, I wonder if the QMTest dir_fixture/file_fixture FS commands hadn't written to disk before the test script ran?
>
> 2. test/Parallel/build_failure.py
>
> I didn't change this file. Since it has sleep commands, I wouldn't be surprised if its just a race condition in the test itself.
>
> PS. Try running them individually and see if they fail...
>
> V/R,
> William
>
> On Thu, Sep 22, 2016 at 5:41 PM, William Blevins <wblevins001 at gmail.com <mailto:wblevins001 at gmail.com>> wrote:
> Yes.
>
> On Thu, Sep 22, 2016 at 4:47 PM, Bill Deegan <bill at baddogconsulting.com <mailto:bill at baddogconsulting.com>> wrote:
> William,
>
> Looks like a couple new failures.
> http://buildbot.scons.org/builders/bd-ubuntu-python-2.7/builds/43/steps/shell/logs/stdio <http://buildbot.scons.org/builders/bd-ubuntu-python-2.7/builds/43/steps/shell/logs/stdio>
>
> Can you take a look and resolve if possible?
>
> Thanks,
> Bill
>
> On Thu, Sep 22, 2016 at 10:55 AM, William Blevins <wblevins001 at gmail.com <mailto:wblevins001 at gmail.com>> wrote:
> No worries. Some others are not, so it's a fair point.
>
> On Thu, Sep 22, 2016 at 1:53 PM, Dirk Bächle <tshortik at gmx.de <mailto:tshortik at gmx.de>> wrote:
> On 22.09.2016 19 <tel:22.09.2016%2019>:21, William Blevins wrote:
> The current PR is from default; I'm just using bookmarks.
>
>
> Sorry, I hadn't noticed...all the better then. ;)
>
>
> Dirk
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org <mailto:Scons-dev at scons.org>
> https://pairlist2.pair.net/mailman/listinfo/scons-dev <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev at scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist2.pair.net/pipermail/scons-dev/attachments/20160922/df4f694d/attachment-0001.html>
More information about the Scons-dev
mailing list