[Buildroot] [PATCH] package/tvheadend: change configuration directory

[Damjan Marion]

> On 30 Nov 2016, at 23:27, Thomas Petazzoni <thomas.petazzoni at free-electrons.com> wrote:
> 
> Hello,
> 
> On Wed, 30 Nov 2016 23:10:53 +0100, Yann E. MORIN wrote:
> 
>> On 2016-11-28 10:45 +0100, Damjan Marion spake thusly:
>>> Currently tvheadend configuration is stored in
>>> /home/tvheadend/.hts/tvheadend. This patch changes this location
>>> to /etc/tvheadend which seems to be more appropriate place.  
>> 
>> I disagree. This location is not only about "configuration" but also
>> about "state". It contains user-specified settings, recordings, DVB scan
>> status...
>> 
>> For example, accesscontrol can be updated from the WebUI to add/remopve
>> users or change their rights. I don't remember the full set of stuff
>> that gets in there, but there's almost no "configuration", but mostly
>> "settings" than can be updated a lot; the most ovious being records (not
>> the media streams, but the settings like channel, start and end times...)
>> 
>> Besides, /etc can be read-only, and we need a writable location. And
>> /etc is not a correct location to expect to be writable. /home is most
>> probably writable by users (although we do not provide that guarantee in
>> Buildroot, it is still much better than /etc).
>> 
>> So, it does not belong to /etc.
>> 
>> What was your rationale for moving it, besides "it seems more appropriate" ?
> 
> Could we have the configuration in /etc and the state in /var, like most
> sane Linux daemons do?

+1
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 1600 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20161130/c06fbff0/attachment-0001.p7s>