feat(pkg): AUR package is changed to reflect packaging guidelines #278
Conversation
|
Hey @iamwacko, thanks for looking into this and opening a PR. @berendsliedrecht is away for the week, he'll take a look at this when he's back next week |
|
@berendsliedrecht bump on this one |
| case $CARCH in | ||
| "pentium4") |
There was a problem hiding this comment.
Is this the proper check for this? Shouldn't we use arch?
There was a problem hiding this comment.
technically yeah, but with archlinux32 the $CARCH isn't usually i686, so it wouldn't properly compile. I have a few computers running archlinux32, so it's a habit I've picked up.
It doesn't matter too much, I doubt anyone's going to run siera-cli on their 17 year old computer, so you could remove it if you want.
|
@iamwacko could you rebase with main? |
Ok, done |
This will fix #243 . The PKGBUILD now builds siera-cli and doesn't do any of the weird sudo curl stuff. The runner is also now an archlinux container, so it can have checksums and such.
This means the siera AUR package is no longer a binary package, but I have made and uploaded a siera-bin (made properly according to package guidelines) basically does what the old siera package used to do. I could put that in the build process and hand over the package, if you want me to.