Skip to content
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

Automate aur package publish after each release #1143

Merged
merged 11 commits into from
Feb 20, 2023

Conversation

olblak
Copy link
Member

@olblak olblak commented Feb 7, 2023

Signed-off-by: Olblak me@olblak.com

Fix updatecli/website#651

This pullrequest is not tested, I'll probably use the 0.45.0 alone

Test

/

Additional Information

Tradeoff

Potential improvement

Signed-off-by: Olblak <me@olblak.com>
@olblak olblak added the enhancement New feature or request label Feb 7, 2023
@olblak olblak added this to the 0.45.0 milestone Feb 7, 2023
@ghost
Copy link

ghost commented Feb 7, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

dduportal
dduportal previously approved these changes Feb 12, 2023
@olblak
Copy link
Member Author

olblak commented Feb 18, 2023

I am waiting for some news from @jneidel before merging this pullrequest

updatecli/website#651 (comment)

.goreleaser.yml Outdated Show resolved Hide resolved
.goreleaser.yml Outdated Show resolved Hide resolved
.goreleaser.yml Outdated Show resolved Hide resolved
.goreleaser.yml Outdated Show resolved Hide resolved
.goreleaser.yml Show resolved Hide resolved
@jneidel
Copy link

jneidel commented Feb 18, 2023

I've added you (olblak) as a co-maintainer. So after the changes feel free to release 0.44.3.

Let me know here once you've done so and I'll test the package and verify everything is working as intended 👍

@olblak
Copy link
Member Author

olblak commented Feb 20, 2023

@jneidel I had a quick look at the package that you published

pkgname=updatecli-bin
pkgver="0.44.0"
pkgrel=1
pkgdesc="A Declarative Dependency Management tool"
arch=(x86_64 aarch64)
url="https://github.com/updatecli/updatecli"
license=("Apache")
depends=()
makedepends=("git")
optdepends=()
provides=(updatecli)
source_x86_64=("https://github.com/updatecli/updatecli/releases/download/v${pkgver}/updatecli_Linux_x86_64.tar.gz")
source_aarch64=("https://github.com/updatecli/updatecli/releases/download/v${pkgver}/updatecli_Linux_arm64.tar.gz")
sha256sums_x86_64=("55502f735a1ca7d752198bca65264051e27f2dcf386149167efd3724f4878e85")
sha256sums_aarch64=("1fbb15c5cab4f0c31ad3cc9ea0274c5cfe1bdf2c84be07113743439093615787")

package() {
  cd "$srcdir"

  install -Dm755 updatecli "$pkgdir/usr/bin/updatecli"
  install -Dm644 LICENSE "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
  install -Dm644 manpages/updatecli.1.gz "$pkgdir/usr/share/man/man1/updatecli.1.gz"
  install -Dm644 completions/updatecli.bash "$pkgdir/usr/share/bash-completion/completions/updatecli"
  install -Dm644 completions/updatecli.fish "$pkgdir/usr/share/fish/vendor_completions.d/updatecli.fish"
  install -Dm644 completions/updatecli.zsh "$pkgdir/usr/share/zsh/site-functions/_updatecli"
}

My understanding is pkgname is set to updatecli-bin but we want to use updatecli so we shouldn't use pkgname in the script. Am I right?

Signed-off-by: Olblak <me@olblak.com>
@olblak olblak enabled auto-merge (squash) February 20, 2023 15:34
@olblak
Copy link
Member Author

olblak commented Feb 20, 2023

I am waiting for the tests to pass then I merge and trigger a release 0.44.3

@olblak olblak merged commit 604c626 into updatecli:main Feb 20, 2023
@olblak olblak removed the enhancement New feature or request label Feb 20, 2023
@jneidel
Copy link

jneidel commented Feb 21, 2023

Can confirm that 0.44.4 works on install and that PKGBUILD looks good 😄

@olblak olblak removed this from the 0.45.0 milestone Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants