Bugs item #1194181, was opened at 2005-05-03 02:39
Message generated for change (Settings changed) made by gbrandl
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1194181&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Python Library
Group: Python 2.4
Status: Closed
Resolution: Fixed
Priority: 8
Submitted By: Bob Ippolito (etrepum)
>Assigned to: Georg Brandl (gbrandl)
>Summary: bz2.BZ2File doesn\'t handle modes correctly

Initial Comment:
I've noticed that if I specify a file mode of 'U' to bz2.BZ2File, it will 
erase my file.  That's bad.  Specifying 'rU' works as expected.  
Basically, it assumes that you want a writable file unless it sees an 
'r' in the mode, or no mode is given.  Ugh.

I've attached a patch that fixes this on CVS HEAD, it looks like it 
should apply cleanly to 2.4 as well.  This is a backport candidate if 
I've ever seen one.

----------------------------------------------------------------------

Comment By: Georg Brandl (birkenfeld)
Date: 2005-06-03 19:47

Message:
Logged In: YES 
user_id=1188172

Ok, checked in as

Modules/bz2module.c r1.24 r1.23.2.1
Lib/test/test_bz2.py r1.17 r1.16.2.1


----------------------------------------------------------------------

Comment By: Bob Ippolito (etrepum)
Date: 2005-06-03 18:23

Message:
Logged In: YES 
user_id=139309

It should still be OK to check in.

----------------------------------------------------------------------

Comment By: Georg Brandl (birkenfeld)
Date: 2005-06-03 18:16

Message:
Logged In: YES 
user_id=1188172

Okay to check in?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1194181&group_id=5470
_______________________________________________
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to