[issue4899] doctest should support fixtures

New submission from Giovanni : The doctest module should have support for fixtures (e.g. setUp and tearDown methods). It is very silly to be forced to always re-import all the modules needed in every function tested with doctest. For example, when you have to document functions that open files

[issue4899] doctest should support fixtures

Giovanni added the comment: I was proposing to adopt doctest in the biopython project (modules for bioinformatics in python, http://biopython.org/). Doctest is very useful to document modules that will be used by many other people: for example, there are many different file formats in

[issue6721] Locks in python standard library should be sanitized on fork

Giovanni Bajo added the comment: If there's agreement that the general problem is unsolvable (so fork and threads just don't get along with each other), what we could attempt is trying to limit the side effects in the standard library, so that fewest users as possible are affect

[issue7213] Popen.subprocess change close_fds default to True

Giovanni Bajo added the comment: Hi Gregory, I saw your commit here: http://code.activestate.com/lists/python-checkins/91914/ This basically means that in 3.2 it is mandatory to specify close_fds to avoid a DeprecationWarning. *BUT* there is no good value that works both on Windows and Linux

[issue7213] Popen.subprocess change close_fds default to True

Giovanni Bajo added the comment: Setting CLOEXEC on the pipes seems like a very good fix for this bug. I'm +1 on it, but I think it should be the default; instead, your proposed patch adds a new argument to the public API. Why do you think it's necessary to do so? At the same time,

[issue7213] Popen.subprocess change close_fds default to True

Giovanni Bajo added the comment: Would you mind elaborating on where is the race condition? -- ___ Python tracker <http://bugs.python.org/issue7213> ___ ___ Pytho

[issue7213] subprocess leaks open file descriptors between Popen instances causing hangs

Giovanni Bajo added the comment: Hi Gregory, will you backport Mirko's patches to subprocess32? The last thing left in this bug is my proposal to change the default of close_fds to True to Windows too, but at the same time detect whether this is possible or not (depending on the

[issue1342] Crash on Windows if Python runs from a directory with umlauts

Changes by Giovanni Bajo: -- nosy: +rasky __ Tracker <[EMAIL PROTECTED]> <http://bugs.python.org/issue1342> __ ___ Python-bugs-list mailing list Unsubs

[issue2128] sys.argv is wrong for unicode strings

New submission from Giovanni Bajo: Under Windows, sys.argv is created through the Windows ANSI API. When you have a file/directory which can't be represented in the system encoding (eg: a japanese-named file or directory on a Western Windows), Windows will encode the filename to the s

[issue2066] Adding new CNS11643, a *huge* charset, support in cjkcodecs

Giovanni Bajo added the comment: Making the standard Windows Python DLL larger is not only a problem of disk size: it will make all packages produced by PyInstaller or py2exe larger, and that means lots of wasted bandwidth. I see that MvL is still -1 on simply splitting CJK codecs out, and

[issue2128] sys.argv is wrong for unicode strings

Giovanni Bajo added the comment: I'm attaching a simple patch that seems to work under Py3k. The trick is that Py3k already attempts (not sure how or why) to decode argv using utf-8. So it's sufficient to setup argv as UTF8-encoded strings. Notice that brings the output of "pyt

[issue2128] sys.argv is wrong for unicode strings

Giovanni Bajo added the comment: mbstowcs uses LC_CTYPE. Is that correct and consistent with the way default encoding under UNIX is handled by Py3k? Would a Py_MainW or similar wrapper be easier on the UNIX guys? I'm just asking, I don't have a def

[issue44359] test_ftplib fails as "env changes" if a socket operation times out in a thread: TimeoutError is not catched

Change by Giovanni Wijaya : -- nosy: +giovanniwijaya nosy_count: 7.0 -> 8.0 pull_requests: +26699 stage: -> patch review pull_request: https://github.com/python/cpython/pull/28228 ___ Python tracker <https://bugs.python.org/i

[issue37496] Support annotations in signature strings.

Giovanni Cappellotto added the comment: Thanks for the info. Yes, I’m still interested in working on the issue, but lately I didn’t have much time to do it. If this is a high priority feel free to re-assign to someone else, otherwise I’ll start working on it next week. On Sun, Dec 1, 2019 at

[issue31485] Tkinter widget.unbind(sequence, funcid) unbind all bindings

Giovanni Lombardo added the comment: http://docs.python.org/devguide/triaging.html#assigned-to -- nosy: +glombardo ___ Python tracker <https://bugs.python.org/issue31

[issue39152] Faulty override of tkinter.Misc.configure in tkinter.ttk.Scale.configure

