Skip to content

Conversation

@LudovicoPiero
Copy link

First time contributor here, let me know if I missed any conventions!

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-glibc)

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Dec 30, 2025
@LudovicoPiero LudovicoPiero force-pushed the init-samrewritten branch 2 times, most recently from 903a000 to c005aac Compare December 31, 2025 02:12
@LudovicoPiero
Copy link
Author

I've disabled the tests and restricted the package to x86_64.

Signed-off-by: Ludovico Piero <lewdovico@gnuweeb.org>
@LudovicoPiero
Copy link
Author

Applying the feedback from #58343 (comment): I included an unnecessary arch restriction here as well. I am updating the template to allow all architectures supported by upstream.

makedepends="openssl-devel dbus-devel gtk4-devel libadwaita-devel"
short_desc="Modern Steam achievements manager for Windows and Linux"
maintainer="Ludovico Piero <lewdovico@gnuweeb.org>"
license="GPL-3.0-or-later"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
license="GPL-3.0-or-later"
license="GPL-3.0-only"

version=20251229.1
revision=1
build_style=cargo
make_check=no # Require network access and a running Steam client.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be after checksum (i.e. line 17)

@PaulCombal
Copy link

Hi, a gschema needs to be compiled as well in the latest version, I thought you needed to know!

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

Labels

new-package This PR adds a new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants