[Scons-users] False line length calculation

Dirk Bächle tshortik at gmx.de
Sat Aug 16 06:07:33 EDT 2014


Hi Roland,

and thanks a lot for your analysis, which looks correct to me. Nice catch!

On 15.08.2014 10:02, Roland.Stark-EXT at continental-corporation.com wrote:
> There is an issue with line length calculation in the following file: 
> SCons/Platform/__init__.py
> (my OS is Windows7)
>
> [...]
>
> Fix B (alternative):
>   length = 0
>   for c in cmd:
>       length += len(c)
>   length += len(cmd) - 1
>   print ">>> Len = %i" % length
>   if length <= maxline:
>       return self.cmd
>
I'd prefer something along option "B" as fix. Can you open a ticket in 
the Tigris bug tracker and attach this alternative as patch? Even better 
would be a pull request...check the instructions in 
http://scons.org/wiki/SconsMercurialWorkflows , or ask if you have 
further questions.

Best regards,

Dirk


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://four.pairlist.net/pipermail/scons-users/attachments/20140816/b5a704e6/attachment.html>


More information about the Scons-users mailing list