Skip to content

brotli: include static libs in build #58935

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.

Sign up for GitHub

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

Open
NyaaaWhatsUpDoc wants to merge 1 commit into
base: master
Choose a base branch
from
Open

brotli: include static libs in build #58935

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion srcpkgs/brotli/template
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
# Template file for 'brotli'
pkgname=brotli
version=1.2.0
revision=1
revision=2
build_style=cmake
configure_args="-DBROTLI_BUILD_FOR_PACKAGE=ON"
short_desc="Generic-purpose lossless compression algorithm"
maintainer="Leah Neukirchen <leah@vuxu.org>"
license="MIT"
Expand All @@ -12,6 +13,11 @@ checksum=816c96e8e8f193b40151dad7e8ff37b1221d019dbcb9c35cd3fadbfe6477dfec

post_install() {
vlicense LICENSE

# Drop the -static suffix from static libs
Copy link
Member

@ahesford ahesford Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? Upstream picks the name; if there is not a compelling reason to override it, it should remain as they intend.

for p in ${DESTDIR}/usr/lib/*-static.a; do
mv -v ${p} ${p%-static.a}.a
done
}

brotli-devel_package() {
Expand All @@ -20,6 +26,7 @@ brotli-devel_package() {
pkg_install() {
vmove usr/include
vmove usr/lib/*.so
vmove usr/lib/*.a
vmove usr/lib/pkgconfig
}
}