[issue39861] French doc __futur__: Bad URL
Matheus Vieira Portela added the comment: The PR has already been merged. -- ___ Python tracker <https://bugs.python.org/issue39861> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue39861] French doc __futur__: Bad URL
Matheus Vieira Portela added the comment: I just opened a PR for that: https://github.com/python/python-docs-fr/pull/1176 -- nosy: +matheus.v.portela ___ Python tracker <https://bugs.python.org/issue39861> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue30802] datetime.datetime.strptime('200722', '%Y%U')
Matheus Vieira Portela added the comment: Agreed that we should attain to ISO 8601, as it solves two questions that occurred in this thread: - According to the specification, it is clear that a week must start on Monday and end on Sunday. Hence datetime.strptime('201726', '%Y%U') could be safely assigned to 2017/06/26 without ambiguity - It also makes clear on what to do when a week overlaps two years. IMHO there is no reason to not implement this feature and remove the requirement of using %U with the year and day of the week. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue30802> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1100942] Add datetime.time.strptime and datetime.date.strptime
Matheus Vieira Portela added the comment: Also, may I move this issue to a GitHub PR? -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue1100942> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1100942] Add datetime.time.strptime and datetime.date.strptime
Matheus Vieira Portela added the comment: The patch is broken against Python 3.7. I'll try working on it. -- nosy: +matheus.v.portela Added file: http://bugs.python.org/file47032/issue1100942_20170722.patch ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue1100942> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue30802] datetime.datetime.strptime('200722', '%Y%U')
Matheus Vieira Portela added the comment: Therefore, should we implement the behavior of assuming the first day of the week if none is provided or not? -- nosy: +matheus.v.portela ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue30802> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23406] interning and list comprehension leads to unexpected behavior
Matheus Vieira Portela added the comment: Applying review comments. Now, there is an internal link to the FAQ entry on multidimensional lists. -- Added file: http://bugs.python.org/file40337/issue23406_doc_stdtypes_and_faq.patch ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue23406> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23406] interning and list comprehension leads to unexpected behavior
Matheus Vieira Portela added the comment: I'm attaching a patch to update the stdtypes.rst documentation according to our discussion. I've replaced the table explanation to "equivalent to adding s to itself n times" and added a link to the FAQ entry. I'm not sure, however, where to put the FAQ link. Should it be directly in the table? In this patch, I've put it after the last line of note #2. -- keywords: +patch Added file: http://bugs.python.org/file40321/issue23406_doc_stdtypes.patch ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue23406> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23406] interning and list comprehension leads to unexpected behavior
Matheus Vieira Portela added the comment: Does anyone else think the note should be expanded? For me, it seems to be pretty accurate although it may indeed be confusing to beginners. If anything, I can work on rewriting it to be more explanatory. -- nosy: +matheus.v.portela ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue23406> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue18697] Unify arguments names in Unicode object C API documentation
Matheus Vieira Portela added the comment: Just checking, it would be required to update Objects/unicodeobject.c, Include/unicodeobject.h, and Doc/c-api/unicode.rst, right? As far as I saw, "unicode" means a Python object with unicode string, "u" is a UTF-8 encoded C string, "str" and "s" are encoded C strings (UTF-8, UTF-7, ASCII, Latin-1, among others). Is it alright to rename Python unicode objects to "unicode" and the others to simply "str"? These names are more meaningful than the single character alternatives. The same logic would be applied to the other types, always keeping the longer name rather than the single character ones. -- nosy: +matheus.v.portela ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue18697> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue23406] interning and list comprehension leads to unexpected behavior
Matheus Vieira Portela added the comment: I was referring to the table of operations. So, what if I replace "n shallow copies of s concatenated" by "repeat s n times and concatenate (to create a multidimensional list, refer to [link to FAQ])"? -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue23406> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue3548] subprocess.pipe function
Matheus Vieira Portela added the comment: Oh, I see... The pipes module uses os.system and os.popen to execute commands under a /bin/sh environment. So you are proposing to re-implement it with subprocess in order to execute commands without shell and, also, extend by accepting argv-style parameters. Is this right this time? How would an argv-style pipe look like? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3548 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue3548] subprocess.pipe function
Matheus Vieira Portela added the comment: Sure. I'll leave this issue for a while before others can emit their opinions. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3548 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue24789] ctypes doc string
Matheus Vieira Portela added the comment: Updated create_string_buffer docstring to create_string_buffer(aBytes, anInteger) - character array. -- keywords: +patch nosy: +matheus.v.portela Added file: http://bugs.python.org/file40279/issue24789_ctypes_doc_string.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24789 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue3548] subprocess.pipe function
Matheus Vieira Portela added the comment: Let me check whether I understood you suggestion... What you are saying is that it is already possible to pipeline shell commands through subprocess, such as in the following line? subprocess.call('ls -l | grep Music', shell=True) However, this requires the command to be executed in a shell environment. Hence, it would be a good idea to extend it to non-shell command execution. Is this right? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3548 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue3548] subprocess.pipe function
Matheus Vieira Portela added the comment: I just found this open issue and I can work on it. What is left to do before closing it? -- nosy: +matheus.v.portela ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue3548 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10415] readline.insert_text documentation incomplete
Matheus Vieira Portela added the comment: I could get readline.insert_text() to work, ish, without calling startup_hook. The script is attached to this message. Apparently, there is some need to call readline.redisplay() to update what's on screen. However, I can't really understand what's happening... The script prints Testing text only when the input has exactly one character. Also, by calling redisplay() after insert_text(), the string is not on the command line buffer, but already printed on screen. I should take a look on the GNU Readline Library documentation to understand better how it works. Anyway, I still think the documentation is unclear on these behaviours and should be updated. -- Added file: http://bugs.python.org/file34366/test_readline.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10415 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue809163] Can't add files with spaces
Matheus Vieira Portela added the comment: As far as I noticed, in bdist_rpm.py, the _make_spec_file() method generates an .spec file for RPM build and attaches the install script to setup.py install -O1 --root=$RPM_BUILD_ROOT --record=INSTALLED_FILES. Later, the .spec refers to the record file as the one which contains all necessary files to build the RPM package with the directive %files -f INSTALLED_FILES. If any path contains an space character, it will triggers an error. I could fix this problem by attaching to the installation a small sed script that appends double quotes to the beginning and the end of each line of INSTALLED_FILES. I'm not sure whether this is allowed though. Any ideas how to make this more Pythonic without using sed? Apparently, it works since the test passed. However, there is an warning which I have no idea what it means: unknown, 0: Warning: using regular magic file `/etc/magic'. I attached the diff file with the changes so far. -- Added file: http://bugs.python.org/file34308/quoted_files.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue809163 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue809163] Can't add files with spaces
Matheus Vieira Portela added the comment: I tried to apply the last patch but it returned me and error of failing hunk. I think it was based on an old version of the test_bdist_rpm.py file. Hence, I made this updated version of the patch and could get the expected failure during the tests. I should try and solve this bug soon. -- Added file: http://bugs.python.org/file34303/test_bdist_rpm_filename_with_whitespaces.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue809163 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue809163] Can't add files with spaces
Matheus Vieira Portela added the comment: Is this issue still going on? -- nosy: +matheus.v.portela ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue809163 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20799] Turn on all tests in test.test_importlib.source.test_file_loader
Matheus Vieira Portela added the comment: Apparently, the methods with leading underscore are being tested by two classes (SourceLoaderBadBytecodeTest and SourcelessLoaderBadBytecodeTest) that implement methods with similar names (but without the underscore). Simply removing the underscore breaks the tests since they cannot differ between the methods with and without underscore. -- nosy: +matheus.v.portela ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20799 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com