Ticket #10467 (closed defect: fixed)

Opened 8 months ago

Last modified 8 months ago

api.py - "with" gives SyntaxError building with python2.5 but ok with python2.6

Reported by: dale.miller@ngc.com Assigned to: bobbysmith007
Priority: normal Component: NeverNotifyUpdaterPlugin
Severity: normal Keywords:
Cc: dale.miller@ngc.com Trac Release: 1.0

Description

I have Trac on two procecessors:

  • openSUSE 10.3 with Python 2.5.1
  • openSUSE 11.2 with Python 2.6.2

I am upgrading to Trac 1.0 and got this error.

using nevernotifyupdaterplugin_0.13-r12113.zip

python setup.py bdist_egg (with Python 2.5.1) 

build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py:34: Warning: 'with' will become a reserved keyword in Python 2.6
  File "build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py", line 34
    with self.env.db_query as db:

            ^
SyntaxError: invalid syntax

Builds clean using Python 2.6.2

From the Warning message - it is strange that it does not build on 2.5.1 but does on 2.6.2. It looks like they decided not to make "with" a reserved word in python 2.6.

Attachments

Change History

10/11/12 16:20:34 changed by anonymous

I did a diff between the api.py for 0.12 and for 0.13 and I should be able to use the 0.12 for my needs. 

> diff Trac-plugins-0.12/nevernotifyupdaterplugin/0.12/nevernotifyupdaterplugin/api.py Trac-plugins-1.0/nevernotifyupdaterplugin/0.13/nevernotifyupdaterplugin/api.py
17a18
>
31,32d31
<         cursor = self.db.cursor()
<         # Suppress the updater from the recipients
35,41c34,43
<         cursor.execute("SELECT author FROM ticket_change WHERE ticket=%s "
<                        "ORDER BY time DESC LIMIT 1", (tktid,))
<         for updater, in cursor: break
<         else:
<           cursor.execute("SELECT reporter FROM ticket WHERE id=%s",
<                          (tktid,))
<           for updater, in cursor: break
---
>         with self.env.db_query as db:
>             cursor = db.cursor()
>             # Suppress the updater from the recipients
>             cursor.execute("SELECT author FROM ticket_change WHERE ticket=%s "
>                            "ORDER BY time DESC LIMIT 1", (tktid,))
>             for updater, in cursor: break
>             else:
>               cursor.execute("SELECT reporter FROM ticket WHERE id=%s",
>                              (tktid,))
>               for updater, in cursor: break
43,44c45,46
<         cursor.execute("SELECT value FROM session_attribute WHERE name='email' and sid=%s;",(updater,))
<         for up_em, in cursor: break
---
>             cursor.execute("SELECT value FROM session_attribute WHERE name='email' and sid=%s;",(updater,))
>             for up_em, in cursor: break
}

10/11/12 16:45:09 changed by Dale

http://bitten.edgewall.org/ticket/217 is similar

10/11/12 16:47:41 changed by bobbysmith007

  • status changed from new to closed.
  • resolution set to fixed.

(In [12150]) added a from future import with_statement so this works in py2.5 fix #10467

10/11/12 16:48:25 changed by bobbysmith007

Thanks much for the bug report. I think the 0.13 branch should work in python 2.5 now

10/11/12 18:25:21 changed by Dale

  • status changed from closed to reopened.
  • resolution deleted.

reopened ticket -

on python-2.5.1 

byte-compiling build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py to api.pyc
  File "build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py", line 6
    from __future__ import with_statement
SyntaxError: from __future__ imports must occur at the beginning of the file

on python-2.6.2 

byte-compiling build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py to api.pyc
SyntaxError: ('from __future__ imports must occur at the beginning of the file', ('build/bdist.linux-x86_64/egg/nevernotifyupdaterplugin/api.py', 6, None, 'from __future__ import with_statement\n'))

10/11/12 18:31:42 changed by Dale

Moving the from __future__ import with statement line to be first line of file gives successful build in both python-2.5.1 and 2.6.1

10/11/12 20:05:29 changed by bobbysmith007

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [12151]) make from future work (move to top). Thanks again fix #10467

Add/Change #10467 (api.py - "with" gives SyntaxError building with python2.5 but ok with python2.6)




Change Properties
Action