New submission from Giovanni Lombardo : The issue arises by simply calling configure on the Scale widget of the themed tk (ttk) widgets set: ``` cursor = scale.configure('cursor')[-1] ``` The above casues the following error: ``` File "C:\Users\Giovanni\Tests\test_scale.py

[issue39152] Faulty override of tkinter.Misc.configure in tkinter.ttk.Scale.configure

Giovanni Lombardo added the comment: I've modified the `Scale.configure` implementation as follows: ``` def configure(self, cnf=None, **kw): """Modify or query scale options. Setting a value for any of the "from", "from_" or &

[issue39152] Faulty override of tkinter.Misc.configure in tkinter.ttk.Scale.configure

Giovanni Lombardo added the comment: Hello Terry J. Reedy, first and foremost thanks for the time you've dedicated to this issue! I really appreciate you work!! Even if I work with Python since 2007, I'm new here, and not very well versed with this issue management system. This h

[issue39152] Faulty override of tkinter.Misc.configure in tkinter.ttk.Scale.configure

Giovanni Lombardo added the comment: @serhiy.storchaka I didn't found any old issue not closed aboud tkinter Scale widget. -- ___ Python tracker <https://bugs.python.org/is

[issue31485] Tkinter widget.unbind(sequence, funcid) unbind all bindings

Giovanni Lombardo added the comment: I propose the below fix: ``` def unbind(self, sequence, funcid=None): """Unbind for this widget for event SEQUENCE the function identified with FUNCID.""" bound = '' if funci

[issue43110] import aiohttp crash closes Python from Windows Installer

New submission from Giovanni Wijaya : Since Python 3.10.0a4, importing aiohttp crash closes Python, only on Python installed from Windows Installer x64. Python installed in Debian (both from source and otherwise) does not produce this issue. -- components: Windows messages: 386180

[issue43110] import aiohttp crash closes Python from Windows Installer

Giovanni Wijaya added the comment: Update: I tried installing Python from source in x64, and same issue persists. -- ___ Python tracker <https://bugs.python.org/issue43

[issue43110] import aiohttp crash closes Python from Windows Installer

Giovanni Wijaya added the comment: I'm not sure whether aiohttp has released packages specifically for the alphas, but I tried rebuilding aiohttp and its extension modules and it currently works. Thank you for the solution. -- resolution: -> not a bug stage: -> reso

[issue41291] Race conditions when opening and deleting a file on Mac OS X

New submission from Giovanni Pizzi : Hello, when creating deleting (with `os.remove`/`os.unlink`) a file and opening it in a different process at (almost) the same time, on Mac OS X relatively often I get a file that is empty instead of either a `FileNotFoundError` exception, or an open

[issue41291] Race conditions when opening and deleting a file on Mac OS X

Giovanni Pizzi added the comment: Thanks for your feedback! I also just reported the issue to Apple. For reference I got the Feedback ID FB8009608. I will let you know if they answer. In any case I agree that also your other error is unexpected. I am writing a library that should provide

[issue13893] Make CGIHTTPServer capable of redirects (and status other than 200)

New submission from Giovanni Funchal : GIHTTPServer.py is a very handy module for quickly setting up a full HTTP server with CGI support. However, I noticed that it doesn't support responses other than "200 OK". So, for instance if the page wants to do a redirect (response 303)

[issue13893] Make CGIHTTPServer capable of redirects (and status other than 200)

Giovanni Funchal added the comment: @Glenn Linderman, can you please share your changes? You can upload a patch. -- versions: +Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.4 ___ Python tracker <http://bugs.python.org/issue13

[issue13893] Make CGIHTTPServer capable of redirects (and status other than 200)

Giovanni Funchal added the comment: @Éric Araujo, yes this is a duplicate of #10487. I now think this should be marked as bug instead of enhancement. The documentation warns about this behavior but it is documenting a bug. -- resolution: -> duplic

[issue10805] traceback.print_exception throws AttributeError when exception is None

Giovanni Funchal added the comment: This bug affects me, found it when migrating from 2.7 to 3.2, in a function calling traceback.print_exc() called while there were no "active" exception being handled. Previous behavior was to print "None". -- nos

[issue10484] http.server.is_cgi fails to handle CGI URLs containing PATH_INFO

Giovanni Funchal added the comment: This is still an issue as of python 3.2.2 and is affecting me. -- nosy: +Giovanni.Funchal ___ Python tracker <http://bugs.python.org/issue10

[issue37284] Not obvious that new required attrs of sys.implementation must have a PEP.

Giovanni Cappellotto added the comment: Hi, I'd like to claim this task if nobody else already started working on it. -- nosy: +potomak ___ Python tracker <https://bugs.python.org/is

[issue37284] Not obvious that new required attrs of sys.implementation must have a PEP.

Change by Giovanni Cappellotto : -- keywords: +patch pull_requests: +14148 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/14328 ___ Python tracker <https://bugs.python.org/issu

[issue37386] [EASY] test_io: test_large_file_ops() failed on AMD64 Windows7 SP1 3.x with: [Errno 28] No space left on device

Giovanni Cappellotto added the comment: Hi, I'd like to claim this task if nobody else already started working on it. -- nosy: +potomak ___ Python tracker <https://bugs.python.org/is

[issue37386] [EASY] test_io: test_large_file_ops() failed on AMD64 Windows7 SP1 3.x with: [Errno 28] No space left on device

Change by Giovanni Cappellotto : -- keywords: +patch pull_requests: +14173 stage: -> patch review pull_request: https://github.com/python/cpython/pull/14356 ___ Python tracker <https://bugs.python.org/issu

[issue13127] xml.dom.Attr.name is not labeled as read-only

Giovanni Cappellotto added the comment: I took a quick look at `minidom.py` and `test_minidom.py`. It seems that you should always use `doc.renameNode(attr, namespace, new_name)` for renaming an attribute (or an element). When `doc.renameNode` is applied on an attribute node, it removes the

[issue37386] [EASY] test_io: test_large_file_ops() failed on AMD64 Windows7 SP1 3.x with: [Errno 28] No space left on device

Giovanni Cappellotto added the comment: Serhiy, Victor: I didn't get what's your suggested solution for this task. Should we just close it as "wontfix", can I go ahead and merge the existing PR, or do you have a better suggestion on how

[issue37548] Document range of atan, acos and asin

Change by Giovanni Cappellotto : -- keywords: +patch pull_requests: +14517 stage: -> patch review pull_request: https://github.com/python/cpython/pull/14717 ___ Python tracker <https://bugs.python.org/issu

[issue37548] Document range of atan, acos and asin

Giovanni Cappellotto added the comment: I created a small diff to update the documentation of `math.atan`, `math.asin` and `math.acos`. -- nosy: +potomak ___ Python tracker <https://bugs.python.org/issue37

[issue13127] xml.dom.Attr.name is not labeled as read-only

Change by Giovanni Cappellotto : -- keywords: +patch pull_requests: +14552 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/14757 ___ Python tracker <https://bugs.python.org/issu

[issue13127] xml.dom.Attr.name is not labeled as read-only

Giovanni Cappellotto added the comment: In https://github.com/python/cpython/pull/14757 I tried updating the implementation of `Attr._set_name` to remove and reset the attr node in the owner element. So now `Attr._set_name` behaves similarly to `Document.renameNode`. All existing tests are

[issue37496] Support annotations in signature strings.

Change by Giovanni Cappellotto : -- nosy: +potomak ___ Python tracker <https://bugs.python.org/issue37496> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue7202] "python setup.py cmd --verbose" does not set verbosity

Change by Giovanni Cappellotto : -- keywords: +patch pull_requests: +14585 stage: -> patch review pull_request: https://github.com/python/cpython/pull/14787 ___ Python tracker <https://bugs.python.org/iss

[issue7202] "python setup.py cmd --verbose" does not set verbosity

Giovanni Cappellotto added the comment: I took a stab at this, see attached PR. I was able encode the first two cases described by @tarek, but not the first one because `_parse_command_opts` doesn't have visibility to the list of global options that have already been parsed. Note:

[issue37613] Broken "Back to top" link in www.python.org (mobile)

Change by Giovanni Cappellotto : -- title: Fix broken "Back to top" link in www.python.org (mobile) -> Broken "Back to top" link in www.python.org (mobile) ___ Python tracker <https://

[issue37613] Fix broken "Back to top" link in www.python.org (mobile)

New submission from Giovanni Cappellotto : The mobile version of https://www.python.org/ has a "Back to top" link at the bottom of every page that should scroll the page back to the top, but it doesn't work in Chrome for Android. Note: it works with Safari on iOS. --

[issue37613] Broken "Back to top" link in www.python.org (mobile)

Giovanni Cappellotto added the comment: I can reproduce the error in Chrome for MacOS when I emulate the resolution of a generic mobile device. I think I have a quick fix for the issue, I will publish a PR soon. -- ___ Python tracker <ht

[issue37496] Support annotations in signature strings.

Giovanni Cappellotto added the comment: I'd like to work on this, but I'm kind of new to the codebase. Do you think I should leave this task to someone more expert on the matter? I took a look at the function you mentioned and I was able to support simple annotations, for instan

[issue37496] Support annotations in signature strings.

Giovanni Cappellotto added the comment: > You might want to look into how PEP 563 is implemented, it has a utility to > turn an AST back into a string (I assume this is what you want). Thanks for your suggestion @levkivskyi. I took a look at https://github.com/python/cpython/pull/4390

[issue37666] urllib.requests.urlopen doesn't support cadata=

Change by Giovanni Cappellotto : -- nosy: +potomak ___ Python tracker <https://bugs.python.org/issue37666> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue21992] New AST node Else() should be introduced

Change by Giovanni Cappellotto : -- nosy: +potomak ___ Python tracker <https://bugs.python.org/issue21992> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37678] Incorrect behaviour for user@password URI pattern in urlparse

Giovanni Cappellotto added the comment: What do you mean that urlparse act as unexpected? I tried your example and I think urlparse's behavior is correct. >From the RFC 1738: > Octets must be encoded if they have no corresponding graphic > character within the US-ASCII coded

[issue37724] [[Errno 17] File exists: ] # Try create directories that are not part of the archive with

Giovanni Cappellotto added the comment: There's also the `makedirs`'s `exist_ok` optional parameter, `False` by default. You could use it to avoid this error, but I'm not an expert on this particular library, so I'm not sure if it would make sense to make this cha

[issue15266] Perform the same checks as PyPI for Description field

Giovanni Cappellotto added the comment: Hi Victor, This is the 3rd issue in a row I found in the "Easy issues" list where you added a comment saying > I remove the "Easy" label Why are can I still find these issues in the "Easy issues" list? Do you mean t

[issue37700] shutil.copyfile does not raise SpecialFileError for socket files

Change by Giovanni Cappellotto : -- nosy: +potomak ___ Python tracker <https://bugs.python.org/issue37700> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue15266] Perform the same checks as PyPI for Description field

Change by Giovanni Cappellotto : -- nosy: -potomak ___ Python tracker <https://bugs.python.org/issue15266> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Christian, there are good semi-crypto hash functions that don't leak as bad as Python's own modified FNV hash, without going all the way to HMAC. SipHash has very good collision resistance and doesn't leak anything: https://www.131002.net/siphash

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: For short strings, you might want to have a look at the way you fetch the final partial word from memory. If the string is >= 8 bytes, you can fetch the last partial word as an unaligned memory fetch followed by a shift, instead of using a switch like in

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Until it's broken with a yet-unknown attack, SipHash is a pseudo-random function and as such it does uniformly distribute values across the output space, and never leak any information on the key (the randomized seed). Being designed by cryptographers,

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 07/nov/2012, alle ore 08:40, Serhiy Storchaka ha scritto: > Serhiy Storchaka added the comment: > > I tested different kind of strings. > > $ ./python -m timeit -n 1 -s "t = b'a' * 10**8" "hash(t)" &g

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 07/nov/2012, alle ore 12:59, Marc-Andre Lemburg ha scritto: > > Marc-Andre Lemburg added the comment: > > On 07.11.2012 12:55, Mark Dickinson wrote: >> >> Mark Dickinson added the comment: >> >> [MAL] >>&

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 11/nov/2012, alle ore 05:56, Chris Rebert ha scritto: > > Chris Rebert added the comment: > > What about CityHash? (http://code.google.com/p/cityhash/ ; unofficial C port: > http://code.google.com/p/cityhash-c/ ) > It's

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 02/gen/2013, alle ore 00:20, Domen Kožar ha scritto: > > Domen Kožar added the comment: > > According to talk at 29c3: > http://events.ccc.de/congress/2012/Fahrplan/events/5152.en.html > > Quote: We also describe a vulnerab

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 02/gen/2013, alle ore 06:52, Christian Heimes ha scritto: > > Christian Heimes added the comment: > > Thanks for the information! I'm working on a PEP for the issue at hand. Since you're collecting ideas on this, I would l

[issue14621] Hash function is not randomized properly

Giovanni Bajo added the comment: Il giorno 02/gen/2013, alle ore 19:51, Christian Heimes ha scritto: > > Christian Heimes added the comment: > > Giovanni, why do you think that hashing of unicode strings is slower than > byte strings? > > First of all ASCII onl

[issue12226] use HTTPS by default for uploading packages to pypi

Giovanni Bajo added the comment: Please notice that a redesign of PyPI and package security is ongoing in catalog-sig. -- nosy: +Giovanni.Bajo ___ Python tracker <http://bugs.python.org/issue12

[issue23554] EchoServerClientProtocol class should be called EchoServerProtocol

New submission from Giovanni Cannata: In 18.5.4.3.2. TCP echo server protocol the class in the example code is called EchoServerClientProtocol. It should be EchoServerProtocol. (The client protocol example is correctly called EchoClientProtocol). -- assignee: docs@python components

[issue23749] asyncio missing wrap_socket

New submission from Giovanni Cannata: It's not possible to wrap a socket in tls. The StreamWriter object should have an option to start a tls negotiation using the SSLContext of the server. This is needed for protocols the have a "start_tls" feature, for example the ldap prot

[issue23749] asyncio missing wrap_socket

Giovanni Cannata added the comment: Thanks, I will look to the new implementation of ssl in 3.5, and try to adapt it for my project (sldap3). I'd like to help, but I'm not skilled in asynchronous programming so I'm not sure if I succeed.