Change by Brandon Schabell :
--
keywords: +patch
nosy: +brandonschabell
nosy_count: 2.0 -> 3.0
pull_requests: +25638
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27090
___
Python tracker
<https://bugs.p
Change by Brandon Schabell :
--
keywords: +patch
nosy: +brandonschabell
nosy_count: 5.0 -> 6.0
pull_requests: +25636
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27088
___
Python tracker
<https://bugs.p
Change by Brandon Weeks :
--
nosy: +bweeks
___
Python tracker
<https://bugs.python.org/issue38820>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Brandon :
Reading the PEP 572 document I don't see anything stating that Walrus operator
in list indexes must be enclosed in parenthesis.
Minimal Example:
'''
In [1]: a = list(range(10))
In [2]: idx = -1
In [3]: a[idx := idx +1]
File "", line 1
a[idx
New submission from Brandon :
The regular expression used for matching numbers in the documentation for the
regular expressions module (the tokenizer section) doesn't match the string
".5", but does match the string "3.".
Here's a link to the tokenizer section of the d
Change by Brandon Stansbury :
--
title: Base 85 encoding initialization race conditiong -> Base 85 encoding
initialization race condition
___
Python tracker
<https://bugs.python.org/issu
New submission from Brandon Stansbury :
Under multi-threading scenarios a race condition may occur where a thread sees
an initialized `_b85chars` table but an uninitialized `_b85chars2` table due to
the guard only checking the first table.
This causes an exception like:
```
File "/us
Change by Brandon James :
--
keywords: +patch
pull_requests: +14833
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15088
___
Python tracker
<https://bugs.python.org/issu
New submission from Brandon James :
When using the ipaddress library, all multicast addresses and networks return
True when using the is_global method for their respective classes. I believe
their are two possible fixes for this.
1) In practice no multicast addresses are globally routable
Brandon VanVaerenbergh - NOAA Affiliate added the comment:
+1 just experienced this issue migrating from Python3.4 to Python3.5 on windows
with bash (MINGW git bash) as primary shell
Temporarily resolved issue (on this workstation) by simply copying
Python35\Lib\venv\scripts\posix\activate
Changes by Brandon Rhodes <bran...@rhodesmill.org>:
--
type: -> enhancement
___
Python tracker <rep...@bugs.python.org>
<http://bugs.pyt
Brandon Rhodes added the comment:
Another important objection against the current text is that it stacks a series
of `and` and `or` operators at the same level of indentation, as though they
naturally evaluate in the order the programmer writes them. In fact, they have
different levels
Changes by Brandon Rhodes <bran...@rhodesmill.org>:
Added file: http://bugs.python.org/file42489/pep8-knuth.patch
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python
Changes by Brandon Rhodes <bran...@rhodesmill.org>:
Removed file: http://bugs.python.org/file42487/pep8-knuth.patch
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python
New submission from Brandon Rhodes:
I am delighted to see that PEP-8 has pivoted to breaking long formulae before,
rather than after, each binary operator! But I would like to pivot the PEP away
from citing my own PyCon Canada talk as the authority on the matter, and toward
citing Knuth
Brandon Milam added the comment:
I've tested the new patch and it is still able to properly find both chrome and
firefox and is able to differentiate between new window and new tab for those
two browsers so it is still working. Would someone review the patch
Brandon Zerbe added the comment:
I am using Python 2.7.5. The segment of code from inspect that I previously
extracted came from line 332 although you may also find it by "finding"
_searchbases.
This is really an issue with Forthon:
http://hifweb.lbl.gov/Forthon/
Specifically t
New submission from Brandon Zerbe:
I am using a possibly non-standard python package called Forthon, and when I
inspect an object that is dependent on the Forthon class, I get the following
error:
File "/Users/zerbeb/homemade_programs/config2class/src/method_parsing.py"
Brandon Milam added the comment:
applying 25005_1.patch
patching file Lib/webbrowser.py
Hunk #1 FAILED at 498
Hunk #2 FAILED at 524
Hunk #3 FAILED at 532
Hunk #4 FAILED at 540
Hunk #5 FAILED at 548
I'm trying to apply your patch after applying webbrowserfix6.patch but I am
encountering problems
Brandon Milam added the comment:
Ok I've been able to test the new patch now and I'm not sure that os.startfile
is going to work. I've been able to get os.startfile() to open a specified
browser (>>> os.startfile("chrome.exe", "open")), however, the functi
Brandon Milam added the comment:
Finally got it rebuilt after having trouble with visual studio for awhile. I've
tested the new patch and it is still able to properly find both chrome and
firefox and is able to differentiate between new window and new tab for those
two browsers so it appears
Brandon Milam added the comment:
Boštjan Mejak the windows issue has been addressed in issue 8232 and recently
patched for 3.5. http://bugs.python.org/issue8232
--
nosy: +jbmilam
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
Brandon Milam added the comment:
Moved the 64 bit browser list to its own loop and switched to browsers.append
rather than +=.
--
Added file: http://bugs.python.org/file39650/webbrowserfix6.patch
___
Python tracker rep...@bugs.python.org
http
Brandon Milam added the comment:
Here's a patch addressing all of the comments in the review. Changing the
browsers from a set to a list though resulted in duplicates in the _tryorder
list that were not present before because the set had filtered the duplicates
before the partial string
Brandon Milam added the comment:
They are correct. 'cum' is not one of the available keywords and so here is the
fix changing it to say 'cumulative' for consistency as ramiro suggested.
--
keywords: +patch
nosy: +jbmilam
Added file: http://bugs.python.org/file39627
Brandon Milam added the comment:
Forgive me the excessive number of patch submissions as I am still getting my
feet wet in contributing to Python. I'm posting another patch that is not
functionally different from the last patch but should better adhere to the PEP8
style guide.
Please let me
Brandon Milam added the comment:
I've been looking over the issue and the error is just raised by the stdout
change not the stderr change (when the stdout line is commented out in the
setAutoFlush function no error is raised). The flush method doesn't seem to be
required as Serhiy pointed out
Changes by Brandon Milam jmilam...@gmail.com:
--
keywords: +patch
Added file: http://bugs.python.org/file39560/csv_skipinitialspace_docfix.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21297
Changes by Brandon Milam jmilam...@gmail.com:
Added file: http://bugs.python.org/file39559/csv_skipinitialspace_testing.csv
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21297
Brandon Milam added the comment:
This code shows what Daniel Andersson was talking about. I changed the
whitespace references in the documentation that Daniel mentioned to say
spaces. Also I changed ignore space at the start of the field to ignore
spaces at the start of the field due
Brandon Milam added the comment:
Here I added on to the Dialects and Formatting Parameters paragraph explaining
that the defaults listed are for the excel dialect and that all the attributes
need to be specified if the user is wanting to create custom dialects through
sub-classing. I
Changes by Brandon Milam jmilam...@gmail.com:
Added file: http://bugs.python.org/file39553/csv.html
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24147
Brandon Milam added the comment:
I believe this was the requested change. Let me know if more was desired.
--
keywords: +patch
nosy: +jbmilam
Added file:
http://bugs.python.org/file39540/register_dialect_docstring_fix.patch
___
Python tracker rep
Brandon Milam added the comment:
Sorry, I forgot an end parentheses in the doc string of the last patch.
--
Added file:
http://bugs.python.org/file39541/register_dialect_docstring_fix.patch
___
Python tracker rep...@bugs.python.org
http
Brandon Milam added the comment:
Hi all,
I've been looking at this bug and am ready to start putting in some work on it
but I have some questions about what is wanting to be done. From what I can
tell these are the possible tasks for this issue.
- Add to the docs under the dialect section
Brandon Milam added the comment:
I went ahead and took the assert statement out and added support for vista
using a union of sets for both the 32 bit and 64 bit locations.
--
Added file: http://bugs.python.org/file39533/webbrowserfix3.patch
Brandon Milam added the comment:
On second thought no type testing is required if sets are used because the
union will take out duplicates anyways and so I removed the type testing and
left in the set union code.
--
Added file: http://bugs.python.org/file39534/webbrowserfix3.patch
Brandon Milam added the comment:
I kept the changes to the WindowsDefault.open() method and used and extended
eryksun's code to build the browser list using the registry. Also I added
support for a few more browsers. Some of the browsers I could not find ways to
differentiate between opening
Brandon Milam added the comment:
I got rid of the __init__ for the WindowsDefault class that I asked about
earlier and changed it to match the sub-classing model that the Unix browsers
use. This caused some changes in the get function too. Due to the _isexecutable
still not completely working
Brandon Milam added the comment:
In order to fix the issue I added on to the WindowsDefault class so that it is
the main browser class for windows platforms as opposed to being a default when
no other browser is given. I gave the class an init where it specifies specific
flags for firefox
Brandon Milam added the comment:
How the _isexecutable function is set up now it would require a full path name
in order to be able to tell if a specific browser is on the system. The area
under platform support for windows checks for multiple browsers using this
function but only passes
Brandon Rhodes added the comment:
Now that I am back at a full keyboard, I see that my previous response
to @BreamoreBoy about this issue is too short and too cryptic to really
serve as a fair answer to his question. And, with some embarrassment, I
note that my words did not even achieve
New submission from Brandon Rhodes:
There was an old document in the howto folder whose advice was in many cases
flat-out wrong, so Raymond Hettinger performed a wonderful public service by
deleting it back in 2011:
http://hg.python.org/cpython/rev/80ff78425419
Unfortunately it looks like
Changes by Brandon Rhodes bran...@rhodesmill.org:
--
assignee: - docs@python
components: +Documentation
nosy: +docs@python
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21956
Brandon Rhodes added the comment:
The question of whether the document ought to be removed is not at issue here.
The document was already deleted, in 2011, by Raymond Hettinger, with the
consent of its author. I told that story merely as background.
The issue here is that the Python web site
Brandon Rhodes added the comment:
I do not find it unreasonable, on a page of Python idioms, the we would call an
example that explicitly says Don't in its title an anti-idiom.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org
New submission from Brandon:
Observe the following code:
import MySQLdb, MySQLdb.cursors, datetime
... mysqlCursor is a cursor object from a connection to database from the
MySQLdb module ...
mysqlCursor.execute(SELECT NOW())
timeRow = mysqlCursor.fetchall()
currentDateTime
Brandon added the comment:
Type returned as datetime, I was not familiar with the MySQLdb code. Sorry for
the bad report.
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21570
Changes by Brandon x...@codeslum.org:
--
resolution: - not a bug
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21570
___
___
Python-bugs-list
New submission from Brandon Rhodes:
Content-Disposition is an optional header field. In its absence, the MUA may
use whatever presentation method it deems suitable. — RFC 2183
The email.message.Message class should gain a get_content_disposition() method
with the three possible return values
Brandon Rhodes added the comment:
Thanks — done! http://bugs.python.org/issue21083
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21079
New submission from Brandon Rhodes:
I love properties and think they should be everywhere. But consistency is more
important, so I suspect that EmailMessage.is_attachment should be demoted to a
normal method. Why? Because if it remains a property then I am likely to first
write
New submission from Brandon Rhodes:
Currently, the new wonderful EmailMessage class ignores the encoding specified
in any Header objects that are provided to it.
import email.message, email.header
m = email.message.Message()
m['Subject'] = email.header.Header('Böðvarr'.encode('latin-1
New submission from Brandon Rhodes:
In Python 3, fileinput.input() returns str lines whether the data is
coming from stdin or from a list of files on the command line. But if
input(mode='rb') is specified, then its behavior becomes inconsistent:
lines from stdin are delivered as already-decoded
New submission from Brandon Rhodes:
Most attachments (in my inbox, at least) specify a filename, and thus
have a Content-Disposition header that looks like:
Content-Disposition: attachment; filename=attachment.gz
In fact, this sample header was generated by the new add_attachment()
method
Brandon Rhodes added the comment:
Oh - this also, happily, explains why iter_attachments() is ignoring
all of the attachments on my email: because it internally relies upon
is_attachment to make the decision. So this fix will also make
iter_attachments() usable
Brandon Rhodes added the comment:
Okay, having looked at the source a bit more it would probably make
more sense to use _splitparam() instead of doing the split manually.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21079
Brandon Rhodes added the comment:
Given that methods like get_param() already exist for pulling data out of
the right-hand-side of the ';' in a parameterized email header, would it
be amiss for EmailMessage to also have a method that either returns
everything to the left of the semicolon
Brandon Rhodes added the comment:
Understood. I wonder where in the documentation the ability to get the content
disposition should wind up? I am almost tempted to suggest a
get_content_disposition() method that parallels get_content_type(), mostly to
avoid having to document the asymmetry
Brandon Rhodes added the comment:
I agree that is_attachment supports the most common use-case of people who need
to inspect the content disposition!
But people implementing heavyweight tools and email clients might additionally
need to distinguish between a MIME part whose disposition
New submission from Brandon Rhodes:
Regular expression re.MatchObject objects are sequences.
They contain at least one “group” string, possibly more,
which are integer-indexed starting at zero.
Today, groups can be accessed in one of two ways.
(1) You can call the method match.group(N).
(2
Changes by Brandon Rhodes bran...@rhodesmill.org:
--
versions: +Python 3.4 -Python 3.5
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19536
Brandon Craig Rhodes added the comment:
Kristján, you are certainly correct that a single-argument that can be either a
filename or a cert is inappropriate; we should not be peeking inside of strings
to guess what they contain.
And I think you also have a good point about Pythonic-ness when
Brandon Craig Rhodes added the comment:
Kristján, your patch is a wonderful idea—I am about to commit production code
that will have to create tens of thousands of temporary files during operation,
one file each time SSL is started up on a socket, which could be avoided if
something like
Brandon Craig Rhodes added the comment:
Adding an entirely separate API for introspection strikes me as
counter-productive — instead of merely having to maintain the logging API that
you already maintain, you will additionally now have an entirely separate and
second API that also has
Brandon Craig Rhodes added the comment:
Larry Hastings rep...@bugs.python.org writes:
Huh. tar *can* infer it from the data itself. On the other hand, it
chooses explicitly not to. I guess tar knows explicit is better
than implicit too ;-)
I am told that the refusal of tar to introspect
New submission from Brandon Craig Rhodes:
A friend (@theomn on Twitter) was just working off of PEP-333 when I mentioned
to him that PEP-, and he had never heard of it, and he expressed the wish
that PEPs would have a banner or something at the top if there is a more recent
version
Brandon Craig Rhodes added the comment:
(Corrected not to note in the title and went with enhancement)
--
title: PEP should not if it has been superseded - PEP should note if it has
been superseded
type: - enhancement
___
Python tracker rep
Brandon Craig Rhodes added the comment:
The original inspiration:
https://twitter.com/theomn/status/309468740611891200
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17370
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
The tarfile module should have a simple command line that allows it to be
executed with -m — even if its only ability was to take a filename and
extract it to the current directory, it could be a lifesaver on Windows
machines
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
In case Google brings anyone else to this bug: this error typically indicates
that a `threading.py` which is not actually the Standard Library's `threading`
module has somehow wound up on an earlier path in `sys.path
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Brett, yes, you are welcome to close this issue — Ned quite handily convinced
me that coverage code belongs in the coverage distribution, not languishing
about in the CPython source tree. That solution also quite beautifully solves
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Ezio and Sandro, thank you very much for your attention to this issue, and for
helping me split it into manageable chunks! To answer the question about why
coverage does not show as high a total as it ought: it's because coverage
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Éric, I think your points are good ones. (And, as I return to this patch after
three months, I should thank the PSF for sponsoring the CPython sprint here at
PyOhio, and creating this opportunity for me to continue trying to land
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Éric, I think your suggestions are all good ones, and I have incorporated them
into the file. (But do note that the departures we are now making from Ned's
own copy of the tracer code — removing the commented-out debugging
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
But if we allow for ellipsis, then would we not also have to start allowing
characters like ≥ and ≤ in Python? And the problem with any of these
(admittedly very attractive) substitutions is that they seem to abandon the
principle
New submission from Brandon W Maister quodlibe...@gmail.com:
The comment just before the random.betavariate method reads in part:
## See
## http://sourceforge.net/bugs/?func=detailbugbug_id=130030group_id=5470
## for Ivan Frohne's insightful analysis of why the original implementation:
## (snip
Brandon W Maister quodlibe...@gmail.com added the comment:
Oh, awesome, thanks!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11848
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Nick Coghlan rep...@bugs.python.org writes:
Regarding __reduce__, other readers will have the same question Éric
did, so that point should definitely go in a comment after the
__reduce_ex__ check.
I just sat down to review
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Nick Coghlan rep...@bugs.python.org writes:
Nick Coghlan ncogh...@gmail.com added the comment:
Regarding __reduce__, other readers will have the same question Éric
did, so that point should definitely go in a comment after
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Éric, after checking line 112 of the two patches and then of the new file, I
figured out that you meant line 112 of the old file — and, yes, that test can
go away too since in python3 complex always exists and unicode never
exists
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Éric, the Makefile in Python trunk seems to include Objects/complexobject.o in
the build unilaterally without any way to turn it off. What is leading you to
believe that Python 3 can conditionally turn the complex type off during
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Antoine, neither this issue, nor either version of my patch, was intended to
assert that 100% test coverage indicates that a test of tests are complete. If
you will point out where in the text this is implied, I will correct
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
The attached patch will bring Lib/copy.py to 100% test coverage.
A bug in coverage results in its only reporting 99% at the moment; see
coverage issue #122 on bitbucket:
https://bitbucket.org/ned/coveragepy/issue/122/for-else
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Benjamin, I would like some way to know when our tests achieve 100% coverage
because otherwise I will keep coming back to this module to add more tests and
have to re-discover code that is not CPython relevant. But for now I have
Changes by Brandon Craig Rhodes bran...@rhodesmill.org:
Removed file: http://bugs.python.org/file21245/test_copy2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue502085
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Benjamin, thanks for the pointers! The attached patch now uses assertIs() and
assertIsNot(), and calls self.fail() instead of using the exception from
support.
In the future I would like some way to determine when test coverage
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
When running the Python regression tests in coverage, the initial outer level
of interpreted code in several standard library modules shows as not having
been covered by the tests, because they were imported during the Python
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Here is a module that solves this problem if the tests are run with the
fullcoverage directory at the front of the PYTHONPATH, like this:
PYTHONPATH=Tools/fullcoverage ./python -m coverage run --pylib
Lib/test/regrtest.py
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
Normally, unittest cleanly reports an exception in a setUpClass method. But if
I place the attached test in a directory by itself and then run python -m
unittest discover -b from inside of the same directory, then instead
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
If you try doing msg.as_string() to a MIMEMultipart message that has not been
given a boundary, then it dies with this exception:
Traceback (most recent call last):
File mime_gen_alt.py, line 40, in module
print
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Here is a patch that fixes the problem. The problem probably only occurs if the
MIMEMultipart is actually given several MIME parts to use in its interior.
--
keywords: +patch
Added file: http://bugs.python.org/file20391
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
The only way to safely build shell command lines from inside of Python — which
is necessary when sending commands across SSH, since that behaves like
os.system() rather than like subprocess.call() — is to use the wonderful
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
The multiprocessing module uses a bare fork() to create child processes under
Linux, so the children get a copy of the entire state of the parent process.
But under Windows, child processes are freshly spun-up Python
Brandon Craig Rhodes bran...@rhodesmill.org added the comment:
Jesse, it's great to learn it's on your wish list too!
Should I design the patch so that (a) there is some global in the module that
needs tweaking to choose the child creation technique, or (b) that an argument
to the Process
New submission from Brandon Craig Rhodes bran...@rhodesmill.org:
On this page, the Style Guide for people who want to try contributing to the
Python documentation:
docs.python.org/documenting/style.html
there is a broken link to the Apple Style Guide. The 2008 edition now seems
gone
New submission from brandon drr...@gmail.com:
Multiprocessing goes into an infinite loop during shutdown, trying to connect
to a remote queue - I *think* during finalization.
The actual loop appears to be the while(1) in connection.py line 251, and I
think it is being called initially from
brandon drr...@gmail.com added the comment:
After getting into the infinite loop, here's exception from CTRL-C (posted just
to show stack trace -- i am still working on getting come sample code together
-- meant to show it is infact in that while(1) a 251 in connection.py):
KeyboardInterrupt
span style=font-family:#39;Times New Roman#39;;font-size:mediumdiv
style=background-repeat:initial;background-color:rgb(255, 255,
255);padding-top:0.6em;padding-right:0.6em;padding-bottom:0.6em;padding-left:0.6em;margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px
span
Brandon Bloom snprbo...@gmail.com added the comment:
That is, in a true CGI environment, there can't be *multiple* requests
made to CGIHandler, and so it can't leak. In normal (i.e. pre-GAE)
long-running web environments, os.environ would not contain any request
information, only
1 - 100 of 114 matches
Mail list logo