Feature #453
openvdr-addon-seriestimer (VDRSeriestimer.pl)
0%
Description
ich hab das mal zusammen mit steffen_b versucht zu verpacken.
http://ppa.launchpad.net/yavdr/unstable-vdr/ubuntu/pool/main/v/vdr-addon-seriestimer/
was könnte noch fehlen ?:
ich muss noch /etc/cron.daily/vdrseriestimer mit ins paket aufnehmen :
#!/bin/sh
# Set LISTSDIR to your episodes lists dir!
LISTSDIR=/var/cache/eplists/episodes/
[ -z "$LISTSDIR" ] && (echo "Set ListsDir!"; exit 1; )
# Use TGET (since|newer than) x (minutes|hours|days|weeks|months|years).
svdrpsend-ng.pl -d eplists.constabel.net -p 2006 -c -o "${LISTSDIR}" TGET newer than 25 hours
/etc/eplists
muss noch irgendwie aufgeräumt werden:
cron.d-eplists.example vdrseriestimer.conf cron.daily-eplists.sh.example epgsearch-vdrseriestimer.conf.example vdrseriestimer.conf.example series.conf.example
examples nach usr/share/doc/vdr-addon-seriestimer/examples
seriestimer braucht noch zusätzlich ein paket :
http://ppa.launchpad.net/yavdr/unstable-vdr/ubuntu/pool/main/libt/libtext-levenshteinxs-perl/
Files
Updated by hoplo about 14 years ago
so wie es ist läuft es bei mir, wie gesagt nur ein voreschlag.
evtl. besteht ja interesse !
Updated by hoplo about 14 years ago
da ich weit weg davon bin alles zu verstehen, mach ich die schritte hier nach und nach.
wäre ganz nett wenn einer der paketbauprofis da ab und zu dazwischengehen könnte, sollte ihm was falsches auffallen.
da ihr eine "umstellung" auf debhelper7 plant ... (ich dachte immer cdbs wäre der debhekper killer ..)
hab ich hier mal dir rules geändert
#!/usr/bin/make -f
export DH_VERBOSE=1
%:
dh $@
override_dh_install:
dh_install
mv debian/vdr-addon-seriestimer/usr/bin/VDRSeriesTimer.pl debian/vdr-addon-seriestimer/usr/bin/vdrseriestimer
eigentlich hab ich hierzu zwei fragen
- ist es richtig perl files sollen/müssen ohne dateiendung installiert werden weil eben "shebang" ausreicht ?
es geht mir nur darum es in zukunft "besser" zu machen.
- ich will die etc/eplists/*example aus den conffiles raushaben.
das hier funktioniert nicht, und ich hab keine ahnung warum nicht:
override_dh_install:
dh_install -X*example
Updated by etobi about 14 years ago
hoplo wrote:
- ist es richtig perl files sollen/müssen ohne dateiendung installiert werden weil eben "shebang" ausreicht ?
Ja "executables" sollten generell keine Dateiendung haben. Das sollte für den User vollkommen transparent sein. So kann man z.B. auch mal ein Perl-Skript durch ein Python-Skript ersetzen o.ä.
- ich will die etc/eplists/*example aus den conffiles raushaben.
das hier funktioniert nicht, und ich hab keine ahnung warum nicht:
Das funktioniert nur für Dateien, die auch mit dh_install installiert werden. dh_auto_install installiert die Dateien aber schon zuvor via `make install`.
Funktionieren würde z.B. das:
DESTDIR=$(CURDIR)/debian/vdr-addon-seriestimer
override_dh_auto_install:
dh_auto_install
find $(DESTDIR) -name "*.example" -exec rm {} \;
mv -v $(DESTDIR)/usr/bin/VDRSeriesTimer.pl $(DESTDIR)/usr/bin/vdrseriestimer
Updated by hoplo about 14 years ago
danke ! und ich such mir ewig einen ab ....
ich hab jetzt noch ein paket hochgeladen :
http://ppa.launchpad.net/yavdr/unstable-vdr/ubuntu/pool/main/v/vdr-addon-seriestimer/vdr-addon-seriestimer_0.3.0~beta6-1yavdr4.dsc
geändert hab ich rules wie oben beschrieben.
#!/usr/bin/make -f
export DH_VERBOSE=1
%:
dh $@
DESTDIR=$(CURDIR)/debian/vdr-addon-seriestimer
override_dh_auto_install:
dh_auto_install
find $(DESTDIR) -name "*.example" -exec rm {} \;
mv -v $(DESTDIR)/usr/bin/VDRSeriesTimer.pl $(DESTDIR)/usr/bin/vdrseriestimer
mv -v $(DESTDIR)/usr/bin/svdrpsend-ng.pl $(DESTDIR)/usr/bin/svdrpsend-ng
mv -v $(DESTDIR)/usr/bin/VDREplistChecker.pl $(DESTDIR)/usr/bin/VDREplistChecker
im makefile hab ich die doc-installation auskommentiert sonst hätte es das 2mal gegeben
usr/share/doc/eplists (vom makefile)
usr/share/doc/vdr-addon-seriestimer (vom paket)
die patches hab ich umbenannt, damit das etwas sauberer aussieht :-)
die examples aus etc hab ich in ein debian/examples gesteckt
und ein debian/cron.daily gibt es jetzt ....
Updated by etobi about 14 years ago
Ein paar Dinge sollten noch gefixt werden:
- Es gibt noch eine Hand voll Lintian-Warnungen und einen Fehler:
W: vdr-addon-seriestimer source: debhelper-overrides-need-versioned-build-depends (>= 7.0.50~) W: vdr-addon-seriestimer source: out-of-date-standards-version 3.8.4 (current is 3.9.1) E: vdr-addon-seriestimer: description-synopsis-is-duplicated W: vdr-addon-seriestimer: manpage-section-mismatch usr/share/man/man1/VDRSeriesTimer.pl.1.gz:132 1 != 5 W: vdr-addon-seriestimer: manpage-section-mismatch usr/share/man/man1/svdrpsend-ng.pl.1.gz:132 1 != 5 W: vdr-addon-seriestimer: binary-without-manpage usr/bin/VDREplistChecker W: vdr-addon-seriestimer: binary-without-manpage usr/bin/svdrpsend-ng W: vdr-addon-seriestimer: binary-without-manpage usr/bin/vdrseriestimer W: vdr-addon-seriestimer: executable-not-elf-or-script ./usr/share/doc/vdr-addon-seriestimer/examples/cron.d-eplists.example W: vdr-addon-seriestimer: extra-license-file usr/share/doc/vdr-addon-seriestimer/COPYING.gz W: vdr-addon-seriestimer: package-contains-upstream-install-documentation usr/share/doc/vdr-addon-seriestimer/INSTALL.gz W: vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/Eplists.pm W: vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/EplistsClient.pm
binary-without-manpage lass ich noch durchgehen solang es nicht ein offizielles Paket wird, aber der Rest kann gefixt werden. - Die cdbs-Build-dependency kann rausfliegen
- tab/space-Mix in postinst - besser nur spaces
- links + install - spaces statt tabs (ist lediglich meine persönliuche präferenz)
- Evtl. zusätzlich die zweite Kontaktadresse debian/copyyright
- shlibs:Depends und vdr:Depends sind überflüssig
- wget fehlt in den Depends (wird in postinst verwendet)
- wget in postinst kann fehlschlagen - warum muss das überhaupt runtergeladen werden?
Updated by hoplo about 14 years ago
autsch
jetzt verstehe ich ... ich sollte den ausgaben bei debuild und lintian nicht trauen...
das reicht nicht :-)
Now running lintian... W: vdr-addon-seriestimer source: newer-standards-version 3.9.1 (current is 3.8.4) Finished running lintian.
ich versuch das jetzt mal weiter .... wobei ich denke der error ist noch das leichte dabei.
in zukunft hab ich es kapiert, lintian kann man nicht veräppeln.
lintian vdr-addon-seriestimer_0.3.0~beta6-1yavdr4_all.deb
E: vdr-addon-seriestimer: description-synopsis-is-duplicated W: vdr-addon-seriestimer: extra-license-file usr/share/doc/vdr-addon-seriestimer/COPYING.gz W: vdr-addon-seriestimer: package-contains-upstream-install-documentation usr/share/doc/vdr-addon-seriestimer/INSTALL.gz W: vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/Eplists.pm W: vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/EplistsClient.pm W: vdr-addon-seriestimer: manpage-section-mismatch usr/share/man/man1/VDRSeriesTimer.pl.1.gz:132 1 != 5 W: vdr-addon-seriestimer: manpage-section-mismatch usr/share/man/man1/svdrpsend-ng.pl.1.gz:132 1 != 5 W: vdr-addon-seriestimer: binary-without-manpage usr/bin/VDREplistChecker W: vdr-addon-seriestimer: binary-without-manpage usr/bin/svdrpsend-ng W: vdr-addon-seriestimer: binary-without-manpage usr/bin/vdrseriestimer W: vdr-addon-seriestimer: executable-not-elf-or-script ./usr/share/doc/vdr-addon-seriestimer/examples/cron.d-eplists.example
aber das ist es halt, wenn man es nicht kennt, dann macht man es nicht.
der rest mit "description", cdbs, wget usw. ist schlamperei von mir.
vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/Eplists.pm
ufff ich glaube da brauch ich hilfe von google ....
aber danke ! jede kleine hilfe von dir und tomg heisst aber auch, in zukunft mach ich es gleich "besser".
Updated by etobi about 14 years ago
hoplo wrote:
vdr-addon-seriestimer: package-installs-nonbinary-perl-in-usr-lib-perl5 usr/lib/perl5/Eplists.pm
ufff ich glaube da brauch ich hilfe von google ....
Du kannst lintian auch mit "-i" aufrufen. Dann wird dir zu jeder Meldung noch ein kleiner Hinweis gegeben. Obige Meldung weist dich nur darauf hin, dass /usr/lib/perl5 eigentlich für Binärmodule reserviert ist. Der Rest gehört nach /usr/share/hab-ich-jetzt-grad-nicht-im-kopf
Updated by hoplo about 14 years ago
ok wieder einen schritt weiter.
W: vdr-addon-seriestimer: binary-without-manpage usr/bin/VDREplistChecker
bleibt leider.
aber evtl. können wir (steffen und ich) da auch noch was "bauen".
wegen :
manpage-section-mismatch usr/share/man/man1/VDRSeriesTimer.pl.1.gz:132 1 != 5
und wegen noch ein paar dingen sollte ich wohl den autor mal anschreiben. :-)
mir ist da eh nicht klar ob das "richtig" ist, das als .gz in den sourcen zu haben ?
aber wieder etwas wo ich mich (noch) nicht auskenne ...
wget in post-install ... da bin ich etwas überfragt, wie man das "elegant" wegbekommt.
das ganze nach cron.daily ?
Updated by etobi about 14 years ago
- File changes.diff changes.diff added
hoplo wrote:
Anbei ein paar kleine Änderungen.
mir ist da eh nicht klar ob das "richtig" ist, das als .gz in den sourcen zu haben ?
aber wieder etwas wo ich mich (noch) nicht auskenne ...
Aus dem Bauch raus würde ich sagen, das passt schon.
wget in post-install ... da bin ich etwas überfragt, wie man das "elegant" wegbekommt.
das ganze nach cron.daily ?
Ist das Runterladen denn überhaupt nötig? Macht das das Addon nicht sowieso bei Bedarf? Auf jeden Fall sollte die Installation auf einem Offline-System dadurch nicht komplett fehlschlagen.
Ein andere Sache, die mir noch aufgefallen ist: /var/log/vdrseriestimer.log - ohne das jetzt getestet zu haben, aber AFAIK wird das vom VDR also i.d.R. mit dem User "vdr" aufgerufen. Der kann standardmäßig nicht nach /var/log schreiben. Also ggf. UV anlegen und vdr Schreibzugriff geben.
Updated by hoplo about 14 years ago
danke für deine hilfe !
solltest du das addon aufnehmen wollen dann das hier nicht "vergessen":
das gab es mal als offizielles paket in debian. warum es das jetzt nicht mehr gibt weiss ich nicht.
ich schau mir das nochmal an, aber "Architecture : any" war denke ich schon falsch.
ich hab das bei uns auf "all" geändert. da es ein test in unstable war wird man dazu aber nichts in changelog sehen
(mea culpa).