Opened 11 years ago
Closed 10 years ago
#681 closed enhancement (fixed)
replace mediagoblin.ini for mediagoblin.ini.sample
Reported by: | nico cesar | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 0.8.0 |
Component: | programming | Keywords: | bitesized |
Cc: | Parent Tickets: |
Description
I feel more natural using ".sample" suffix instead of having 2 confusing .ini files
Change History (6)
follow-up: 3 comment:1 by , 11 years ago
Keywords: | bitesized added |
---|---|
Status: | new → accepted |
comment:2 by , 11 years ago
Owner: | set to |
---|---|
Status: | accepted → in_progress |
comment:3 by , 11 years ago
Replying to cwebber:
- Add a makefile rule to do this copy if mediagoblin.ini does not exist.
- Update docs if needed, which given the makefile update, it might not be needed at all.
Exactly what should be done here? Should a cp
operation be hard-coded into Makefile.in
? If so, should this command be added under install
?
So far, I've renamed the file and added *.ini.sample
to the recursive-includes in MANIFEST.in
.
comment:4 by , 10 years ago
Milestone: | → 0.8.0 |
---|
This really ought to happen. Moving to 0.8.0.
Our makefile situation is pretty... unclear at the moment. As such, I suggest this being a docs change, among other things.
Bonus points if mediagoblin doesn't see the file and does the copying itself. I think that would actually make this easiest.
comment:5 by , 10 years ago
qguv, not sure if you're still interested in working on this... (sorry about the long delay on replying, too). Let me know.
comment:6 by , 10 years ago
Owner: | removed |
---|---|
Resolution: | → fixed |
Status: | in_progress → closed |
Closing. We are now using mediagoblin.example.ini, and the makefile does things right.
This is a good idea, and a bitesized issue for whoever might want to take it on.
There are really only three steps I think:
git mv mediagoblin.ini mediagoblin.ini.sample