@eht16 commented on this pull request.
> @@ -135,7 +137,8 @@ _getpkg() {
if [ "$use_cache" = "yes" ]; then
package_info=$(pacman -Qi mingw-w64-$ABI-$1)
package_version=$(echo "$package_info" | grep "^Version " | cut
-d':' -f 2 | tr -d '[[:space:]]')
- ls $cachedir/mingw-w64-${ABI}-${1}-${package_version}-* | sort
-V | tail -n 1
+ # use @(gz|xz|zst) to filter out signature files (e.g.
mingw-w64-x86_64-...-any.pkg.tar.zst.sig)
+ ls
$cachedir/mingw-w64-${ABI}-${1}-${package_version}-*.tar.@(gz|xz|zst) | sort -V
| tail -n 1
In what way would be these variants better? For me, they are harder to read and
understand than the commented `*.tar.@(gz|xz|zst)` variant.
As already said, requiring a bash extension, is not an issue in this context.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2590#discussion_r724059766
