Neil Muller | 1 Nov 14:55 2011
Picon

psycopg2 autocommit fix missing from stable

Revision 4428 on trunk fixes a bug with psycog2's autocommit
attribute, but, although committed in July, the fix didn't make it
into 1.1.3 or the stable branch, AFAICS.

Would it be possible to get a 1.1.4 release including this fix?

Thanks.

--

-- 
Neil Muller
drnlmuller <at> gmail.com

I've got a gmail account. Why haven't I become cool?

------------------------------------------------------------------------------
RSA&reg; Conference 2012
Save &#36;700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Neil Muller | 1 Nov 14:59 2011
Picon

Fixing debian bug 605185

Debian currently has a bug against sqlobject for an insecure use of
PYTHONPATH in the docs/rebuild script -
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=605185 . While it is
a minor issue, it's easy enough to do the right thing, so seems worth
fixing.

Patch attached.

--

-- 
Neil Muller
drnlmuller <at> gmail.com

I've got a gmail account. Why haven't I become cool?
Attachment (sqlobject_605185.diff): text/x-diff, 620 bytes
------------------------------------------------------------------------------
RSA&reg; Conference 2012
Save &#36;700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss
Oleg Broytman | 1 Nov 15:03 2011
X-Face

Re: psycopg2 autocommit fix missing from stable

Hi!

On Tue, Nov 01, 2011 at 03:55:24PM +0200, Neil Muller wrote:
> Revision 4428 on trunk fixes a bug with psycog2's autocommit
> attribute, but, although committed in July, the fix didn't make it
> into 1.1.3 or the stable branch, AFAICS.
> 
> Would it be possible to get a 1.1.4 release including this fix?

   My opinion about the fix is that it's rather major change and I am a
bit nervous to include it into a minor release. It will be in SQLObject
1.2, I'm going to release the first beta this week.

> I've got a gmail account. Why haven't I become cool?

   :-D

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA&reg; Conference 2012
Save &#36;700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Oleg Broytman | 1 Nov 15:07 2011
X-Face

Re: Fixing debian bug 605185

On Tue, Nov 01, 2011 at 03:59:23PM +0200, Neil Muller wrote:
> Debian currently has a bug against sqlobject for an insecure use of
> PYTHONPATH in the docs/rebuild script -
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=605185 . While it is
> a minor issue, it's easy enough to do the right thing, so seems worth
> fixing.
> 
> Patch attached.
> 
> -- 
> Neil Muller
> drnlmuller <at> gmail.com

   Thank you!

> I've got a gmail account. Why haven't I become cool?

> Index: docs/rebuild
> ===================================================================
> --- docs/rebuild	(revision 4465)
> +++ docs/rebuild	(working copy)
>  <at>  <at>  -3,7 +3,7  <at>  <at> 
>  here=`pwd`
>  parent=`dirname $here`
>  echo "Adding $parent to \$PYTHONPATH"
> -export PYTHONPATH=$parent:$PYTHONPATH
> +export PYTHONPATH=$parent${PYTHONPATH:+:$PYTHONPATH}

   Strange syntax ${PYTHONPATH:+:$PYTHONPATH} . Are you sure? Shouldn't
it be just ${PYTHONPATH:+$PYTHONPATH} ?
(Continue reading)

Neil Muller | 1 Nov 15:31 2011
Picon

Re: Fixing debian bug 605185

On 1 November 2011 16:07, Oleg Broytman <phd <at> phdru.name> wrote:
> On Tue, Nov 01, 2011 at 03:59:23PM +0200, Neil Muller wrote:
>> Debian currently has a bug against sqlobject for an insecure use of
>> PYTHONPATH in the docs/rebuild script -
>> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=605185 . While it is
>> a minor issue, it's easy enough to do the right thing, so seems worth
>> fixing.
>>
>> Patch attached.
>>
>> --
>> Neil Muller
>> drnlmuller <at> gmail.com
>
>   Thank you!
>
>> I've got a gmail account. Why haven't I become cool?
>
>> Index: docs/rebuild
>> ===================================================================
>> --- docs/rebuild      (revision 4465)
>> +++ docs/rebuild      (working copy)
>>  <at>  <at>  -3,7 +3,7  <at>  <at> 
>>  here=`pwd`
>>  parent=`dirname $here`
>>  echo "Adding $parent to \$PYTHONPATH"
>> -export PYTHONPATH=$parent:$PYTHONPATH
>> +export PYTHONPATH=$parent${PYTHONPATH:+:$PYTHONPATH}
>
>   Strange syntax ${PYTHONPATH:+:$PYTHONPATH} . Are you sure? Shouldn't
(Continue reading)

