Sorry to ping the list again, but I've addressed the issues raised in the issue to complete the "os.getlogin" feature. Is there any way I can get another review to get this merged please?<div><br></div><div>Best,</div>
<div>Mitchell</div><div><br><div class="gmail_quote">On Mon, Aug 22, 2011 at 4:53 AM, Mitchell Hashimoto <span dir="ltr"><<a href="mailto:mitchell.hashimoto@gmail.com">mitchell.hashimoto@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Amaury,<div><br></div><div>I've implemented one method (getlogin) and have created an issue + patch for it:</div>
<div>
<br></div><div><a href="https://bugs.pypy.org/issue841" target="_blank">https://bugs.pypy.org/issue841</a></div><div>
<br></div><div>Best,</div><div>Mitchell<font color="#888888"><br><br></font><div class="gmail_quote"><div class="im">On Sun, Aug 21, 2011 at 2:20 PM, Amaury Forgeot d'Arc <span dir="ltr"><<a href="mailto:amauryfa@gmail.com" target="_blank">amauryfa@gmail.com</a>></span> wrote:<br>
</div><div><div></div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br><br><div class="gmail_quote"><div>2011/8/21 Mitchell Hashimoto <span dir="ltr"><<a href="mailto:mitchell.hashimoto@gmail.com" target="_blank">mitchell.hashimoto@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>I noticed the 'os' module is incomplete, and I'd like to help complete this.</div></blockquote><div><br></div></div><div>You are very welcome!</div><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>CPython does this by simply having these methods available on "posix" "nt" "os2" etc. and the "os" module imports those. It appears that PyPy does the same thing. I was able to successfully add 'getlogin' as practice, but I wanted to post here before going further. Some questions below:</div>
<div><br></div><div>1.) Should I mimic CPython and add the functionality to the OS-specific modules?</div></blockquote><div><br></div></div><div>Yes we should mimic CPython: fortunately these modules have different names but share the same source file.</div>
<div>With CPython it's Modules/posixmodule.c, with PyPy it's in pypy/module/posix.</div><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>2.) I don't have a Windows computer on hand. What is the standard practice for implementing some stdlib for one OS but not the other? Would PyPy accept this temporarily?</div>
</blockquote><div><br></div></div><div>Yes, no problem. In this case, I think it's best to let the test fail on Windows so that someone may notice and fix it.</div><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>3.) There are many missing methods, to simplify implementation time and the patch, would it be okay to submit a patch for each stdlib method, so that this was built up over time?</div></blockquote><div><br></div></div>
<div>
Yes, smaller patches are easier to read and merge. I'd be happy to review and commit them. </div></div><div><br></div>-- <br><font color="#888888">Amaury Forgeot d'Arc<br>
</font></blockquote></div></div></div><br></div>
</blockquote></div><br></div>