Makefile variable names LCLBLD and PLGCFG
Commit Message
Hello,
Should probably have spoken earlier, but is there any particular reason
for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
have to be short for anything, it's not like we're running out of space
for their names anywhere, is it? Besides, I don't think LCLBLD describes
what its effects are very well, INPLACE would sounds better to me.
The ship may already have sailed for plgcfg in vdr.pc as it's being used
by many plugin Makefiles already, but I believe the attached patches
should be safe for 2.0.0.
Comments
On 17.03.2013 12:46, Ville Skyttä wrote:
> Hello,
>
> Should probably have spoken earlier, but is there any particular reason
> for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
> I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
> have to be short for anything, it's not like we're running out of space
> for their names anywhere, is it? Besides, I don't think LCLBLD describes
> what its effects are very well, INPLACE would sounds better to me.
>
> The ship may already have sailed for plgcfg in vdr.pc as it's being used
> by many plugin Makefiles already, but I believe the attached patches
> should be safe for 2.0.0.
I'm afraid it's too late for that.
Sorry for the time you wasted - you should have asked first.
Klaus
On 2013-03-17 14:00, Klaus Schmidinger wrote:
> I'm afraid it's too late for that.
HWLLBDCLLBTHVTRWY
On 03/17/13 13:00, Klaus Schmidinger wrote:
> On 17.03.2013 12:46, Ville Skyttä wrote:
>> Hello,
>>
>> Should probably have spoken earlier, but is there any particular reason
>> for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
>> I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
>> have to be short for anything, it's not like we're running out of space
>> for their names anywhere, is it? Besides, I don't think LCLBLD describes
>> what its effects are very well, INPLACE would sounds better to me.
>>
>> The ship may already have sailed for plgcfg in vdr.pc as it's being used
>> by many plugin Makefiles already, but I believe the attached patches
>> should be safe for 2.0.0.
>
> I'm afraid it's too late for that.
I do not understand why it should be too late.
2.0 is not out yet.
But if they really cannot have readable names,
at least I would not provide a default, but let
the makefile issue a message that they must
be set (as I suggested in the other thread).
Carsten.
On 17.03.2013 18:00, Carsten Koch wrote:
> On 03/17/13 13:00, Klaus Schmidinger wrote:
>> On 17.03.2013 12:46, Ville Skyttä wrote:
>>> Hello,
>>>
>>> Should probably have spoken earlier, but is there any particular reason
>>> for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
>>> I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
>>> have to be short for anything, it's not like we're running out of space
>>> for their names anywhere, is it? Besides, I don't think LCLBLD describes
>>> what its effects are very well, INPLACE would sounds better to me.
>>>
>>> The ship may already have sailed for plgcfg in vdr.pc as it's being used
>>> by many plugin Makefiles already, but I believe the attached patches
>>> should be safe for 2.0.0.
>>
>> I'm afraid it's too late for that.
>
> I do not understand why it should be too late.
> 2.0 is not out yet.
But we're in the final testing phase.
Besides, Ville's patch also touched the Makefiles of plugins - and
plugin authors and distribution managers react very intense when they
have to modify their files ;-).
> But if they really cannot have readable names,
> at least I would not provide a default, but let
> the makefile issue a message that they must
> be set (as I suggested in the other thread).
Sorry, no more changes for version 2.0.
Just bugfixes, if anything is really broken.
Klaus
Carsten Koch wrote:
> But if they really cannot have readable names,
> at least I would not provide a default, but let
> the makefile issue a message that they must
> be set (as I suggested in the other thread).
They don't have to be set, so why should the Makefile issue a message?
Yours
Manuel
Am Sonntag, den 17.03.2013, 18:06 +0100 schrieb Klaus Schmidinger:
> On 17.03.2013 18:00, Carsten Koch wrote:
> > On 03/17/13 13:00, Klaus Schmidinger wrote:
> >> On 17.03.2013 12:46, Ville Skyttä wrote:
> >>> Hello,
> >>>
> >>> Should probably have spoken earlier, but is there any particular reason
> >>> for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
> >>> I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
> >>> have to be short for anything, it's not like we're running out of space
> >>> for their names anywhere, is it? Besides, I don't think LCLBLD describes
> >>> what its effects are very well, INPLACE would sounds better to me.
> >>>
> >>> The ship may already have sailed for plgcfg in vdr.pc as it's being used
> >>> by many plugin Makefiles already, but I believe the attached patches
> >>> should be safe for 2.0.0.
> >>
> >> I'm afraid it's too late for that.
> >
> > I do not understand why it should be too late.
> > 2.0 is not out yet.
>
> But we're in the final testing phase.
> Besides, Ville's patch also touched the Makefiles of plugins - and
> plugin authors and distribution managers react very intense when they
> have to modify their files ;-).
If I am not mistaken, Ville’s issue was that the names were not clear.
Could a comment be added to the Makefile what the meaning is then. That
should not break anything.
[…]
Thanks,
Paul
On 20.03.2013 18:02, Paul Menzel wrote:
> Am Sonntag, den 17.03.2013, 18:06 +0100 schrieb Klaus Schmidinger:
>> On 17.03.2013 18:00, Carsten Koch wrote:
>>> On 03/17/13 13:00, Klaus Schmidinger wrote:
>>>> On 17.03.2013 12:46, Ville Skyttä wrote:
>>>>> Hello,
>>>>>
>>>>> Should probably have spoken earlier, but is there any particular reason
>>>>> for the ugly and hard to read Makefile variable names LCLBLD and PLGCFG?
>>>>> I suppose they're short for LOCALBUILD and PLUGINCONFIG, but why do they
>>>>> have to be short for anything, it's not like we're running out of space
>>>>> for their names anywhere, is it? Besides, I don't think LCLBLD describes
>>>>> what its effects are very well, INPLACE would sounds better to me.
>>>>>
>>>>> The ship may already have sailed for plgcfg in vdr.pc as it's being used
>>>>> by many plugin Makefiles already, but I believe the attached patches
>>>>> should be safe for 2.0.0.
>>>>
>>>> I'm afraid it's too late for that.
>>>
>>> I do not understand why it should be too late.
>>> 2.0 is not out yet.
>>
>> But we're in the final testing phase.
>> Besides, Ville's patch also touched the Makefiles of plugins - and
>> plugin authors and distribution managers react very intense when they
>> have to modify their files ;-).
>
> If I am not mistaken, Ville’s issue was that the names were not clear.
> Could a comment be added to the Makefile what the meaning is then. That
> should not break anything.
Something like this?
# Use 'make LCLBLD=1' to build locale and plugin files under the source directory:
ifdef LCLBLD
...
# Use this if you want to have a central place where you configure compile time
# parameters for plugins:
#PLGCFG = $(CONFDIR)/plugins.mk
Klaus
From 79a900a9995c1d401543e2804027cc019b5f7375 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ville=20Skytt=C3=A4?= <ville.skytta@iki.fi>
Date: Sun, 17 Mar 2013 13:43:40 +0200
Subject: [PATCH 2/2] Rename PLGCFG to PLUGINCONFIG.
---
Make.config.template | 2 +-
Makefile | 2 +-
PLUGINS/src/dvbhddevice/Makefile | 4 ++--
PLUGINS/src/dvbsddevice/Makefile | 4 ++--
PLUGINS/src/epgtableid0/Makefile | 4 ++--
PLUGINS/src/hello/Makefile | 4 ++--
PLUGINS/src/osddemo/Makefile | 4 ++--
PLUGINS/src/pictures/Makefile | 4 ++--
PLUGINS/src/rcu/Makefile | 4 ++--
PLUGINS/src/servicedemo/Makefile | 4 ++--
PLUGINS/src/skincurses/Makefile | 4 ++--
PLUGINS/src/status/Makefile | 4 ++--
PLUGINS/src/svdrpdemo/Makefile | 4 ++--
newplugin | 4 ++--
14 files changed, 26 insertions(+), 26 deletions(-)
@@ -62,7 +62,7 @@ endif
# Use this if you want to have a central place where you configure compile time
# parameters for plugins:
-#PLGCFG = $(CONFDIR)/plugins.mk
+#PLUGINCONFIG = $(CONFDIR)/plugins.mk
### The remote control:
@@ -147,7 +147,7 @@ vdr.pc:
@echo "resdir=$(RESDIR)" >> $@
@echo "libdir=$(LIBDIR)" >> $@
@echo "locdir=$(LOCDIR)" >> $@
- @echo "plgcfg=$(PLGCFG)" >> $@
+ @echo "plgcfg=$(PLUGINCONFIG)" >> $@
@echo "apiversion=$(APIVERSION)" >> $@
@echo "cflags=$(CFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
@echo "cxxflags=$(CXXFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
LOCDIR = $(call PKGCFG,locdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -19,7 +19,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
LOCDIR = $(call PKGCFG,locdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -34,7 +34,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -19,7 +19,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
LOCDIR = $(call PKGCFG,locdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -34,7 +34,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -19,7 +19,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN1).c | awk '{ pr
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -34,7 +34,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -19,7 +19,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
LOCDIR = $(call PKGCFG,locdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -34,7 +34,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -18,7 +18,7 @@ VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).c | awk '{ pri
# Use package data if installed...otherwise assume we're under the VDR source directory:
PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))
LIBDIR = $(call PKGCFG,libdir)
-PLGCFG = $(call PKGCFG,plgcfg)
+PLUGINCONFIG = $(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -33,7 +33,7 @@ APIVERSION = $(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include $(PLGCFG)
+-include $(PLUGINCONFIG)
### The name of the distribution archive:
@@ -78,7 +78,7 @@ VERSION = \$(shell grep 'static const char \\*VERSION *=' \$(PLUGIN).c | awk '{
PKGCFG = \$(if \$(VDRDIR),\$(shell pkg-config --variable=\$(1) \$(VDRDIR)/vdr.pc),\$(shell pkg-config --variable=\$(1) vdr || pkg-config --variable=\$(1) ../../../vdr.pc))
LIBDIR = \$(call PKGCFG,libdir)
LOCDIR = \$(call PKGCFG,locdir)
-PLGCFG = \$(call PKGCFG,plgcfg)
+PLUGINCONFIG = \$(call PKGCFG,plgcfg)
#
TMPDIR ?= /tmp
@@ -93,7 +93,7 @@ APIVERSION = \$(call PKGCFG,apiversion)
### Allow user defined options to overwrite defaults:
--include \$(PLGCFG)
+-include \$(PLUGINCONFIG)
### The name of the distribution archive:
--
1.7.11.7