From 9213ed85f0633f84013e1e449d30a836eb5dae24 Mon Sep 17 00:00:00 2001 From: Tim Schumacher Date: Fri, 3 Jun 2022 17:10:54 +0200 Subject: [PATCH] Ports: Replace manually linking `libvorbis` with a libtool patch --- Ports/libvorbis/package.sh | 11 +-- ...hared-library-support-for-SerenityOS.patch | 76 +++++++++++++++++++ Ports/libvorbis/patches/ReadMe.md | 16 ++++ 3 files changed, 93 insertions(+), 10 deletions(-) create mode 100644 Ports/libvorbis/patches/0001-libtool-Enable-shared-library-support-for-SerenityOS.patch create mode 100644 Ports/libvorbis/patches/ReadMe.md diff --git a/Ports/libvorbis/package.sh b/Ports/libvorbis/package.sh index b2422f0f01b..3bb9f19a8de 100755 --- a/Ports/libvorbis/package.sh +++ b/Ports/libvorbis/package.sh @@ -2,17 +2,8 @@ port=libvorbis version=1.3.7 useconfigure=true +configopts=("--disable-static" "--enable-shared") use_fresh_config_sub=true files="https://github.com/xiph/vorbis/releases/download/v${version}/libvorbis-${version}.tar.gz libvorbis-${version}.tar.gz 0e982409a9c3fc82ee06e08205b1355e5c6aa4c36bca58146ef399621b0ce5ab" auth_type=sha256 depends=("libogg") - -install() { - run make DESTDIR=${SERENITY_INSTALL_ROOT} "${installopts[@]}" install - ${CC} -shared -o ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbis.so -Wl,-soname,libvorbis.so -Wl,--whole-archive ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbis.a -Wl,--no-whole-archive -logg - rm -f ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbis.la - ${CC} -shared -o ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisenc.so -Wl,-soname,libvorbisenc.so -Wl,--whole-archive ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisenc.a -Wl,--no-whole-archive -lvorbis - rm -f ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisenc.la - ${CC} -shared -o ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisfile.so -Wl,-soname,libvorbisfile.so -Wl,--whole-archive ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisfile.a -Wl,--no-whole-archive -lvorbis - rm -f ${SERENITY_INSTALL_ROOT}/usr/local/lib/libvorbisfile.la -} diff --git a/Ports/libvorbis/patches/0001-libtool-Enable-shared-library-support-for-SerenityOS.patch b/Ports/libvorbis/patches/0001-libtool-Enable-shared-library-support-for-SerenityOS.patch new file mode 100644 index 00000000000..8a54b8755f5 --- /dev/null +++ b/Ports/libvorbis/patches/0001-libtool-Enable-shared-library-support-for-SerenityOS.patch @@ -0,0 +1,76 @@ +From ba4fc4788b1955cbfd2edea3756ad65cbccec395 Mon Sep 17 00:00:00 2001 +From: Tim Schumacher +Date: Sun, 29 May 2022 15:01:28 +0200 +Subject: [PATCH] libtool: Enable shared library support for SerenityOS + +For some odd reason, libtool handles the configuration for shared +libraries entirely statically and in its configure script. If no +shared library support is "present", building shared libraries is +disabled entirely. + +Fix that by just adding the appropriate configuration options for +`serenity`. This allows us to finally create dynamic libraries +automatically using libtool, without having to manually link the +static library into a shared library. +--- + configure | 23 +++++++++++++++++++++++ + 1 file changed, 23 insertions(+) + +diff --git a/configure b/configure +index e72b5c7..bfebd8a 100755 +--- a/configure ++++ b/configure +@@ -5945,6 +5945,10 @@ tpf*) + os2*) + lt_cv_deplibs_check_method=pass_all + ;; ++ ++serenity*) ++ lt_cv_deplibs_check_method=pass_all ++ ;; + esac + + fi +@@ -9091,6 +9095,10 @@ lt_prog_compiler_static= + lt_prog_compiler_static='-Bstatic' + ;; + ++ serenity*) ++ lt_prog_compiler_can_build_shared=yes ++ ;; ++ + *) + lt_prog_compiler_can_build_shared=no + ;; +@@ -10613,6 +10621,10 @@ $as_echo "$lt_cv_irix_exported_symbol" >&6; } + hardcode_shlibpath_var=no + ;; + ++ serenity*) ++ ld_shlibs=yes ++ ;; ++ + *) + ld_shlibs=no + ;; +@@ -11693,6 +11705,17 @@ uts4*) + shlibpath_var=LD_LIBRARY_PATH + ;; + ++serenity*) ++ version_type=linux ++ need_lib_prefix=no ++ need_version=no ++ library_names_spec='${libname}${release}${shared_ext}${versuffix} ${libname}${release}${shared_ext}${major} ${libname}${shared_ext}' ++ soname_spec='${libname}${release}${shared_ext}${major}' ++ shlibpath_var=LD_LIBRARY_PATH ++ shlibpath_overrides_runpath=no ++ dynamic_linker='SerenityOS LibELF' ++ ;; ++ + *) + dynamic_linker=no + ;; +-- +2.36.1 + diff --git a/Ports/libvorbis/patches/ReadMe.md b/Ports/libvorbis/patches/ReadMe.md new file mode 100644 index 00000000000..880b81742ef --- /dev/null +++ b/Ports/libvorbis/patches/ReadMe.md @@ -0,0 +1,16 @@ +# Patches for libvorbis on SerenityOS + +## `0001-libtool-Enable-shared-library-support-for-SerenityOS.patch` + +libtool: Enable shared library support for SerenityOS + +For some odd reason, libtool handles the configuration for shared +libraries entirely statically and in its configure script. If no +shared library support is "present", building shared libraries is +disabled entirely. + +Fix that by just adding the appropriate configuration options for +`serenity`. This allows us to finally create dynamic libraries +automatically using libtool, without having to manually link the +static library into a shared library. +