AUR (en) - bluetuith
whiteman808 commented on 2025-09-21 17:41 (UTC) (edited on 2025-09-21 17:44 (UTC) by whiteman808)
tamirzb commented on 2025-09-17 10:11 (UTC)
@whiteman808 Why go back to v0.2.5? And why use epoch?
jhenson commented on 2025-08-29 03:39 (UTC)
@TrialnError, the replacement is a good idea and I was unaware that vercmp would treat the _rc1 suffix as "newer" like that (TIL!).
I've pushed and updated pkgbuild with the version fixed. Thanks for the suggestion!
TrialnError commented on 2025-08-28 18:43 (UTC) (edited on 2025-08-28 18:48 (UTC) by TrialnError)
In such cases substitution could be used:
pkgver=0.2.5_rc1
And wherever the real one is needed: ${pkgver/_/-} which replaces _ with -.
Formally the full pkgver from upstream should be seen in the pkgver. Not only for maintenance reasons like tamirzb noted, but it is expected. That being said I am not sure how I would handle it. Although it seems the proposed case above would be recognized by vercmp? (vercmp 0.1 0.1_rc1). And if not some other substitution should work: pkgver=0.2.5_1 and ${pkgver/_/-rc}?
Edit: Some correction. I missed a - in the output of vercmp xD
jhenson commented on 2025-08-28 13:29 (UTC)
Normally I would agree but 0.2.5-rc1 is not valid in the pkgver:
==> ERROR: pkgver is not allowed to contain colons, forward slashes, hyphens or whitespace.
That said, even if it was, -rc1 would be considered a downgrade given that 0.2.5 had already been published which would require a bump to the epoch for pacman to treat it as an upgrade. Given that this is the first time upstream has done this (afaik), I think it's better to just work around it and hope they don't use pre-release versions as bugfixes in the future. If it starts to become a problem then some other way of versioning this package will likely be needed.
tamirzb commented on 2025-08-28 07:27 (UTC)
@jhenson Yeah their usage of -rc1 is a bit weird. In this case it might be better to put the full 0.2.5-rc1 in the pkgver as usually the convention is to increase pkgver when the code changes and increase pkgrel when only the PKGBUILD changes. You can then also use the $pkgver variable in other parts of the PKGBUILD instead of hardcoding it for easier maintenance.
jhenson commented on 2025-08-27 17:23 (UTC)
Ah! Thanks for that, @TrialnError. I totally missed when they added that support. I've updated the package (and fixed the source URL that I hadn't noticed was broken).
TrialnError commented on 2025-08-27 17:12 (UTC)
pacman/makepkg is able to produce checksums since 6.1 if the tag fragment is used with a git source. And #tag is used.
jhenson commented on 2025-08-27 12:07 (UTC)
As I said in the commit message, upstream has (apparently) used the -rc1 suffix as a bugfix release (it was released after 0.2.5 and contains a fix for a nil pointer dereference). That's why I left this package at 0.2.5.
I dropped the SHA since the pakage is now building from the git repo and is now setting the version the same way upstream does it.
tamirzb commented on 2025-08-27 05:14 (UTC)
Why the shift to a release candidate instead of the stable version?
Also why remove the sha256 checksum?
@tamirzb mistake was corrected
Anyone who upgraded this to v0.2.5 should do:
I'm doing contributions to AUR first time, let me know if there any problems with this package