Message140956
I confirm fixing 0o755 makes the tests pass for me.
Your code gives one warning:
build_scripts.py:241: BytesWarning: str() on a bytes instance
hdr = "#!%(executable)s%(options)s\n" % locals()
The object with the name executable or options is bytes, which you should explicitly convert to a string with decode. I also don’t like using locals(), but that’s a personal style thing.
Your test should catch stdout (see other packaging tests for how to do that), so that people or buildbots running the tests don’t see “Hello world!”, and so that you can run asserts for the output. |
|
Date |
User |
Action |
Args |
2011-07-23 09:45:31 | eric.araujo | set | recipients:
+ eric.araujo, mhammond, fdrake, vinay.sajip, ceder, tim.golden, tarek, alexis, higery |
2011-07-23 09:45:31 | eric.araujo | set | messageid: <1311414331.45.0.371042586354.issue12394@psf.upfronthosting.co.za> |
2011-07-23 09:45:30 | eric.araujo | link | issue12394 messages |
2011-07-23 09:45:30 | eric.araujo | create | |
|