Bug 8310

Summary:	All parts an instance should be under one path
Product:	Koha	Reporter:	David Dean <ddean>
Component:	Packaging	Assignee:	David Dean <ddean>
Status:	CLOSED INVALID	QA Contact:
Severity:	enhancement
Priority:	P5 - low	CC:	chris, jonathan.druart, robin
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Attachments:	Patch storing all instance folders under /var/lib/koha/instance name, with symlinks Bug 8310 : Patch to store all instances in /var/lib/koha - For use with the debian packages

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)

Created attachment 10716 [details] [review]
Patch storing all instance folders under /var/lib/koha/instance name, with symlinks

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.