Kompilieren von xmltv2vdr schlägt fehl

M-Reimer · 23. Dezember 2017

Hallo,

mit aktuellem gcc bekomme ich folgenden Fehler, der dann eine Reihe von Folgefehlern nach sich zieht:

Code

In file included from /usr/include/unicode/utypes.h:38:0,
from /usr/include/unicode/ucnv_err.h:88,
from /usr/include/unicode/ucnv.h:52,
from /usr/include/libxml2/libxml/encoding.h:31,
from /usr/include/libxml2/libxml/parser.h:810,
from parse.h:12,
from source.h:15,
from import.h:16,
from soundex.cpp:1:
/usr/include/unicode/umachine.h:347:13: error: 'char16_t' does not name a type; did you mean 'wchar_t'?
typedef char16_t UChar;
^~~~~~~~
wchar_t

Alles anzeigen

Wie kann man das reparieren?

fnu · 23. Dezember 2017

Was ist bei Dir der aktuelle gcc?

Hast Du den letzten GIT Stand von xmltv2vdr (20.03.2017)?

wolfi.m · 23. Dezember 2017

Hi,

M-Reimer schrieb:

mit aktuellem gcc bekomme ich folgenden Fehler

mit gcc-7(7.2.0) unter Ubuntu 18.04(bionic) läuft das Build durch - GIT xmltv2vdr commit: Changed code for c++11 compatibility

...Arch ist wohl Version 7.2.1

Gruss

Wolfgang

fnu · 23. Dezember 2017

wolfi.m schrieb:

mit gcc-7(7.2.0) unter Ubuntu 18.04(bionic) läuft das Build durch

Ja, eben ...

seahawk1986 · 24. Dezember 2017

Ich denke da spielt dieser Patch für das Makefile mit hinein, der gnu++98 als C++ Standard setzt: https://github.com/VDR4Arch/vd…r_newmakefile_v2.diff#L78

Wenn man das auf -std=c++11 setzt, hängt es an anderen Stellen - so baut es zumindest (vielleicht kann da noch jemand drüber schauen, der C++ besser beherrscht und das Plugin tatsächlich nutzt):

Diff

