Defect #715
PVS kompiliert nicht auf AMD64-System
| Status: | Erledigt | Start date: | 06/03/2010 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | 06/11/2010 | |
| Assignee: | jl | % Done: | 100% |
|
| Category: | - | |||
| Target version: | - | Estimated time: | 1.00 hour | |
| Resolution: | fixed |
Description
Beim Bau des PVS auf einem AMD64-System erscheint die Meldung
[ 64%] Building CXX object CMakeFiles/pvsmgrtouch.dir/src/net/pvsListenServer.cpp.o /home/brs/Projects/pool-video-switch/pvs-own/src/net/pvsListenServer.cpp: In member function 'bool PVSListenServer::init()': /home/brs/Projects/pool-video-switch/pvs-own/src/net/pvsListenServer.cpp:249: error: cast from 'SslServer*' to 'int' loses precision make[2]: *** [CMakeFiles/pvsmgrtouch.dir/src/net/pvsListenServer.cpp.o] Error 1
denn sizeof(int)==4 während sizeof(void*)==8. Der angehängte Patch führt eine Funktion ptr2String ein, die den %p-Formatcode verwendet um einen Pointer auf portable Weise in einen String umzuwandeln.
History
#1 Updated by dvs almost 2 years ago
- Due date set to 06/11/2010
- Assignee set to jl
- Estimated time set to 1.00
Hm, das hat wohl keiner so recht getestet :) Hier könnten Simon und Johann mal einen qualifizierten Blick werfen!
#2 Updated by sr almost 2 years ago
Dieser Bug wurde eigentlich schon mal gefixt, indem die Zeile komplett entfernt wurde. Das Problem trat damals nämlich bei Ben auch auf, der bei uns der einzige 64Bit-User war. :)
Ich habe mich leider noch nicht in das git Repo eingearbeitet bzw. weiß nicht, ob ich überhaupt Zugang dazu hätte. ;)
Das Einfachste wäre, wenn jemand mit entsprechendem Zugriff entweder den Patch von Sébastien einpflegt, oder die Ausgabe einfach entfernt.
Alternativ etwas sinnvolleres als den Pointer ausgeben, z.B. die Portnummer: _listenServer->serverPort()
#3 Updated by jl almost 2 years ago
- Status changed from Neu to Erledigt
- % Done changed from 0 to 100
- Resolution set to fixed
Habe einfach den Patch vom Sébastien genommen. Sollte nun im git repo drin ein.