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)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: davedash, Assigned: davedash)
References
Details
Attachments
(1 file)
379.79 KB,
patch
|
clouserw
:
review-
|
Details | Diff | Splinter Review |
Conf files can be shell scripts, use the standard python way of getting DB settings to populate the config file.
Assignee | ||
Updated•15 years ago
|
Summary: Rewrite conf file in python → Rewrite sphinx conf file in python
Assignee | ||
Comment 1•15 years ago
|
||
* 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 2•15 years ago
|
||
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-
Assignee | ||
Comment 3•15 years ago
|
||
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
Assignee | ||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Attachment #404727 -
Flags: review?(chizu)
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•