Skip to content

Conversation

stone-w4tch3r
Copy link
Contributor

@stone-w4tch3r stone-w4tch3r commented Dec 3, 2024

In issue #298 we discussed distributing rpm/deb packages via dedicated repositories.

Initial idea was to use packagecloud, but since it is now not accessible for new users, buildkite was used as a replacement. Actually, it was quite hard to handle all it's bugs, but now it works!

You can try test packages here: rpm and deb.

To make it all work, you need:

  • register an organization named sourcegit on buildkite
  • create two repositories: sourcegit-deb and sorcegit-rpm
  • in personal settings create a token with write_packages and read_packages scopes
  • configure BUILDKITE_TOKEN secret in repository settings
  • push a release!

@stone-w4tch3r stone-w4tch3r marked this pull request as ready for review December 3, 2024 11:26
@aikawayataro
Copy link
Contributor

Looks like a working solution for #298

@aikawayataro
Copy link
Contributor

BTW I couldn't find a workflow run for this in your fork. Can you do a quick release test there?

@stone-w4tch3r
Copy link
Contributor Author

BTW I couldn't find a workflow run for this in your fork. Can you do a quick release test there?

It... disappeared somehow? Anyway look at new release: https://github.com/stone-w4tch3r/sourcegit-fork/actions/runs/12139933496

@stone-w4tch3r
Copy link
Contributor Author

Note that actual push failed because I already changed repository/organization names in an action. But previous packages were pushed successfully

@aikawayataro
Copy link
Contributor

I see, but action is not failed because error is ignored. Would you add --fail to curl and see if workflow fails properly with it 😁

@stone-w4tch3r
Copy link
Contributor Author

Now proper error
https://github.com/stone-w4tch3r/sourcegit-fork/actions/runs/12140279854

@aikawayataro
Copy link
Contributor

Looks good

@love-linger love-linger self-assigned this Dec 4, 2024
@love-linger love-linger added enhancement New feature or request os-linux This issue only exists on Linux labels Dec 4, 2024
@love-linger love-linger merged commit 4392893 into sourcegit-scm:develop Dec 4, 2024
13 checks passed
@love-linger
Copy link
Collaborator

Hi everyone, the new CI workflow release run failed. Please help me fix it.

@sourcegit-scm
Copy link
Owner

I've pushed these packages manually, and modify the release.yml with following changes

@@ -24,8 +24,9 @@ jobs:
     uses: ./.github/workflows/package.yml
     with:
       version: ${{ needs.version.outputs.version }}
   publish-packages:
+    needs: [package, version]
     name: Publish Packages
     uses: ./.github/workflows/publish-packages.yml
     secrets:
       BUILDKITE_TOKEN: ${{ secrets.BUILDKITE_TOKEN }}

Is it right?

@stone-w4tch3r
Copy link
Contributor Author

I've pushed these packages manually, and modify the release.yml with following changes

@@ -24,8 +24,9 @@ jobs:
     uses: ./.github/workflows/package.yml
     with:
       version: ${{ needs.version.outputs.version }}
   publish-packages:
+    needs: [package, version]
     name: Publish Packages
     uses: ./.github/workflows/publish-packages.yml
     secrets:
       BUILDKITE_TOKEN: ${{ secrets.BUILDKITE_TOKEN }}

Is it right?

Yep. Checked, without this line publish fails. You need to return it back to release.yml

@love-linger
Copy link
Collaborator

the original PR missed this line. I've added it to develop branch

@stone-w4tch3r
Copy link
Contributor Author

I can't find repos on buildkite. What links to use? Checked https://buildkite.com/organizations/sourcegit/packages/registries/sourcegit-deb, but only prompt for login is present

@love-linger
Copy link
Collaborator

@BernatBC
Copy link
Contributor

BernatBC commented Dec 9, 2024

rpm package via buildkite works as expected

@stone-w4tch3r
Copy link
Contributor Author

Added appimagehub into readme #798

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request os-linux This issue only exists on Linux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants