-
Notifications
You must be signed in to change notification settings - Fork 7
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
Linker error on compile #1
Comments
so and crossplatform replace |
@DemonSinusa this answer is coming three years late. Maybe it helps. @a-ilin you might want to update two files. After fixing these two bugs the code compiles on linux. The linker compile bug effected compiling on linux and other derivatives. Windows compiling was not effected as the MS windows check was a different function in Vidalia.cpp It might have been better to port Vidalia 0.2.21 Vidalia 0.3.3 definitely feels alpha and some functionality is broken. Circuits do not display or refresh quickly. A QT dev with experience should be able to optimize the code and correct issues with this QT4 to QT5 port. #
replace with
# This resolves the linker error on compile
After insert the section of code appears as
|
@kayfab3 Thanks for the answer, but now so much has changed in the libraries ... Some of the macros and classes QT5 and CMake is deprecated. |
I was able to compile the repo code using the steps outlined above. I can't speak to changes or deprecated macros and classes in QT5. I know this isn't the answer you were hoping for. If someone else is going to port tor further it's not going to be me. |
Fix suggested in a-ilin#1 (comment)
[100%] Building CXX object src/vidalia/CMakeFiles/vidalia.dir/vidalia_autogen/GXDUDAM45N/qrc_vidalia_i18n.cpp.o
[100%] Linking CXX executable vidalia
CMakeFiles/vidalia.dir/Vidalia.cpp.o: In function
VidaliaNativeEventFilter::VidaliaNativeEventFilter()': Vidalia.cpp:(.text._ZN24VidaliaNativeEventFilterC2Ev[_ZN24VidaliaNativeEventFilterC5Ev]+0x1b): undefined reference to
vtable for VidaliaNativeEventFilter'cmake parm:
$ cmake .. -DUSE_QT5=ON -DUSE_GEOIP=1
The text was updated successfully, but these errors were encountered: