[vdr] Filesystem hierachy standard patch needs review.

Udo Richter udo_richter at gmx.de
Sat Apr 7 13:22:46 CEST 2012


Am 06.04.2012 15:01, schrieb Christopher Reimer:
> could someone please review the attached patch? It's originally posted
> by Maniac in this thread -->

On the original topic, I see room for one improvement:

The defaults of the new --cachedir and --resdir parameter are the
defaults set by Make.config's CACHEDIR and RESDIR. This is a minimal
change in behavior to previous versions, where these were handled by
--video and --config. So any old VDR that gets updated will behave
different in case that --video or --config is set, and now needs to also
have --cachedir and --resdir set. On the other hand, there needs to be a
way for distributions to set their defaults so that VDR finds them
automatically.


My suggestion:
Allow to keep the default CACHEDIR and RESDIR un-set (empty), and if
--cachedir or --resdir is not present, default to whatever --video and
--config is actually set to. That way, the package builder can decide
whether to set CACHEDIR and RESDIR or not, and if they're set, he has to
make sure that the startup scripts get modified to support --cachedir
and --resdir if necessary.

For an extra, an explicit --cachedir="" and --resdir="" could also
override an explicit CACHEDIR and RESDIR and revert to duplicating
--video and --config.


Cheers,

Udo



More information about the vdr mailing list