Bug#637192: /var/lib/mpd-sima/sima.db should be group writeable/readable

[Geoffroy Youri Berret]

Hi,

Le 27/08/2011 13:05, efrim at azylum.org wrote:
> Le 09/08/2011 12:45, Nicolas Évrard a écrit :
>> […]
>> The file /var/lib/mpd-sima/sima.db should be group readable/writeable
>> [...]
>> I also think that in the case where simadb_cli do not find a local
>> sima.db file it should use the system wide database file.
> 
> It would be great, but I don't know how FHS [0] is honoured in other
> distributions and thus how this feature would affect them.

Actually, I think, upstream maintainer's hat on, that looking for the system
wide database within /var/lib/ is not that straight forward if it has to be
done upstream. It depends on how package maintainers did name the package and
thus the directory within /var/lib. The upstream software name is "MPD_sima"
which is renamed "mpd-sima" in debian.

Then I believe, as package maintainer now, it is probably better to duplicate
simadb_cli wrapper used in debian package.
We'll have one system wide wrapper looking for /var/lib/mpd-sima/sima.db and
the second looking in the regular user directory:
  "${XDG_DATA_HOME}/mpd_sima/sima.db"

I guess I would set "simadb_cli_sw" to look for system wide database and leave
"simadb_cli" as is.

In the mean time you can use a shell alias:
	alias simadb_cli="simadb_cli --dbfile=/var/lib/mpd-sima/sima.db"

Thanks for your report Nicolas

Cheers,
	Geoff