[Scons-dev] Contribution to SCons development.
Shreedhar Manek
shreedharmanek at gmail.com
Thu Dec 4 16:39:17 EST 2014
I need help with adapting tests.
I change the integer the equivalent string in chmod.py (eg. 0444 to S_IRWXU
and S_IRWXG and S_IRWXO) but there is problem ahead into the program and
the test fails at this point -
s = S_IMODE(os.stat(test.workpath('f1'))[ST_MODE])
test.fail_test(s != 0444)
What do I do about this?
On Tue, Nov 18, 2014 at 1:59 PM, Dirk Bächle <tshortik at gmx.de> wrote:
> Shreedhar,
>
> On 18.11.2014 07:29, Shreedhar Manek wrote:
>
>> Hi,
>> I had emailed on here a few days back asking for some help so that I can
>> contribute to SCons. I'm grateful for the response and I read the guides
>> that I was directed to.
>>
>> I read up to chapter 8 of the documentation beyond which it was getting a
>> tad bit complicated for me. I shall absolutely get back to it later
>> (college end semester exams coming up!). I also looked around on the other
>> links (man page, developer's guide).
>>
>> I checked the bug marked 'easy' on tigris and I found this one which I'd
>> like to fix - http://scons.tigris.org/issues/show_bug.cgi?id=2494.
>> Unfortunately, I have absolutely no idea what I should do next to go about
>> fixing the same. What do I do next?
>>
>> the next steps for you would be:
>
> - ensuring that you can run your forked version of the repo (see
> README.rst in top-level folder)
> - running a full regression test with "python runtest.py -a"
> - finding the "Chmod" method in the sources
> - ensuring that it's the right place you've found, for example by adding
> a simple print statement to it
> - changing the "Chmod" to show the wanted behaviour
> - running the regression test again, to ensure that you didn't break
> anything
> - adapting/adding tests and documentation for the new/changed
> functionality
> - create your pull request
>
> or something along those lines.
>
>
> Best regards,
>
> Dirk
>
>
> _______________________________________________
> 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/20141205/d2834d33/attachment.html>
More information about the Scons-dev
mailing list