-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[new package] add rio terminal #22012
Conversation
"${MINGW_PACKAGE_PREFIX}-cc" | ||
"${MINGW_PACKAGE_PREFIX}-pkgconf" | ||
"${MINGW_PACKAGE_PREFIX}-cmake" | ||
"${MINGW_PACKAGE_PREFIX}-freetype" |
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.
Are you sure fontconfig and freetype dependencies are required in this build? Those are not shown in the runtime libraries list in CI log.
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.
rust only can be left. cmake is usually used for libz-ng-sys, which is not used for package, freetype and fontconfig are probably used for Linux/macos, I didn't find them all in build logs
build() { | ||
cd "${_realname}-${pkgver}" | ||
|
||
cargo build --frozen --release --all-features |
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.
is --all-features
required? I see all useful features are already enabled by default upstream
pkgrel=1 | ||
pkgdesc="A hardware-accelerated GPU terminal emulator powered by WebGPU (mingw-w64)" | ||
arch=('any') | ||
mingw_arch=('mingw64' 'ucrt64' 'clang64') |
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.
mingw_arch=('mingw64' 'ucrt64' 'clang64') | |
mingw_arch=('ucrt64' 'clang64' 'clangarm64') |
rio package has been added from #22248 |
Fixed the LTO issue on upstream!