Hello,

I'm a bit confused about the locale dir stuff in VDR 1.6.0 especially wrt. 
plugins.  newplugin hardcodes LOCALEDIR to $(VDRDIR)/locale without taking 
Make.config into account at all.  The result is that plugins will try to 
place their *.mo into a wrong location if LOCDIR was customized in in VDR's 
Make.config and/or "make install" was used.  I suppose the attached patch 
would fix that.

However that does not address weirdness in main VDR's locale dir handling; the 
value set for LOCDIR in Make.config gets built into vdr as the compile time 
default locale dir, but but "make install-i18n" will install the *.mo 
somewhere else unless one sets identical value for LOCALEDIR in Makefile 
(setting it in Make.config does not have any effect).  Are both LOCDIR and 
LOCALEDIR needed?  I'd try to clean this up and post a patch that fixes it if 
I knew what the intention is :)
--- newplugin~	2008-01-13 15:00:23.000000000 +0200
+++ newplugin	2008-04-06 19:34:52.000000000 +0300
@@ -84,6 +84,7 @@
 VDRDIR = ../../..
 LIBDIR = ../../lib
 TMPDIR = /tmp
+LOCDIR = \$(VDRDIR)/locale
 
 ### Allow user defined options to overwrite defaults:
 
@@ -129,9 +130,8 @@
 ### Internationalization (I18N):
 
 PODIR     = po
-LOCALEDIR = \$(VDRDIR)/locale
 I18Npo    = \$(wildcard \$(PODIR)/*.po)
-I18Nmsgs  = \$(addprefix \$(LOCALEDIR)/, \$(addsuffix /LC_MESSAGES/vdr-\$(PLUGIN).mo, \$(notdir \$(foreach file, \$(I18Npo), \$(basename \$(file))))))
+I18Nmsgs  = \$(addprefix \$(LOCDIR)/, \$(addsuffix /LC_MESSAGES/vdr-\$(PLUGIN).mo, \$(notdir \$(foreach file, \$(I18Npo), \$(basename \$(file))))))
 I18Npot   = \$(PODIR)/\$(PLUGIN).pot
 
 %.mo: %.po
@@ -144,7 +144,7 @@
 	msgmerge -U --no-wrap --no-location --backup=none -q \$\@ \$<
 	[EMAIL PROTECTED] \$\@
 
-\$(I18Nmsgs): \$(LOCALEDIR)/%/LC_MESSAGES/vdr-\$(PLUGIN).mo: \$(PODIR)/%.mo
+\$(I18Nmsgs): \$(LOCDIR)/%/LC_MESSAGES/vdr-\$(PLUGIN).mo: \$(PODIR)/%.mo
 	[EMAIL PROTECTED] -p \$(dir \$@)
 	cp \$< \$\@
 
_______________________________________________
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr

Reply via email to