499358 - reading a README is too hard, clouserw wants an automatic settings.py

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Maintenance Scripts, defect, P5)

Description

[Jeff Balogh (:jbalogh)]

Attachment: figure out schematic settings automatically

Particularly, one that reads the db connection parameters from config.php and passes them along to schematic.  He asked for a php interpreter, I give him some regex.

Comment 1

[Fred Wenzel [:wenzel]]

When you are done with this, you should write a PHP interpreter in Python. It's like PyPy [1], except for that it's not.

[1] http://codespeak.net/pypy/dist/pypy/doc/

Comment 2

[Jeff Balogh (:jbalogh)]

I shall call it "sadness".

Comment 3

[Wil Clouser [:clouserw]]

Tried to review this but schematic.py won't run for me:

Traceback (most recent call last):
  File "bin/schematic/schematic.py", line 83, in ?
    @exception
  File "bin/schematic/schematic.py", line 80, in exception
    return type(f.__name__, (SchematicError,), {'__init__': __init__})
TypeError: a new-style class can't have only classic bases

Comment 4

[Jeff Balogh (:jbalogh)]

Fie on your python 2.4!

Comment 5

[Jeff Balogh (:jbalogh)]

Committed a py24 fix for schematic in r28230.

Comment 6

[Fred Wenzel [:wenzel]]

Comment on attachment 384140 [details] [diff] [review]
figure out schematic settings automatically

FWIW, this works for me.

Comment 7

[Wil Clouser [:clouserw]]

Attachment: ignore comments

Wenzzeeellll!!!!

Yeah, this works for me too.  It didn't at first because I have some stuff commented out in my file to make switching db's easy.  Patch attached to ignore comments.

Comment 8

[Fred Wenzel [:wenzel]]

(In reply to comment #7)
> Wenzzeeellll!!!!

hope this isn't the sound of clouserw wanting to kill me ;)

Comment 9

[Wil Clouser [:clouserw]]

(In reply to comment #8)
> (In reply to comment #7)
> > Wenzzeeellll!!!!
> 
> hope this isn't the sound of clouserw wanting to kill me ;)

It was my best captain kirk voice.

Comment 10

[Jeff Balogh (:jbalogh)]

Added in r28284 with clouser's fix.

QA: this is a dev tool, so you should probably persuade a developer to verify.