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

Link errors when using -DBUILD_SHARED_LIBS=OFF #6846

Closed
ghost opened this issue Mar 1, 2021 · 3 comments · Fixed by #6849
Closed

Link errors when using -DBUILD_SHARED_LIBS=OFF #6846

ghost opened this issue Mar 1, 2021 · 3 comments · Fixed by #6849

Comments

@ghost
Copy link

ghost commented Mar 1, 2021

Just thought I would bring this up again now that you are releasing some new versions. With the following cmake command:
cmake -L -DCMAKE_INSTALL_PREFIX:PATH=$DSTDIR -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=OFF ../
I am getting these link errors and have been for a long while.... Have not had a clean compile for a long time with no shared libs on both fully updated Debian Buster and LUbuntu 18.04.5
...
[ 99%] Linking C executable xfreerdp
/usr/bin/ld: ../../channels/video/client/libvideo-client.a(video_main.c.o): in function video_onMappedGeometryClear': video_main.c:(.text+0x122): undefined reference to mappedGeometryUnref'
/usr/bin/ld: ../../channels/video/client/libvideo-client.a(video_main.c.o): in function PresentationContext_unref': video_main.c:(.text+0x5ea): undefined reference to mappedGeometryUnref'
/usr/bin/ld: ../../channels/video/client/libvideo-client.a(video_main.c.o): in function video_control_on_data_received': video_main.c:(.text+0xd0d): undefined reference to mappedGeometryRef'
/usr/bin/ld: ../../channels/video/client/libvideo-client.a(video_main.c.o): in function VideoFrame_free': video_main.c:(.text+0x1777): undefined reference to mappedGeometryUnref'
/usr/bin/ld: video_main.c:(.text+0x17c6): undefined reference to mappedGeometryUnref' /usr/bin/ld: ../../channels/video/client/libvideo-client.a(video_main.c.o): in function video_data_on_data_received':
video_main.c:(.text+0x25dc): undefined reference to mappedGeometryRef' /usr/bin/ld: video_main.c:(.text+0x2717): undefined reference to mappedGeometryUnref'
/usr/bin/ld: ../../channels/geometry/client/libgeometry-client.a(geometry_main.c.o): in function geometry_DVCPluginEntry': geometry_main.c:(.text+0x14d6): undefined reference to mappedGeometryUnref'
collect2: error: ld returned 1 exit status
make[2]: *** [client/X11/CMakeFiles/xfreerdp-client.dir/build.make:364: client/X11/xfreerdp] Error 1
make[1]: *** [CMakeFiles/Makefile2:3906: client/X11/CMakeFiles/xfreerdp-client.dir/all] Error 2
make: *** [Makefile:152: all] Error 2
...
When I do a cmake like this:
cmake -L -DCMAKE_INSTALL_PREFIX:PATH=$DSTDIR -DCMAKE_BUILD_TYPE=Release ../
it's fine. What Iam I missing?

@akallabeth
Copy link
Member

looks like you´re the first raising this question.
looks like there is some linker line missing as video channel requires geometry channel linked in

@akallabeth akallabeth added this to the next milestone Mar 1, 2021
@ghost
Copy link
Author

ghost commented Mar 1, 2021

Possibly related to the fix that was done in #6643

@akallabeth akallabeth linked a pull request Mar 2, 2021 that will close this issue
@akallabeth
Copy link
Member

@user-36428 yes, looks like I did not backport that one.

@akallabeth akallabeth modified the milestones: next, stable-next Mar 2, 2021
@akallabeth akallabeth modified the milestones: 2.3.1, stable-next Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant