[issue17845] Clarify successful build message

2013-07-03 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Patch to modify setup.py comments on successful build -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30758/issue17845.patch ___ Python tracker rep...@bugs.python.org http

[issue17953] sys.modules cannot be reassigned

2013-05-24 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: You are most welcome Brett. I am just starting to contribute with this doc fix. Didn't think a small detail in it will generate such a long debate in the python-committers mailing list. Is there a plan in action for 'fixing' Misc/NEWS. I now realize

[issue17953] sys.modules cannot be reassigned

2013-05-24 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: That sounds good. I will open up an issue (if not for anything else other than to get more eyes on this issue) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17953

[issue18053] Add checks for Misc/NEWS in make patchcheck

2013-05-24 Thread Yogesh Chaudhari
New submission from Yogesh Chaudhari: Based on the threads about http://mail.python.org/pipermail/python-committers/2013-May/002529.html and discussion on http://bugs.python.org/issue17953; It would be good to add some (very high level)checks inside patchcheck that will tell the user if Misc

[issue18053] Add checks for Misc/NEWS in make patchcheck

2013-05-24 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I am not sure which component patchcheck corresponds to, so I have left that out and added people on nosy list in http://bugs.python.org/issue17953. If I have missed the concerned maintainer, kindly let me know. -- nosy: +brett.cannon, pitrou

[issue17953] sys.modules cannot be reassigned

2013-05-18 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Documentation added for sys.modules -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30305/issue17953.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org

[issue17914] add os.cpu_count()

2013-05-16 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Minor modifications based on review comments. 1. Change mib array size to 2, 2. return value set to 0 consistently (in C code), and 3. removed IRIX #defines -- Added file: http://bugs.python.org/file30282/issue17914-6.patch

[issue17914] add os.cpu_count()

2013-05-16 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Typo fix -- Added file: http://bugs.python.org/file30284/issue17914-7.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17914

[issue17914] add os.cpu_count()

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Stinner: 1. While I agree with your idea of what you have done in test_os, (particularly, for determining if platform is supported or not) there seems to be no reason(AFAIK) to have a shutil for cpu_count. I agree with neologox there. 2. Also I am

[issue17962] Broken OpenSSL version in Windows builds

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I would like to take a crack at it. Would it be possible(I mean permission wise)? How can I proceed? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17962

[issue9856] Change object.__format__(s) where s is non-empty to a TypeError

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: +1 to Terry for If its class does not override .__format__, then it seems that it should act the same as a direct object instance -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9856

[issue17914] add os.cpu_count()

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Modified patch based on further comments and review. 1. Removed *everything* from os.py 2. removed typecasting for ncpu where not required. 3. removed redundant comments -- Added file: http://bugs.python.org/file30246/issue17914-5.patch

[issue17962] Broken OpenSSL version in Windows builds

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Antoine and @Georg: You are right. There is nothing much to be done. The only difference is those of auto-generated files like Makefile and other config files created after running ./config script. This just needs a simple add/commit from the openssl

[issue17914] add os.cpu_count()

2013-05-13 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Based on the last 3 messages by Ned, Charles and Antoine, I keep thinking that arguments made by Charles are very valid ones and that it would be better to return 1. I say this (partly from the 'type' argument, but), mainly, *if* its known

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Terry: Thanks for the info. I seem to have the elusive * after my username now. I am not sure how this works, but can you review/test/apply the patch now? -- ___ Python tracker rep...@bugs.python.org http

[issue17940] extra code in argparse.py

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I guess that was simple enough. I am new to this issue tracker for Cpython. How will the patch review/commit proceed? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17940

[issue17962] Broken OpenSSL version in Windows builds

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: The following patch will make use of 1.0.1e version of OpenSSL -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30232/issue17962.patch ___ Python tracker rep...@bugs.python.org

[issue9856] Change object.__format__(s) where s is non-empty to a TypeError

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Eric: when you say: If the type of the object really is object, then it can use string formatting. It's only for non-objects that I want to add the error.. I am confused. Let me demonstrate what I'm thinking according to the statement above. First let

[issue9856] Change object.__format__(s) where s is non-empty to a TypeError

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Please replace integer.__format__(s) with integer.__format__('') -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9856

[issue17914] add os.cpu_count()

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @STINNER: I don't understand. Where exactly should the patch handle this? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17914

[issue17914] add os.cpu_count()

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Modified patch based on review by neologix -- Added file: http://bugs.python.org/file30236/issue17914-4.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17914

[issue13461] Error on test_issue_1395_5 with Python 2.7 and VS2010

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: This patch should work for 2.7 branch -- hgrepos: +191 keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30238/issue13461-27.patch ___ Python tracker rep...@bugs.python.org http

