invalid syntax? about cruzdb

[brentp]

I haven't worked on cruzdb for some time. It's likely that there are many parts of it that do not support python3. I'd accept a pull request to fix that.

[radwaraed]

Many thanks Brent for your reply. I am new to this, so I didn't realize there was a pull request.

[sammosummo]

It seems this issue was never fixed, I experienced it today (2021-09-14).

[marcmaxson]

Still a problem today (2021-10-07)

`Python 3.8.11 (default, Aug  6 2021, 08:56:27) 
[Clang 10.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import cruzdb
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/anaconda3/envs/qc38/lib/python3.8/site-packages/cruzdb/__init__.py", line 5, in <module>
    from . import soup
  File "/opt/anaconda3/envs/qc38/lib/python3.8/site-packages/cruzdb/soup.py", line 1, in <module>
    from . import sqlsoup
  File "/opt/anaconda3/envs/qc38/lib/python3.8/site-packages/cruzdb/sqlsoup.py", line 458
    except KeyError, ke:
                   ^
SyntaxError: invalid syntax`

In python 3, the proper except is:

try:
    .....
except Exception as e:
    raise Exception(e)

the comma is no longer supported.

[brentp]

happy to accept a PR that fixes python3 compat.

[marcmaxson]

Thanks. I can't work on it immediately, but I will send in a PR if I get a chance to update it.

…

On Fri, Oct 8, 2021 at 1:53 AM Brent Pedersen ***@***.***> wrote: happy to accept a PR that fixes python3 compat. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA5DLVJEKOZMURIIBY2NG23UF2PRNANCNFSM4FTHOZMA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.