-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
libplacebo: update to 5.229.2 #40807
Conversation
78721f2
to
eb0b561
Compare
eb0b561
to
55408fd
Compare
55408fd
to
62b9a2b
Compare
hostmakedepends="pkg-config python3-Mako Vulkan-Headers" | ||
makedepends="shaderc Vulkan-Headers vulkan-loader $(vopt_if lcms lcms2-devel) | ||
$(vopt_if opengl libepoxy-devel)" | ||
-Dlcms=$(vopt_if lcms en dis)abled -Ddemos=false -Dtests=true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests should only be enabled when checks are enabled:
-Dlcms=$(vopt_if lcms en dis)abled -Ddemos=false -Dtests=true" | |
-Dlcms=$(vopt_if lcms en dis)abled -Ddemos=false" |
short_desc="Reusable library for GPU-accelerated video/image rendering" | ||
maintainer="Enrico Belleri <idesmi@protonmail.com>" | ||
license="LGPL-2.1-or-later" | ||
homepage="https://code.videolan.org/videolan/libplacebo" | ||
distfiles="https://code.videolan.org/videolan/libplacebo/-/archive/v${version}/libplacebo-v${version}.tar.gz" | ||
checksum=7b3c857934ee3d30f743e43d7f0606e10950806661ea0ea385f8a1f06cbab854 | ||
checksum=a42035c69e4a6373cb9856cc0c5736f24af4485006f3b29605ddffba377264db | ||
|
||
if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then | ||
makedepends+=" libatomic-devel" | ||
LDFLAGS+=" -latomic" | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fi | |
fi | |
if [ -n "$XBPS_CHECK_PKGS" ]; then | |
configure_args+=" -Dtests=true" | |
else | |
configure_args+=" -Dtests=false" | |
fi |
(I like to add both branches just in case the default value of the option changes)
make_check=no # Jinja2 raises exceptions | ||
|
||
do_check() { | ||
nosetests | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make_check=no # Jinja2 raises exceptions | |
do_check() { | |
nosetests | |
} | |
make_check=no # Jinja2 raises exceptions | |
do_check() { | |
nosetests | |
} |
Tested PR (also not with Vulkan but OpenGL) and can confirm it works. |
Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it. |
I am unable to test Vulkan output, but OpenGL works fine.
Testing the changes
Local build testing