Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation von vzlogger schlägt fehl #295

Closed
UdoSchake opened this issue Feb 12, 2017 · 14 comments
Closed

Installation von vzlogger schlägt fehl #295

UdoSchake opened this issue Feb 12, 2017 · 14 comments
Assignees

Comments

@UdoSchake
Copy link

UdoSchake commented Feb 12, 2017

Bei make kommen folgende Fehler:
1.
/vzlogger/src/api/Volkszaehler.cpp: In member function
‘virtual void vz::api::Volkszaehler::send()’:
2.
/vzlogger/src/api/Volkszaehler.cpp:149:25: error: comparison between
signed and unsigned integer expressions [-Werror=sign-compare]

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 13, 2017

Hast du irgendwas anders gemacht als sonst?
Optionen, andere Distri, neue gcc-Version?

Hab gestern nur mein Repo neu gebaut, da hatte ich keine Probleme in der Art. Lass jetzt aber mal das öffentliche durchrödeln...

@UdoSchake
Copy link
Author

Alsooo: Das System ist ein älteres Bananian:
Linux bananapi 3.4.103+ #1 SMP PREEMPT Mon Dec 15 17:37:57 CET 2014 armv7l
gcc ist die richtige Version.
root@bananapi ~ # gcc --version
gcc (Debian 4.7.2-5) 4.7.2
Alle weiteren dependencies für Wheezy wurden installiert.
Heute noch mal neu ausprobiert (war mir nicht sicher ob ich gcc erneuert hatte.) Aber der Fehler trat wieder auf:

[ 64%] Building CXX object src/api/CMakeFiles/vz-api.dir/Volkszaehler.cpp.o cd /root/vzlogger/src/api && /usr/bin/c++ -DHAVE_CONFIG_HPP -W -Wall -Wextra -Werror -Wnon-virtual-dtor -Wno-system-headers -Winit-self -Wmissing-include-dirs -Wno-pragmas -Wredundant-decls -Wno-unused-parameter -std=c++11 -fpermissive -Wno-error=redundant-decls -Wno-ignored-qualifiers -I/usr/local/include -I/root/vzlogger -I/root/vzlogger/include -g3 -o CMakeFiles/vz-api.dir/Volkszaehler.cpp.o -c /root/vzlogger/src/api/Volkszaehler.cpp /root/vzlogger/src/api/Volkszaehler.cpp: In member function ‘virtual void vz::api::Volkszaehler::send()’: /root/vzlogger/src/api/Volkszaehler.cpp:149:25: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] cc1plus: all warnings being treated as errors make[2]: *** [src/api/CMakeFiles/vz-api.dir/Volkszaehler.cpp.o] Error 1 make[2]: Leaving directory /root/vzlogger'
make[1]: *** [src/api/CMakeFiles/vz-api.dir/all] Error 2
make[1]: Leaving directory /root/vzlogger' make: *** [all] Error 2

Gruß
Udo

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 13, 2017

Mein Bananian ist leider etwas neuer: 3.4.108, basiernd auf Debian 8.4.
Gcc ist 4.9.2 wie auch aufm Rpi.
Die betreffende Stelle lief gerade durch, ohne Probleme.

Was mich allerdings stutzig macht: Volkzaehler.cpp kommt bei mir bei ~20%, nicht erst bei 64%.

@UdoSchake
Copy link
Author

wie installierst du denn? Zu Fuß oder mit dem script?

Gruß
Udo

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 13, 2017

Ich hab das damals glaub ohne Script gemacht.

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 13, 2017

Grad auch nochmal mit neuem libsml und json-c probiert - geht. Ohne Script.

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 13, 2017

install.sh geht gar nicht weil mein Bananian kein sudo kennt.

@mbehr1
Copy link
Contributor

mbehr1 commented Feb 13, 2017 via email

@mbehr1
Copy link
Contributor

mbehr1 commented Feb 13, 2017 via email

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 14, 2017

(btw: du solltest nicht als root compilieren…)

In der Installationsanleitung stehen die ganzen sudo auch mit drin. :/

@UdoSchake
Copy link
Author

UdoSchake commented Feb 14, 2017

In der Installationsanleitung stehen die ganzen sudo auch mit drin. :/

Das ist auch richtig so. Nur bei 'Building vzlogger' wird unterschieden.

Das ich bei dem alten Bananian hier mit root arbeite liegt daran, dass kein user eingerichtet ist. Ich weiß, dass das nicht ok ist. Hat aber mit dem Fehler nichts zu tun.

Gruß
Udo

@UdoSchake
Copy link
Author

Änder mal
/root/vzlogger/src/api/Volkszaehler.cpp
Zeile 149 in
if (_values.size() <= (size_t) MAX_CHUNK_SIZE) {

Ja, das war es. Damit läuft make durch.

Gruß
Udo

@J-A-U
Copy link
Collaborator

J-A-U commented Feb 14, 2017

In der Installationsanleitung stehen die ganzen sudo auch mit drin. :/

Das ist auch richtig so. Nur bei 'Building vzlogger' wird unterschieden.

Wenn man das richtige Verzeichnis wählt (in dem man Schreibrechte hat) dann ist (geschätzt) die Hälfte davon unnötig. Vergleiche mal mit install.sh. Da stehen nur 6 sudo drin.

@J-A-U J-A-U self-assigned this Feb 14, 2017
J-A-U pushed a commit to J-A-U/vzlogger that referenced this issue Feb 14, 2017
compile failure signed/unsigned
@andig
Copy link
Contributor

andig commented Feb 14, 2017 via email

J-A-U added a commit that referenced this issue Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants