Closed Bug 520642 Opened 15 years ago Closed 15 years ago

Rewrite sphinx conf file in python

Categories

(addons.mozilla.org Graveyard :: Public Pages, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: davedash, Assigned: davedash)

References

Details

Attachments

(1 file)

Conf files can be shell scripts, use the standard python way of getting DB settings to populate the config file.
Summary: Rewrite conf file in python → Rewrite sphinx conf file in python
Attached patch v1Splinter Review
* Moved the sphinx config to its own directory
* Wrote a localsettings.py which reads config.php to get mysql settings and can be overridden by IT without much fuss.

This will help Chizu to be able to automate sphinx updates.
Attachment #404727 - Flags: review?(thardcastle)
Attachment #404727 - Flags: review?(clouserw)
Comment on attachment 404727 [details] [diff] [review]
v1

r- because this has a ton of .po file changes (?)

Also, we're already reading the settings out of config.php for the migration scripts.  Can't we add these settings to config.php and read them out with python as well?  That way we could keep all the config in one file.
Attachment #404727 - Flags: review?(clouserw) → review-
Sorry the diff was probably bad.  I accidentally checked this (the sphinx conf) in this morning  :/

But for the record it *does* read the config.php file like the migration scripts, etc.  So you wouldn't have to change anything.  The localsettings.py just reads config.php (using the migrations code).

Sorry about the accidental check in, I was fixing something in the prime-sphinx-index.py script and forgot about this change:

http://viewvc.svn.mozilla.org/vc?view=revision&revision=52935
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Attachment #404727 - Flags: review?(chizu)
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: