After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 675415 - build: fix librfb linking on win32
build: fix librfb linking on win32
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other All
: Normal normal
: 0.10.24
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-05-03 21:45 UTC by Marc-Andre Lureau
Modified: 2012-05-07 23:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: fix librfb linking on win32 (2.53 KB, patch)
2012-05-03 21:46 UTC, Marc-Andre Lureau
committed Details | Review

Description Marc-Andre Lureau 2012-05-03 21:45:51 UTC
libtool: link:  i686-w64-mingw32-gcc -shared .libs/libgstrfbsrc.dll.def
.libs/libgstrfbsrc_la-gstrfbsrc.o  -Wl,--whole-archive ./.libs/librfb.a
-Wl,--no-whole-archive  -L/usr/i686-w64-mingw32/sys-root/mingw/lib
-lgstbase-0.10 -lgstreamer-0.10 -lxml2 -lgobject-2.0 -lgthread-2.0
-lgmodule-2.0 -lglib-2.0 -lintl  -mms-bitfields -mms-bitfields -O2   -o
.libs/libgstrfbsrc.dll -Wl,--enable-auto-image-base -Xlinker
--out-implib -Xlinker .libs/libgstrfbsrc.dll.a
Creating library file: .libs/libgstrfbsrc.dll.a
./.libs/librfb.a(librfb_la-rfbdecoder.o): In function
`rfb_decoder_read':
/home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:164:
undefined reference to `_imp__recv@16'
./.libs/librfb.a(librfb_la-rfbdecoder.o): In function
`rfb_decoder_connect_tcp':
/home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:112:
undefined reference to `_imp__socket@12'
/home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:119:
undefined reference to `_imp__htons@4'
/home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:123:
undefined reference to `_imp__inet_addr@4'
/home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:125:
undefined reference to `_imp__connect@12'

It needs -lws2_32 which is already in $(WIN32_LIBS)
Comment 1 Marc-Andre Lureau 2012-05-03 21:46:00 UTC
Created attachment 213416 [details] [review]
build: fix librfb linking on win32
Comment 2 Tim-Philipp Müller 2012-05-05 19:03:15 UTC
Doesn't it need at least WSAStartup/WSACleanup bits too then in order to actually work?
Comment 3 Marc-Andre Lureau 2012-05-05 20:27:10 UTC
(In reply to comment #2)
> Doesn't it need at least WSAStartup/WSACleanup bits too then in order to
> actually work?

Sorry, no idea, I just tried to build atm, I use other modules.
Comment 4 Tim-Philipp Müller 2012-05-07 23:07:18 UTC
commit 9503daa56ab5a33bad27bcb0e5f3bc492e3963cb
Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
Date:   Mon May 7 23:57:08 2012 +0100

    rfbsrc: call WSAStartup/Cleanup on win32
    
    If we link against libwinsock2, we should probably call
    Startup/Cleanup as well.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=675415

commit 2f20ec731202b72a03c5013239ab122fa2f22337
Author: Marc-André Lureau <marcandre.lureau@gmail.com>
Date:   Thu May 3 23:42:13 2012 +0200

    rfbsrc: fix librfb linking on win32
    
    libtool: link:  i686-w64-mingw32-gcc -shared .libs/libgstrfbsrc.dll.def
    .libs/libgstrfbsrc_la-gstrfbsrc.o  -Wl,--whole-archive ./.libs/librfb.a
    -Wl,--no-whole-archive  -L/usr/i686-w64-mingw32/sys-root/mingw/lib
    -lgstbase-0.10 -lgstreamer-0.10 -lxml2 -lgobject-2.0 -lgthread-2.0
    -lgmodule-2.0 -lglib-2.0 -lintl  -mms-bitfields -mms-bitfields -O2   -o
    .libs/libgstrfbsrc.dll -Wl,--enable-auto-image-base -Xlinker
    --out-implib -Xlinker .libs/libgstrfbsrc.dll.a
    Creating library file: .libs/libgstrfbsrc.dll.a
    ./.libs/librfb.a(librfb_la-rfbdecoder.o): In function
    `rfb_decoder_read':
    /home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:164:
    undefined reference to `_imp__recv@16'
    ./.libs/librfb.a(librfb_la-rfbdecoder.o): In function
    `rfb_decoder_connect_tcp':
    /home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:112:
    undefined reference to `_imp__socket@12'
    /home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:119:
    undefined reference to `_imp__htons@4'
    /home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:123:
    undefined reference to `_imp__inet_addr@4'
    /home/elmarco/pkg/rhel/mingw-gstreamer-plugins-bad-free/gst-plugins-bad-0.10.23/build_win32/gst/librfb/../../../gst/librfb/rfbdecoder.c:125:
    undefined reference to `_imp__connect@12'
    
    It needs -lws2_32 which is already in $(WINSOCK2_LIBS)
    
    https://bugzilla.gnome.org/show_bug.cgi?id=675415
Comment 5 Tim-Philipp Müller 2012-05-07 23:07:44 UTC
Comment on attachment 213416 [details] [review]
build: fix librfb linking on win32

pushed with s/WIN32_LIBS/WINSOCK2_LIBS/ , which I changed recently.