[gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in media-libs/libsdl2/files: libsdl2-2.0.3-static-libs.patch

[Samuli Suominen]

On 12/05/14 18:56, Julian Ospald (hasufell) wrote:
> hasufell    14/05/12 15:56:05
>
>   Added:                libsdl2-2.0.3-static-libs.patch
>   Log:
>   version bump
>
>   (Portage version: 2.2.10/cvs/Linux x86_64, signed Manifest commit with key BDEED020)
>
> Revision  Changes    Path
> 1.1                  media-libs/libsdl2/files/libsdl2-2.0.3-static-libs.patch
>
> file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/media-libs/libsdl2/files/libsdl2-2.0.3-static-libs.patch?rev=1.1&view=markup
> plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/media-libs/libsdl2/files/libsdl2-2.0.3-static-libs.patch?rev=1.1&content-type=text/plain
>
> Index: libsdl2-2.0.3-static-libs.patch
> ===================================================================
> --- SDL2-2.0.2.orig/Makefile.in
> +++ SDL2-2.0.2/Makefile.in
> @@ -33,10 +33,10 @@
>  OBJECTS = @OBJECTS@
>  VERSION_OBJECTS = @VERSION_OBJECTS@
>
> -SDLMAIN_TARGET = libSDL2main.a
> +SDLMAIN_TARGET = libSDL2main.la
>  SDLMAIN_OBJECTS = @SDLMAIN_OBJECTS@
>
> -SDLTEST_TARGET = libSDL2_test.a
> +SDLTEST_TARGET = libSDL2_test.la
>  SDLTEST_OBJECTS = @SDLTEST_OBJECTS@
>
>  SRC_DIST = *.txt acinclude Android.mk autogen.sh android-project build-scripts cmake configure configure.in debian include Makefile.* sdl2-config.in sdl2.m4 sdl2.pc.in SDL2.spec.in src test VisualC.html VisualC Xcode Xcode-iOS
> @@ -123,15 +123,13 @@
>  .PHONY: all update-revision install install-bin install-hdrs install-lib install-data uninstall uninstall-bin uninstall-hdrs uninstall-lib uninstall-data clean distclean dist $(OBJECTS:.lo=.d)
>
>  $(objects)/$(TARGET): $(OBJECTS) $(VERSION_OBJECTS)
> -	$(LIBTOOL) --mode=link $(CC) -o $@ $(OBJECTS) $(VERSION_OBJECTS) $(LDFLAGS) $(EXTRA_LDFLAGS) $(LT_LDFLAGS)
> +	$(LIBTOOL) --mode=link $(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS) $(EXTRA_LDFLAGS) $(LT_LDFLAGS)

You know that adding $(LDFLAGS) so late in the linker line makes whole
-Wl,--as-needed get ignored? Should almost certainly be $(CC) $(LDFLAGS)
$(CFLAGS) ...

- Samuli