Re: Closing files (patch)

On 07/04/2017 10:41 PM, Bill Deegan wrote:
> Any chance you can create a pull request on bitbucket for this?
> It's a lot simpler to merge that way.
Yeah, I'll submit one soon.


 >> P.S. On scons.org <http://scons.org>, Development -> Development, 
the "Developer's Guidelines" link is broken.

 > The link is working for me.
 > What's the URL it's pointing you to?
 > For me it's pointing to:
 > http://scons.org/dev.html <http://scons.org/dev.html>

"Development -> Development" works, and points me at that page. But *on* 
that page there's a link to "Developer's Guidelines", " The complete 
statement of guiding principles and procedures for SCons development."

That link goes to http://scons.org/%7Bfilenames%7D/development/guidelines.md

(The first path component is /{filenames}/ if you don't know hex 
encoding of symbols off the top of your head...)

Evan


>
> Thanks,
> Bill
>
> On Tue, Jul 4, 2017 at 10:47 PM, Evan Driscoll <[email protected] 
> <mailto:[email protected]>> wrote:
>
>     I recently gave running SCons under PyPy a shot. It actually
>     worked, except for running out of file descriptors due to the GC
>     not closing things immediately.
>
>     I went through and added some explicit close calls (actually,
>     mostly 'with's) and then it worked fine. It looks like you
>     actually picked up most of them (probably all the important ones)
>     in 8716da8, but in case you wanted, I thought I'd send a patch
>     with a few more. (We're using 2.5.0, and I'm not sure if the
>     commit cited above landed in time for .1 or after that.)
>
>     I don't know if the remainder matter though, or in the case of the
>     MD5 change, if it's in code that's even called. (Though if it is
>     called, it may well matter.) I just did a search for 'open' and
>     made replacements for ones where it looked easy to do and I didn't
>     have to think too hard. :-)
>
>     Evan
>
>     P.S. On scons.org <http://scons.org>, Development -> Development,
>     the "Developer's Guidelines" link is broken.
>
>     P.P.S. The FAQ still has the question "why is SCons written for
>     Python 2.4"
>
>
>     _______________________________________________
>     Scons-dev mailing list
>     [email protected] <mailto:[email protected]>
>     https://pairlist2.pair.net/mailman/listinfo/scons-dev
>     <https://pairlist2.pair.net/mailman/listinfo/scons-dev>
>
>
>
>
> _______________________________________________
> Scons-dev mailing list
> [email protected]
> https://pairlist2.pair.net/mailman/listinfo/scons-dev

_______________________________________________
Scons-dev mailing list
[email protected]
https://pairlist2.pair.net/mailman/listinfo/scons-dev