[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