Oleg Broytman | 1 Nov 15:41 2011
X-Face

Re: Fixing debian bug 605185

On Tue, Nov 01, 2011 at 04:31:43PM +0200, Neil Muller wrote:
> On 1 November 2011 16:07, Oleg Broytman <phd <at> phdru.name> wrote:
> >   Strange syntax ${PYTHONPATH:+:$PYTHONPATH} . Are you sure? Shouldn't
> > it be just ${PYTHONPATH:+$PYTHONPATH} ?
> 
> It does look a little strange, but it is correct.
> 
> It's the POSIX shell alternate value syntax with ":$PYTHONPATH" as the
> alternate value. If the ':' is excluded,  there's no separator between
> $parent and $PYTHONPATH when PYTHONPATH is set, and the separator must
> only be added when PYTHONPATH is already set to fix the bug.

   Got it. Thank you!

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA&reg; Conference 2012
Save &#36;700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Oleg Broytman | 1 Nov 15:10 2011
X-Face

Re: Fixing debian bug 605185

On Tue, Nov 01, 2011 at 06:07:39PM +0400, Oleg Broytman wrote:
> > +export PYTHONPATH=$parent${PYTHONPATH:+:$PYTHONPATH}
> 
>    Strange syntax ${PYTHONPATH:+:$PYTHONPATH} . Are you sure? Shouldn't
> it be just ${PYTHONPATH:+$PYTHONPATH} ?

   Ah, you want that ':' to be used with $PYTHONPATH -
$parent:$PYTHONPATH when $PYTHONPATH is not empty. I see now!

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA&reg; Conference 2012
Save &#36;700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Oleg Broytman | 2 Nov 20:04 2011
X-Face

Re: Fixing debian bug 605185

On Tue, Nov 01, 2011 at 03:59:23PM +0200, Neil Muller wrote:
> +export PYTHONPATH=$parent${PYTHONPATH:+:$PYTHONPATH}

   Applied and committed in the revision 4466 in the trunk. Will be in
version 1.2. Thank you! You can report to the Debian bug tracker the bug
is fixed upstream:
http://sourceforge.net/mailarchive/forum.php?thread_name=E1RLg2r-0000jo-GB%40webwareforpython.org&forum_name=sqlobject-cvs

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Oleg Broytman | 2 Nov 20:13 2011
X-Face

Re: Fixing docs/rebuild script (was: debian bug 605185)

I also did two commits fixing other minor problems with the script.

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
Oleg Broytman | 5 Nov 15:58 2011
X-Face

Re: SQLObject 1.2.0b1

On Sat, Nov 05, 2011 at 06:52:03PM +0400, Oleg Broytman wrote:
> * ForeignKey('Table', refColumn='refcol_id') to allow ForeignKey to
>   point to a non-id column; the referred column must be a unique integer
>   column.

   Consider this as an experimental feature. Use with care and test
thoroughly. Please note - MultipleJoin still uses `id` column.

> * Quoting rules changed for PostgreSQL: SQLObject uses E'' escape string
>   if the string contains characters escaped with backslash.

   Take care if you have your own objects that implement __sqlrepr__ -
if you have code like sqlrepr(obj)[1:-1] to remove quotes - it should be
written now as unqoute_str(sqlrepr(obj)). Code like "'%s'" % str should
be now quote_str(str).

Oleg.
--

-- 
     Oleg Broytman            http://phdru.name/            phd <at> phdru.name
           Programmers don't die, they just GOSUB without RETURN.

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1

Gmane