Claudiu.Popa added the comment:
Hello, I signed the electronic contributor agreement (I received a mail with
the final copy afterwards). Is there something else that I should do or that is
the entire process?
--
___
Python tracker rep
Claudiu.Popa added the comment:
Yes, it iterates over every member of the namespace path. The new attached
patch fixes this behaviour, by checking that each loader path starts with
top_level_dir when set_implicit_top is False (if set_implicit_top is True,
top_level_dir is irrelevant
Claudiu.Popa added the comment:
Here's the new patch. Also, I noticed a test failing when running ./python -m
test, pyclbr, complaining about _Aifc_params not present in some dict, but I
don't really know how to fix it..
--
Added file: http://bugs.python.org/file31034/aifc_4.patch
Claudiu.Popa added the comment:
Here's the new modifications.
--
Added file: http://bugs.python.org/file31036/aifc_5.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
New submission from Claudiu.Popa:
Both sndhdr.whathdr an sndhdr.what returns a tuple with various information,
while it could return a namedtuple. I attached a patched for this, with tests
as well.
--
components: Library (Lib)
files: sndhdr.patch
keywords: patch
messages: 194080
nosy
Claudiu.Popa added the comment:
Here's a patch with tests attached.
--
keywords: +patch
nosy: +Claudiu.Popa
versions: +Python 3.4 -Python 3.1
Added file: http://bugs.python.org/file31210/xmlrpc.patch
___
Python tracker rep...@bugs.python.org
http
Claudiu.Popa added the comment:
Here's my patch attempt. The repr of a match object has the following format:
(groups=\d+, span=(start, end), group0=the entire group or the first X
characters, where X is represented by a new constant in sre_constants.h,
SRE_MATCH_REPR_SIZE
Claudiu.Popa added the comment:
The patch for Python 2.7 is pretty similar with the one provided for Python 3.
--
nosy: +Claudiu.Popa
Added file:
http://bugs.python.org/file31386/unittest_method_name_difference_27.patch
___
Python tracker rep
Claudiu.Popa added the comment:
Here's an updated patch for the stdlib, with adapted tests.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file31388/code.patch
___
Python tracker rep...@bugs.python.org
http
Claudiu.Popa added the comment:
Here's a basic patch, using a classmethod. It doesn't support the full distb
API (if the traceback is not given, try to retrieve the last one), because
Bytecode.from_tb() looks pretty weird.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http
Claudiu.Popa added the comment:
Here's a basic patch. I have two issues with it: the flag name used for
enabling the exceptions seems too verbose (use_python_exceptions, I'm opened to
new suggestions) and I don't know if the fallback to Fault exception if the
remote exception is not known
New submission from Claudiu.Popa:
This is similar to what was done for issue17818 and issue17487 and will gain
for all the three audio libraries, aifc, wave and sunau a similar API. Patch
and tests attached.
--
components: Library (Lib)
files: sunau.patch
keywords: patch
messages
Claudiu.Popa added the comment:
I love this idea! I was thinking while working on sunau/aifc/wave patches that
we can do more than this, unify the entire audio modules,
getting rid of Aifc_write/read and Wave_write/read was in fact my first desire.
One way that I thought about was to provide
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18615
___
___
Python-bugs
Claudiu.Popa added the comment:
Hello. Here's a basic patch with tests which accomplishes your request.
Currently, it defaults to None if no item failed, but probably it can be set
only when something fails the predicate (and the user will check using
hasattr(t, 'last
Claudiu.Popa added the comment:
Serhiy, at the first glance, that repr doesn't make sense to me, thus it seems
a little difficult to comprehend.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Claudiu.Popa added the comment:
Here's the new version. I added a few replies on the Rietveld.
--
Added file: http://bugs.python.org/file31744/sre_repr2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Claudiu.Popa added the comment:
Added the new version.
--
Added file: http://bugs.python.org/file31746/sre_repr3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Claudiu.Popa added the comment:
Hello. I attached a simple patch for the urlencode case. I'll try to make one
for parse_qs(l) as well.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file31782/urllib.patch
___
Python
Claudiu.Popa added the comment:
Added patch for parse_qsl as well.
--
Added file: http://bugs.python.org/file31785/urllib_null_value.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18857
Claudiu.Popa added the comment:
No problem, David, working on these patches is just an occasion for me to learn
more and be useful at the same time.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18857
Claudiu.Popa added the comment:
Hello, here's a basic patch. Currently, the header info is printed by default,
while the source can be retrieved by using --source (although I would prefer
them to be switched, the source should be shown by default and the header info
only when requested
Claudiu.Popa added the comment:
Hello. The 'File objects' section was removed in Python 3, with this issue
http://bugs.python.org/issue7508, but not the topic from pydoc.py. As I
understand, there is no point in having this, because there isn't a `file` type
anymore.
The attached patched
New submission from Claudiu.Popa:
Hello. The following seems a little weird:
Python 3.3.2 (v3.3.2:d047928ae3f6, May 16 2013, 00:03:43) [MSC v.1600 32 bit
(Intel)] on win32
m = memoryview(b'123')
list(m[::-1])
[51, 50, 49]
list(reversed(m))
Traceback (most recent call last):
File stdin
Claudiu.Popa added the comment:
So, in the following case, len shouldn't return 4, but the number of items?
Also, is it ok to assume that the number of items is
((*shape)[0] * ... * (*shape)[ndims-1])?
x = np.array([[1, 2, 3], [4, 5, 6], [4,5,6], [4,4,4]], np.int32)
x.shape
(4, 3)
x.shape[0
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17473
___
___
Python-bugs-list
Claudiu.Popa added the comment:
For multidimensional arrays it doesn't seem to work (yet).
x = numpy.array([[1,2,3], [4,5,6]])
list(reversed(x))
[array([4, 5, 6]), array([1, 2, 3])]
x.data
memory at 0x8032d06b8
list(reversed(x.data))
Traceback (most recent call last):
File stdin, line 1
Claudiu.Popa added the comment:
Hello. Attached patch which uses the new inspect.unwrap, added in
http://hg.python.org/cpython/rev/2aa6c1e35b8a.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file31902/inspect_getsource.patch
Claudiu.Popa added the comment:
Could anyone review this patch, please?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17442
___
___
Python-bugs
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19146
___
___
Python-bugs-list
Claudiu.Popa added the comment:
I already thought of that, but that doesn't work: the iterator version
would return the stack in the wrong order (note the .reverse() call in the
code).
Then, couldn't this:
stack = list(_extract_stack_iter(_get_stack(f), limit=limit
Claudiu.Popa added the comment:
Serhiy, are there any left issues with my latest patch? It would be nice if we
could get this into 3.4.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Claudiu.Popa added the comment:
Added the new patch, which addresses Serhiy's comments.
Also, this approach fails when bytes are involved:
import re
re.search(ba, ba)
Assertion failed: (PyUnicode_Check(op)), function _PyUnicode_CheckConsistency,
file Objects/unicodeobject.c, line 309
Claudiu.Popa added the comment:
Latest patch attached.
--
Added file: http://bugs.python.org/file32144/sre_repr5.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Claudiu.Popa added the comment:
I could use self-pattern-logical_size, but it seems that I still need the
call to getstring for bytes co, to obtain the view to the underlying buffer
(otherwise the group0 part from the repr will contain random bytes). I didn't
find a simpler way to achieve
Claudiu.Popa added the comment:
Working on a patch for this.
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19282
Claudiu.Popa added the comment:
Here's a patch.
--
keywords: +patch
Added file: http://bugs.python.org/file32203/dbm.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19282
Claudiu.Popa added the comment:
Attached patch checks that the db is actually closed after the `with`. Also, I
noticed that dbm.dumb doesn't behave like the rest of the variants, as seen in
the following:
import dbm.dumb as d
db = d.open('test.dat', 'c')
db.close()
db.keys()
Traceback
Claudiu.Popa added the comment:
Added patch based on Serhiy's, which addresses your comments. It drops the
group count and renames group0 to `match`.
--
Added file: http://bugs.python.org/file32216/sre_repr6.patch
___
Python tracker rep
New submission from Claudiu.Popa:
There are two small typos in statistics.rst.
--
assignee: docs@python
components: Documentation
files: typos.patch
keywords: patch
messages: 200480
nosy: Claudiu.Popa, docs@python
priority: normal
severity: normal
status: open
title: Typo
Claudiu.Popa added the comment:
Attached an attemptive patch.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file32322/reversed_proxy.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19359
Claudiu.Popa added the comment:
Nick, could you review this, please? It would be nice if we can flesh it out
and add it to 3.4.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17916
Claudiu.Popa added the comment:
Here's the updated version according to your comments. Because there is no need
for show_info() and info(), the current class is not a subclass of Bytecode (as
I implied from your first message). After issue19378 is fixed, we could change
the line_offset
New submission from Claudiu.Popa:
The actual error:
==
FAIL: test_exceptions_propagate (test.test_py_compile.PyCompileTests)
--
Traceback (most recent call
New submission from Claudiu.Popa:
This problem occurred in issue19282. Basicly, dbm.dumb is not consistent with
dbm.gnu and dbm.ndbm when the database is closed, as seen in the following:
import dbm.dumb as d
db = d.open('test.dat', 'c')
db.close()
db.keys()
Traceback (most recent call
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
keywords: +patch
Added file: http://bugs.python.org/file32347/dbm_dumb.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19385
Claudiu.Popa added the comment:
Yes, you can create files in a read-only directory if you are root, that's why
os.chmod(directory, stat.S_IREAD is redundant there.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19384
Claudiu.Popa added the comment:
There is issue19385 for the inconsistency of dbm.dumb, with a patch.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19282
Claudiu.Popa added the comment:
Here's the new version which addresses your last comment. Regarding the first
issue, I don't believe that the result will be as readable (but I agree with
you that it will be better). For instance, `items` will probably look like this:
try:
return
New submission from Claudiu.Popa:
When using a directive that is not recognised by time.strptime, the following
occurs in Python 3:
time.strptime(, %D)
Traceback (most recent call last):
File C:\Python33\lib\_strptime.py, line 320, in _strptime
format_regex = _TimeRE_cache.compile
New submission from Claudiu.Popa:
I think that wave.open should work with the `with` statement, given the fact
that Lib/aifc.py does this. I attached a simple patch for this. Thanks in
advance.
--
components: Library (Lib)
files: wave_context_manager.patch
keywords: patch
messages
Claudiu.Popa added the comment:
Hello. I received the confirmation for my CLA, could you commit the patch?
Thank you!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17487
New submission from Claudiu.Popa:
Hello!
Given the fact that issue 17487 was accepted, I think that is a good idea for
aifc.Aifc_read/Aifc_write.getparams to return a namedtuple as well, so that
both modules will behave consistently with each other.
I've attached a patch for this.
Thanks
Changes by Claudiu.Popa pcmantic...@gmail.com:
Added file: http://bugs.python.org/file29988/aifc_1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
Claudiu.Popa added the comment:
Hello. I changed the patch according to your review comments. Here is the new
version.
--
Added file: http://bugs.python.org/file30028/aifc_2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Claudiu.Popa added the comment:
I've modified the patch according to your comments, thanks! Now the result of
getparams() is picklable again.
--
Added file: http://bugs.python.org/file30052/aifc_3.patch
___
Python tracker rep...@bugs.python.org
http
Claudiu.Popa added the comment:
There is a regression with the latest patch, `getparams` output is no longer
picklable. The attached patch fixes this issue. Also, I renamed the internal
namedtuple to something more meaningful.
--
Added file: http://bugs.python.org/file30245
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
status: closed - open
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17487
___
___
Python-bugs
Claudiu.Popa added the comment:
Can anyone review the latest patch, please? Thanks.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17818
Claudiu.Popa added the comment:
Hello. Can I do something to move this issue forward?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17457
Changes by Claudiu.Popa pcmantic...@gmail.com:
Added file: http://bugs.python.org/file32453/dbm_dumb1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19385
Claudiu.Popa added the comment:
And the patch.
--
keywords: +patch
Added file: http://bugs.python.org/file32455/dbm_dumb_open.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18039
Claudiu.Popa added the comment:
Here's a patch which adds support for the `n` value of the flag. It makes the
dbm consistent across implementations and it's thus more reliable.
Also, attached a script which replicates the problem for Windows (where
dbm.dumb is used).
My output (after
Claudiu.Popa added the comment:
Hello. Patch attached.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file32473/logging.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18345
Claudiu.Popa added the comment:
Added documentation and the chown parameter for the friends of FileHandler.
Should tests be added for those classes as well or is it enough to test
FileHandler?
--
Added file: http://bugs.python.org/file32474/logging.patch
Changes by Claudiu.Popa pcmantic...@gmail.com:
Added file: http://bugs.python.org/file32475/logging.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18345
Claudiu.Popa added the comment:
Sure, thanks for the review, it's really insightful regarding API design. I
will try to contribute an example for the cookbook later this day.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15745
___
___
Python-bugs-list
New submission from Claudiu.Popa:
time.strptime leaks an IndexError, as seen in the following traceback.
[root@clnstor /tank/libs/cpython]# ./python
Python 3.4.0a4+ (default:0aa2aedc6a21+, Nov 5 2013, 17:10:42)
[GCC 4.2.1 20070831 patched [FreeBSD]] on freebsd8
Type help, copyright, credits
New submission from Claudiu.Popa:
Various exceptions raised by configparser module leaks implementation detail,
by chaining KeyErrors, as seen below:
Python 3.4.0a4+ (default:0aa2aedc6a21+, Nov 5 2013, 17:10:42)
[GCC 4.2.1 20070831 patched [FreeBSD]] on freebsd8
Type help, copyright, credits
Claudiu.Popa added the comment:
Michael, is any chance for this to go into Python 3.4? I would love to make any
changes necessary in order for this to happen.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17457
Claudiu.Popa added the comment:
Hello. Here's a preliminary patch.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file32596/resource.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19343
Claudiu.Popa added the comment:
Cool, thanks!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19282
___
___
Python-bugs-list mailing list
Claudiu.Popa added the comment:
Attached new patch, which addresses Ezio's comments on Rietveld.
--
Added file: http://bugs.python.org/file32664/unittest-17457-2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17457
Claudiu.Popa added the comment:
Hello! Here's a preliminary implementation, with tests for collections ABC. It
doesn't have any docs yet, though. I chose the name verify_full_api, but I'm ok
with check_methods as well.
--
keywords: +patch
nosy: +Claudiu.Popa
versions: +Python 3.4
New submission from Claudiu.Popa:
When issue17087 was accepted, the documentation wasn't changed to reflect the
new changes. The attached patch fixes this, replacing the old _sre.Match
object ... with the deterministic repr of the match object.
--
assignee: docs@python
components
Claudiu.Popa added the comment:
Eric, thank you for your comment! I modified the patch to check for __path__
instead, is this a safe enough check for this use case? Since ModuleSpec isn't
landed yet, I didn't use your __spec__ example.
--
Added file: http://bugs.python.org/file32697
Claudiu.Popa added the comment:
Ping, please review. I guess it is minimal enough to get into 3.4.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18615
Claudiu.Popa added the comment:
Hi. Since Python 3.2, compileall functions supports the optimization level
through the `optimize` parameter. I guess you are using Python2.7 or so? Also,
there's a note in compileall's documentation regarding the command line switch
for the optimization level
Claudiu.Popa added the comment:
Nick, what's your opinion on my latest patch?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17916
Claudiu.Popa added the comment:
Here's a patch which adds support for controlling the maxlevels on command
line. Currently, compileall uses a binary choice, either we don't process
subdirectories or we process at most 10 subdirectories. This seems to be the
case since its inception
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
versions: +Python 3.4 -Python 2.7, Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19628
Claudiu.Popa added the comment:
Thanks, Nick! Attached the new version, I hope that I did it right this time.
--
Added file: http://bugs.python.org/file32762/dis_tb_3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17916
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18864
___
___
Python-bugs-list
Claudiu.Popa added the comment:
Hello! Attached patch which uses ModuleSpec, tested with
http://hg.python.org/features/pep-451/ repo.
--
Added file: http://bugs.python.org/file32771/unittest_discovery_spec2.patch
___
Python tracker rep
Claudiu.Popa added the comment:
Added patch which addresses Eric's comments. It contains a situation which I'm
not so happy, mostly due to not knowing in depth (or at least at a comfortable
level) the import mechanics: in the AttributeError except handler, if the spec
can't be determined
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19545
___
___
Python-bugs-list
New submission from Claudiu.Popa:
Hello. I attached a minimal patch which fixes the re.compile display in
documentation, after issue13592 was closed.
--
assignee: docs@python
components: Documentation
files: sre_re.patch
keywords: patch
messages: 204402
nosy: Claudiu.Popa, docs@python
Claudiu.Popa added the comment:
Sorry for the delay. Here's a minimal doc patch. I don't excel at writing
documentation, it's not quite my strong point, so feel free to modify anything
you seem fit.
--
Added file: http://bugs.python.org/file32842/namespace.patch
Claudiu.Popa added the comment:
Here's a simple patch which adds this flag. It has no tests, because I noticed
that there are no tests for per-platform flags in test_socket.py (only for CAN,
RDS and general flags).
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http
Claudiu.Popa added the comment:
It was used by pydoc.py until 2006, when it was removed with this commit:
Changeset:
37821 (3135648026c4) Second phase of refactoring for runpy, pkgutil, pydoc, and
setuptools.
Is it worth to fix this?
--
nosy: +Claudiu.Popa
New submission from Claudiu.Popa:
Hi!
test_touch_common fails when using 8.3-STABLE FreeBSD 8.3-STABLE and Python
3.4.0b1 (default:a0ec33efa743+, Nov 30 2013, 10:36:58).
Here are the tracebacks:
==
FAIL: test_touch_common
Claudiu.Popa added the comment:
Hello! Here's a simple patch which makes sqlite.Row to act like a real sequence.
--
keywords: +patch
nosy: +Claudiu.Popa
versions: +Python 3.5 -Python 2.7, Python 3.1, Python 3.2
Added file: http://bugs.python.org/file32902/sqlite1.patch
Claudiu.Popa added the comment:
I believe it's similar, both test_os and test_pathlib fails when executed from
within a ZFS container. I checked, I did a fresh checkout of Python inside a
normal directory and run the tests there, they ran without problems
Changes by Claudiu.Popa pcmantic...@gmail.com:
--
nosy: +Claudiu.Popa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19840
___
___
Python-bugs-list
Claudiu.Popa added the comment:
Hi. Here's a patch which adds `copy_function` parameter to `shutil.move`.
--
keywords: +patch
Added file: http://bugs.python.org/file32917/shutil.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Claudiu.Popa added the comment:
Hello. Here's a patch for `expanduser()`.
--
keywords: +patch
nosy: +Claudiu.Popa
Added file: http://bugs.python.org/file32919/pathlib.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19776
Claudiu.Popa added the comment:
I guess not, the documentation already states that Row tries to mimic a tuple
in most of its features. Probably a MISC/News entry is required.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Claudiu.Popa added the comment:
Thanks, Vajrasky! Here's an updated patch.
--
Added file: http://bugs.python.org/file32968/sqlite2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10203
Claudiu.Popa added the comment:
Thanks, Vajrasky! Here's the new version of the patch.
--
Added file: http://bugs.python.org/file32980/pathlib1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19776
1 - 100 of 271 matches
Mail list logo