Yeah that's what I was thinking. Not foolproof but better. Will that work
on Windows?

On Thu, Sep 22, 2016 at 7:04 PM, Tim Jenness <tjenn...@lsst.org> wrote:

> Maybe add a slash after “build”? So “build/"
>
> On Sep 22, 2016, at 16:03, Bill Deegan <b...@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 <wblevins...@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 <b...@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 <wblevins...@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 <wblevins...@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 <
>>>>> wblevins...@gmail.com> wrote:
>>>>>
>>>>>> Yes.
>>>>>>
>>>>>> On Thu, Sep 22, 2016 at 4:47 PM, Bill Deegan <
>>>>>> b...@baddogconsulting.com> wrote:
>>>>>>
>>>>>>> William,
>>>>>>>
>>>>>>> Looks like a couple new failures.
>>>>>>> http://buildbot.scons.org/builders/bd-ubuntu-python-2.7/buil
>>>>>>> ds/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 <
>>>>>>> wblevins...@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 <tshor...@gmx.de>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> On 22.09.2016 19: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@scons.org
>>>>>>>>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Scons-dev mailing list
>>>>>>>> Scons-dev@scons.org
>>>>>>>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Scons-dev mailing list
>>>>>>> Scons-dev@scons.org
>>>>>>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> Scons-dev mailing list
>>>> Scons-dev@scons.org
>>>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>>>
>>>>
>>>
>>> _______________________________________________
>>> Scons-dev mailing list
>>> Scons-dev@scons.org
>>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>>
>>>
>>
>> _______________________________________________
>> Scons-dev mailing list
>> Scons-dev@scons.org
>> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>>
>>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>
>
>
> _______________________________________________
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>
>
_______________________________________________
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Reply via email to