[issue13461] Error on test_issue_1395_5 with Python 2.7 and VS2010

2013-05-12 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: -- hgrepos: -191 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13461 ___ ___ Python-bugs-list

[issue13461] Error on test_issue_1395_5 with Python 2.7 and VS2010

2013-05-12 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: Added file: http://bugs.python.org/file30239/issue13461-3x.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13461

[issue17940] extra code in argparse.py

2013-05-12 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: -- hgrepos: -190 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17940 ___ ___ Python-bugs-list

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @terry and @paul: I have 2 argparse related patches, sitting idle for quiet sometime now. Can you suggest any more names who can take this forward? (I could not find anyone else related to argparse in http://docs.python.org/devguide/experts.html

[issue9856] Change object.__format__(s) where s is non-empty to a TypeError

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: It's this case that is currently an error, but it need not be: format(object(), '1') Traceback (most recent call last): File stdin, line 1, in module TypeError: non-empty format string passed to object.__format__ I believe that should continue to remain

[issue12634] Random Remarks in class documentation

2013-05-12 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: -- hgrepos: -188 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12634 ___ ___ Python-bugs-list

[issue17962] Broken OpenSSL version in Windows builds

2013-05-12 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I am not sure how http://www.openssl.org/source/openssl-1.0.1e.tar.gz can be provided to python svn. Doesn't that require svn credentials to check-out/add/check-in? -- ___ Python tracker rep...@bugs.python.org

[issue17940] extra code in argparse.py

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Removed the duplicated code from argparse.py -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30214/issue17940.patch ___ Python tracker rep...@bugs.python.org http

[issue17940] extra code in argparse.py

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Remove extra code in argparse.py for 2.7 branch -- hgrepos: +190 Added file: http://bugs.python.org/file30215/issue17940-27.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17940

[issue17940] extra code in argparse.py

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I have added a patch for default branch as well, because IMO the same issue exists in all branches -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17940

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Based on the conversation and the particular inputs to the thread form neologix and ezio, I would like to submit this patch. It probably needs modification(s) as I am not sure what to do with the implementation that is already present in multiprocessing

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Appreciate everyone's feedback. I have modified the patch based on further messages in the thread. @Neologix modified posixmodule according to one in Trent's branch and used that for cpu_count(). Kindly suggest improvements/changes if any. @Ned: Thanks

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Serhiy Sorry, I missed your comments in the thread. I have made 2 changes and ignored the cpu_count() returning 0, because it returns -1 on failure, else give the number of CPUs. Also the test_os, checks for 0 return if that was to e the case

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Returning None from C code sounds reasonable to me. Anyone else wants to pitch in with suggestions for/against this? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17914

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Modified patch to return None from C code -- Added file: http://bugs.python.org/file30226/issue17914-3.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17914

[issue17914] add os.cpu_count()

2013-05-11 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: @Ned: if cpus is None, then this will raise an exception in Python 3: `cpus = 1 or cpus == None` I understand that cpus = INTEGER will raise an exception and have already modified the condition to remove that kind of check. I was merely stating

[issue12634] Random Remarks in class documentation

2013-05-05 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Similar changes for 2.7 branch -- hgrepos: +188 Added file: http://bugs.python.org/file30135/issue12634-27.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12634

[issue12634] Random Remarks in class documentation

2013-05-05 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Based on Teery's comments, this patch makes the changes to the random remarks section of the class documentation -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30134/issue12634.patch

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-04 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: I am not sure if I am missing something. I had filled out the form at http://www.python.org/psf/contrib/contrib-form/ on the day I submitted the patch and even got back an email from Ewa Jodlowska. However, I don't see any * after my name. I submitted

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-03 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: A user generated line with multiple spaces may be essential if we are getting the usage data (particularly from) a csv file (that may have random spaces in certain fields) or some other form of stored or auto-generated data

[issue17871] Wrong signature of TextTestRunner's init function

2013-05-02 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: The changes in Docs/library/unittest.rst have been made to reflect proper arguments from Lib/unittest/runner.py -- keywords: +patch nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30106/mywork.patch

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-02 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: -- keywords: +patch Added file: http://bugs.python.org/file30108/7691d1d4b955.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17890

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-02 Thread Yogesh Chaudhari
Yogesh Chaudhari added the comment: Made similar required changes for version 2.7 -- nosy: +Yogesh.Chaudhari Added file: http://bugs.python.org/file30110/Issue17890-27.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17890

[issue17890] argparse: mutually exclusive groups full of suppressed args can cause AssertionErrors

2013-05-02 Thread Yogesh Chaudhari
Changes by Yogesh Chaudhari mr.yog...@gmail.com: Removed file: http://bugs.python.org/file30108/7691d1d4b955.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue17890