diff --git a/Makefile b/Makefile
index aa344a4..10ff426 100644
--- a/Makefile
+++ b/Makefile
@@ -1,60 +1,57 @@
#
# Makefile for a Video Disk Recorder plugin
#
-# $Id$
+# $Id: Makefile 2.18 2013/01/12 13:45:01 kls Exp $
# The official name of this plugin.
# This name will be used in the '-P...' option of VDR to load the plugin.
# By default the main source file also carries this name.
-# IMPORTANT: the presence of this macro is important for the Make.config
-# file. So it must be defined, even if it is not used here!
-#
+
PLUGIN = xmltv2vdr
### The version number of this plugin (taken from the main source file):
VERSION = $(shell grep 'static const char \*VERSION *=' $(PLUGIN).h | awk '{ print $$6 }' | sed -e 's/[";]//g')
-### The C++ compiler and options:
-
-CXX ?= g++
-CXXFLAGS ?= -fPIC -g -O2 -Wall -Wextra -Woverloaded-virtual -Wno-parentheses
-PKG-CONFIG ?= pkg-config
-
### The directory environment:
-VDRDIR = ../../..
-LIBDIR = ../../lib
-TMPDIR = /tmp
+# 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)
+LOCDIR = $(call PKGCFG,locdir)
+PLGCFG = $(call PKGCFG,plgcfg)
+#
+TMPDIR ?= /tmp
-### Make sure that necessary options are included:
+### The compiler options:
--include $(VDRDIR)/Make.global
+export CFLAGS = $(call PKGCFG,cflags)
+export CXXFLAGS = $(call PKGCFG,cxxflags)
-### Allow user defined options to overwrite defaults:
+### The version number of VDR's plugin API:
--include $(VDRDIR)/Make.config
+APIVERSION = $(call PKGCFG,apiversion)
-### The version number of VDR's plugin API (taken from VDR's "config.h"):
+### Allow user defined options to overwrite defaults:
-APIVERSION = $(shell sed -ne '/define APIVERSION/s/^.*"$.*$".*$$/\1/p' $(VDRDIR)/config.h)
+-include $(PLGCFG)
### The name of the distribution archive:
ARCHIVE = $(PLUGIN)-$(VERSION)
PACKAGE = vdr-$(ARCHIVE)
-### Includes and Defines (add further entries here):
+### The name of the shared object file:
+
+SOFILE = libvdr-$(PLUGIN).so
-PKG-LIBS += libxml-2.0 libpcrecpp sqlite3
-PKG-INCLUDES += libxml-2.0 libpcrecpp sqlite3
+### Includes and Defines (add further entries here):
-INCLUDES += -I$(VDRDIR)/include
+CXXFLAGS += $(shell pkg-config --cflags libxml-2.0 libpcrecpp sqlite3) -std=c++11
-DEFINES += -D_GNU_SOURCE -D_XOPEN_SOURCE -DPLUGIN_NAME_I18N='"$(PLUGIN)"'
+LIBS += $(shell pkg-config --libs libxml-2.0 libpcrecpp sqlite3)
-INCLUDES += $(shell $(PKG-CONFIG) --cflags $(PKG-INCLUDES))
-LIBS += $(shell $(PKG-CONFIG) --libs $(PKG-LIBS))
+DEFINES += -DPLUGIN_NAME_I18N='"$(PLUGIN)"'
### The object files (add further files here):
@@ -62,12 +59,12 @@ OBJS = $(PLUGIN).o soundex.o extpipe.o parse.o source.o import.o event.o setup.o
### The main target:
-all: libvdr-$(PLUGIN).so i18n
+all: $(SOFILE) i18n
### Implicit rules:
%.o: %.cpp
- $(CXX) $(CXXFLAGS) -c $(DEFINES) $(INCLUDES) $<
+ $(CXX) $(CXXFLAGS) -c $(DEFINES) $(INCLUDES) -o $@ $<
### Dependencies:
@@ -81,47 +78,47 @@ $(DEPFILE): Makefile
### 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))))))
+I18Nmo = $(addsuffix .mo, $(foreach file, $(I18Npo), $(basename $(file))))
+I18Nmsgs = $(addprefix $(DESTDIR)$(LOCDIR)/, $(addsuffix /LC_MESSAGES/vdr-$(PLUGIN).mo, $(notdir $(foreach file, $(I18Npo), $(basename $(file))))))
I18Npot = $(PODIR)/$(PLUGIN).pot
%.mo: %.po
msgfmt -c -o $@ $<
-$(I18Npot): $(wildcard *.cpp *.h)
- xgettext -C -cTRANSLATORS --no-wrap --no-location -k -ktr -ktrNOOP --msgid-bugs-address='<see README>' -o $@ $^
+$(I18Npot): $(wildcard *.cpp)
+ xgettext -C -cTRANSLATORS --no-wrap --no-location -k -ktr -ktrNOOP --package-name=vdr-$(PLUGIN) --package-version=$(VERSION) --msgid-bugs-address='<see README>' -o $@ `ls $^`
%.po: $(I18Npot)
- msgmerge -U --no-wrap --no-location --backup=none -q $@ $<
+ msgmerge -U --no-wrap --no-location --backup=none -q -N $@ $<
@touch $@
-$(I18Nmsgs): $(LOCALEDIR)/%/LC_MESSAGES/vdr-$(PLUGIN).mo: $(PODIR)/%.mo
- @mkdir -p $(dir $@)
- cp $< $@
+$(I18Nmsgs): $(DESTDIR)$(LOCDIR)/%/LC_MESSAGES/vdr-$(PLUGIN).mo: $(PODIR)/%.mo
+ install -D -m644 $< $@
.PHONY: i18n
-i18n: $(I18Nmsgs) $(I18Npot)
+i18n: $(I18Nmo) $(I18Npot)
+
+install-i18n: $(I18Nmsgs)
### Targets:
-libvdr-$(PLUGIN).so: $(OBJS)
+$(SOFILE): $(OBJS)
$(CXX) $(CXXFLAGS) -shared $(OBJS) $(LIBS) -o $@
- @cp --remove-destination $@ $(LIBDIR)/$@.$(APIVERSION)
-install:
+install-lib: $(SOFILE)
+ install -D $^ $(DESTDIR)$(LIBDIR)/$^.$(APIVERSION)
+
+install: install-lib install-i18n
-dist: clean
+dist: $(I18Npo) clean
@-rm -rf $(TMPDIR)/$(ARCHIVE)
@mkdir $(TMPDIR)/$(ARCHIVE)
- @cp -a *.cpp *.h HISTORY COPYING Makefile README po $(TMPDIR)/$(ARCHIVE)
- @mkdir -p $(TMPDIR)/$(ARCHIVE)/dist/epgdata2xmltv
- @cp -a dist/epgdata2xmltv/*.cpp dist/epgdata2xmltv/*.h dist/epgdata2xmltv/Makefile dist/epgdata2xmltv/INSTALL dist/epgdata2xmltv/COPYING dist/epgdata2xmltv/epgdata2xmltv.dist dist/epgdata2xmltv/epgdata2xmltv.xsl $(TMPDIR)/$(ARCHIVE)/dist/epgdata2xmltv
- @mkdir -p $(TMPDIR)/$(ARCHIVE)/dist/patches
- @cp -a dist/patches/* $(TMPDIR)/$(ARCHIVE)/dist/patches
- @tar czf $(PACKAGE).tgz -C $(TMPDIR) --exclude debian --exclude CVS --exclude .svn $(ARCHIVE)
+ @cp -a * $(TMPDIR)/$(ARCHIVE)
+ @tar czf $(PACKAGE).tgz -C $(TMPDIR) $(ARCHIVE)
@-rm -rf $(TMPDIR)/$(ARCHIVE)
@echo Distribution package created as $(PACKAGE).tgz
clean:
- @-rm -f $(OBJS) $(DEPFILE) *.so *.tgz core* *~ $(PODIR)/*.mo $(PODIR)/*.pot
+ @-rm -f $(PODIR)/*.mo $(PODIR)/*.pot
+ @-rm -f $(OBJS) $(DEPFILE) *.so *.tgz core* *~
diff --git a/dist/epgdata2xmltv/Makefile b/dist/epgdata2xmltv/Makefile
index 3301bd2..75ab3c9 100644
--- a/dist/epgdata2xmltv/Makefile
+++ b/dist/epgdata2xmltv/Makefile
@@ -67,9 +67,8 @@ install: epgdata2xmltv
@mkdir -p $(INSTALL)
@install $(STRIP) epgdata2xmltv $(INSTALL)
@mkdir -p $(DESTDIR)/var/lib/epgsources
- @mkdir -p /var/lib/epgsources/epgdata2xmltv-img
- @cp epgdata2xmltv.dist /var/lib/epgsources/epgdata2xmltv
- @chown -R vdr.vdr /var/lib/epgsources
+ @mkdir -p $(DESTDIR)/var/lib/epgsources/epgdata2xmltv-img
+ @cp epgdata2xmltv.dist $(DESTDIR)/var/lib/epgsources/epgdata2xmltv
dist: clean
@-rm -rf $(TMPDIR)/epgdata2xmltv
diff --git a/event.cpp b/event.cpp
index ae33002..1df43ec 100644
--- a/event.cpp
+++ b/event.cpp
@@ -528,7 +528,7 @@ void cXMLTVEvent::GetSQL(const char *Source, int SrcIdx, const char *ChannelID,
return;
}
- string si=sql_insert;
+ std::string si=sql_insert;
int ireps;
ireps=pcrecpp::RE("'").GlobalReplace("''",&si);
ireps+=pcrecpp::RE("\\^").GlobalReplace("'",&si);
@@ -540,7 +540,7 @@ void cXMLTVEvent::GetSQL(const char *Source, int SrcIdx, const char *ChannelID,
}
*Insert=sql_insert;
- string su=sql_update;
+ std::string su=sql_update;
int ureps;
ureps=pcrecpp::RE("'").GlobalReplace("''",&su);
ureps+=pcrecpp::RE("\\^").GlobalReplace("'",&su);
diff --git a/import.cpp b/import.cpp
index 4fcf6a3..1064764 100644
--- a/import.cpp
+++ b/import.cpp
@@ -250,7 +250,7 @@ char *cImport::Add2Description(char *description, const char *Name, const char *
char *cImport::Add2Description(char *description, const char *Name, int Value)
{
char *value=NULL;
- if (asprintf(&value,"%i",Value)==-1) return false;
+ if (asprintf(&value,"%i",Value)==-1) return NULL;
description = strcatrealloc(description,Name);
description = strcatrealloc(description,": ");
description = strcatrealloc(description,value);
@@ -261,7 +261,7 @@ char *cImport::Add2Description(char *description, const char *Name, int Value)
char *cImport::AddEOT2Description(char *description, bool checkutf8)
{
- const char nbspUTF8[]={0xc2,0xa0,0};
+ const char nbspUTF8[]={(char)(0xc2),(char)(0xa0),(char)(0)};
if (checkutf8)
{
@@ -277,7 +277,7 @@ char *cImport::AddEOT2Description(char *description, bool checkutf8)
}
else
{
- const char nbsp[]={0xa0,0};
+ const char nbsp[]={(char)(0xa0),(char)(0)};
description=strcatrealloc(description,nbsp);
}
}
@@ -1401,7 +1401,7 @@ bool cImport::UpdateXMLTVEvent(cEPGSource *Source, sqlite3 *Db, cXMLTVEvent *xEv
return false;
}
- string ed=shortdesc;
+ std::string ed=shortdesc;
int reps;
reps=pcrecpp::RE("'").GlobalReplace("''",&ed);
@@ -1511,7 +1511,7 @@ bool cImport::UpdateXMLTVEvent(cEPGSource *Source, sqlite3 *Db, const cEvent *Ev
return false;
}
- string ed=eitdescription;
+ std::string ed=eitdescription;
int reps;
reps=pcrecpp::RE("'").GlobalReplace("''",&ed);
@@ -1649,7 +1649,7 @@ cXMLTVEvent *cImport::SearchXMLTVEvent(sqlite3 **Db,const char *ChannelID, const
return NULL;
}
- string st=sqltitle;
+ std::string st=sqltitle;
int reps;
reps=pcrecpp::RE("'").GlobalReplace("''",&st);

Alles anzeigen

M-Reimer · 24. Dezember 2017

Ich habe den genannten Patch jetzt auch schonmal ganz rausgelassen, komme aber auf genau die von seahawk1986 genannten Fehler.

Gibt es denn irgendeinen C++-Standard mit dem es ganz ohne "Fixen" wieder kompiliert? Eigentlich hält sich mein Interesse daran, fremden Code ganz zu machen, in Grenzen. Ich selber nutze das Plugin zudem auch nicht. Brauche es nur für den VDR von einem Bekannten.

M-Reimer · 24. Dezember 2017

seahawk1986 Kannst du mir erklären warum du beim Type-Casting zwischen "unsigned" und "signed" variierst? Wäre nicht "unsigned" generell OK?

Allgemeine Frage: Warum braucht man den Mist überhaupt? Der Compiler kann doch garnicht so doof sein nicht zu wissen, dass er das, was ich ins Array direkt zuweisen will, natürlich in dem Format ist, das das Array selbst hat. Wozu soll ich z.B. bei einem Array dem ich 100 Elemente zuweise jetzt 100 mal eine Zahl nach char casten?

M-Reimer · 24. Dezember 2017

Nachtrag: Läuft jetzt. Letztlich haben tatsächlich nur die "std::string" gefehlt. Falls es jemand für eine andere Distribution braucht:

https://github.com/VDR4Arch/vd…ltv2vdr_gcc721_fixes.diff

Jetzt kompiliert das ganze vdr4arch-Repo wieder. Fehlt nun lediglich vdr-loadepg und da weiß ich nichtmal für was das gut sein soll. Wenn es jemand braucht soll er einen Patch schicken.

fnu · 24. Dezember 2017

Sind die Makefile Anpassungen ArchLinux spezifisch?

Wenn nein, könntet Ihr ja mal ein bzw. beide Patches über vdr-developer einreichen ... ist nicht so, das Jochen gar nimmer antwortet, sonst gäbe es z.B. den letzten Commit auch nicht ...

Regards

fnu

M-Reimer · 24. Dezember 2017

Die Makefile-Anpassungen sind von Copperhead und ziehen das alte Makefile auf den neuen Stand, der auch bei VDR 2.2 schon Standard sein sollte. Damit ist das Paketieren eleganter.

Im Prinzip könnte man den auch ganz weglassen und das PKGBUILD entsprechend umfummeln. Aber wegen "never change a running system" habe ich ihn doch dringelassen.

Nachtrag:

Bug #2543 Build errors with GCC 7.2.1

Feature #2542 Upgrade the Makefile to the new VDR plugin Makefile standard

seahawk1986 · 24. Dezember 2017

M-Reimer schrieb:

seahawk1986 Kannst du mir erklären warum du beim Type-Casting zwischen "unsigned" und "signed" variierst? Wäre nicht "unsigned" generell OK?

Ist das nicht Implementationsabhängig, ob ein char signed oder unsigned ist (http://www.cplusplus.com/reference/climits/)? Aber du hast Recht, ein unsigned char sollte vom Wertebereich besser passen.

seahawk1986 · 24. Dezember 2017

M-Reimer schrieb:

Allgemeine Frage: Warum braucht man den Mist überhaupt? Der Compiler kann doch garnicht so doof sein nicht zu wissen, dass er das, was ich ins Array direkt zuweisen will, natürlich in dem Format ist, das das Array selbst hat. Wozu soll ich z.B. bei einem Array dem ich 100 Elemente zuweise jetzt 100 mal eine Zahl nach char casten?

Da ein char nicht den Wertebereich eines int abdeckt, könnte es bei einer Umwandlung dazu kommen, dass der Wert verfälscht wird. Implizite Umwandlungen sind nur dann unproblematisch, wenn garantiert ist, dass der Wertebereich vom Zieltyp abgedeckt wird: http://en.cppreference.com/w/c…guage/implicit_conversion

Bei einem char-Array kann man auch die chars escaped in einem String setzen, wenn man keine Umwandlung von int nach char will:

C

#include <iostream>
#include <cstring>
int main() {
const char nbspUTF8[]={(char)(0xc2),(char)(0xa0),(char)(0)};
const char r[] = "\xc2\xa0"; // \0 is appended automatically
if (std::strcmp(nbspUTF8, r) == 0) {
std::cout << "char arrays are equal" << std::endl;
} else {
std::cout << "char arrays are not equal" << std::endl;
}
return 0;
}

Alles anzeigen

Teilen