c't vdr vs debian vdr (Patchlevel)

Tobias Grimm pkg-vdr-dvb-devel@lists.alioth.debian.org
Tue, 22 Jun 2004 21:46:09 +0200


This is a multi-part message in MIME format.
--------------050206060001000606060709
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Hallo!

Thomas Schmidt wrote:

>Also ich denke, die eine Zeile sollte eigentlich niemanden stören,
>solange der build-prozess nicht abbricht, falls z.B.
>/usr/include/vdr/patchlevel nicht existiert oder so.
>  
>
Also ich hab' heute mal noch ein bischen drüber nachgedacht. Bisher ist 
es so, dass der Patchlevel in die Beschreibung des Paketes mit 
eingetragen wird. Besser wäre evtl, hierfür ein neues custom field zu 
kreieren:

XB-VDR-patchlevel: ${patchlevel}

(siehe beiliegendes diff)

Das hätte den Vorteil, dass dieses Feld im Binärpaket garnicht 
auftaucht, wenn ${patchlevel} leer ist.

Eine andere Variante wäre, auch in c't vdr ganz auf die 
Patchlevel-Prüfung zu verzichten. Wenn jemand dann mit einem gepatchten 
vdr ein nicht binär-kompatibles Plugin laden will, hat er halt Pech gehabt.

Wichtige fände ich halt nur, dass die Plugin-Source-Pakete in c't vdr 
und Debian letztendlich am besten identisch sind und evtl nur noch auf 
Alioth gepflegt werden.

Peter, was meinst du?

bye,

Tobias

--------------050206060001000606060709
Content-Type: text/plain;
 name="patchlevel.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="patchlevel.diff"

Index: debian/control
===================================================================
RCS file: /cvsroot/pkg-vdr-dvb/vdr/vdr-plugin-femon/debian/control,v
retrieving revision 1.3
diff -r1.3 control
13a14
> XB-VDR-Patchlevel: ${patchlevel}
Index: debian/rules
===================================================================
RCS file: /cvsroot/pkg-vdr-dvb/vdr/vdr-plugin-femon/debian/rules,v
retrieving revision 1.2
diff -r1.2 rules
17a18,22
> VDR=vdr
> PACKAGE=$(VDR)-plugin-femon
> VDRDIR=/usr/include/$(VDR)
> DVBDIR=/usr/include
> 
30c35
< 	$(MAKE) all DVBDIR=/usr/include VDRDIR=/usr/include/vdr LIBDIR=. REMOTE=lirc
---
> 	$(MAKE) all DVBDIR=$(DVBDIR) VDRDIR=$(VDRDIR) LIBDIR=.
38d42
< 	rm -f debian/vdr-plugin-femon.substvars
71a76,84
> 
> # VDR can be customized with quite a lot of different patches, which can make
> # the plugins binary incompatible.
> # Although Debian will not provide different binary packages for VDR, we
> # will include a patchlevel info, which can be checked before a plugin is 
> # loaded.
> 	[ -r $(VDRDIR)/patchlevel ] || echo 'patchlevel=' >> debian/$(PACKAGE).substvars
> 	[ ! -r $(VDRDIR)/patchlevel ] || cat $(VDRDIR)/patchlevel >> debian/$(PACKAGE).substvars
> 

--------------050206060001000606060709--