Bug 8310 - All parts an instance should be under one path
Summary: All parts an instance should be under one path
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Packaging (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: David Dean
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-25 13:57 UTC by David Dean
Modified: 2015-12-03 22:00 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Who signed the patch off:
Text to go in the release notes:
Version(s) released in:


Attachments
Patch storing all instance folders under /var/lib/koha/instance name, with symlinks (3.98 KB, patch)
2012-07-09 14:41 UTC, David Dean
Details | Diff | Splinter Review
Bug 8310 : Patch to store all instances in /var/lib/koha - For use with the debian packages (4.40 KB, patch)
2012-08-23 23:47 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description David Dean 2012-06-25 13:57:57 UTC
For mobility and to potentially enable chroot down the line, the directories now created under /etc/koha/sites and /var/(spool|lib|lock|run|log)/koha should be created under a common directory.

I'd be happy to contribute a patch if a good common path can be found.
Comment 1 Robin Sheat 2012-06-25 14:04:00 UTC
I'm not convinced that debian would let us get away with that. Also, /etc and /var are the appropriate places for what goes into them. 

This said, it wouldn't be the end of the world to symlink them together, so they still appear like they are now, but everything is really under /var/lib/koha.
 
Additionally, it would only really need to be /var/lib/koha, all the rest are transient data.

Not completely sold on the need for it vs. the additional complexity though.
Comment 2 David Dean 2012-07-09 14:39:32 UTC
Fair enough! :)

Here is a patch in case it's helpful to others looking to move instances around, or to give hosted catalog customers limited shell access.
Comment 3 David Dean 2012-07-09 14:41:05 UTC Comment hidden (obsolete)
Comment 4 Chris Cormack 2012-08-23 23:43:17 UTC
Patch is not a git patch, reformatting now
Comment 5 Chris Cormack 2012-08-23 23:47:23 UTC
Created attachment 11819 [details] [review]
Bug 8310 : Patch to store all instances in /var/lib/koha - For use with the debian packages
Comment 6 Jared Camins-Esakov 2012-08-27 20:05:02 UTC
As I understand it, candidate packages for inclusion in Debian must use the standard Debian file layout, as in the current packages. Therefore, I would suggest the patch is probably not suitable for inclusion in Koha master, though the patch might be useful as an option for people running their own hand-rolled packages.