Skip to content

Conversation

@Jakuje
Copy link
Contributor

@Jakuje Jakuje commented Mar 11, 2025

Mostly adding gpgcheck and links to the final release.

Add -s option to do git tagging and gpg signing only on request, so it
is easier to test the release script w/o messing with signatures.

Ensure tarballs are actually compressed, change to tar.gz as that's the
format git archive understand by default.

Signed-off-by: Simo Sorce <[email protected]>
@Jakuje Jakuje marked this pull request as draft March 11, 2025 18:18
@Jakuje
Copy link
Contributor Author

Jakuje commented Mar 11, 2025

will have to figure out the source for the rpm build as for now it is using the crate. Likely based on the script from #177.

simo5 and others added 3 commits March 11, 2025 14:28
This way reviewers and committers have a checkilist to not forget
about required steps

Signed-off-by: Simo Sorce <[email protected]>
Signed-off-by: Jakub Jelen <[email protected]>
@Jakuje Jakuje force-pushed the packaging-release branch from bd808da to 6b4d73e Compare March 12, 2025 12:00
@Jakuje Jakuje marked this pull request as ready for review March 12, 2025 12:17
@Jakuje Jakuje force-pushed the packaging-release branch 4 times, most recently from bb72cd6 to b3072c9 Compare March 12, 2025 14:39
@Jakuje Jakuje force-pushed the packaging-release branch from b3072c9 to b931ea0 Compare March 12, 2025 14:47
@Jakuje Jakuje mentioned this pull request Mar 12, 2025
@Jakuje Jakuje requested a review from simo5 March 13, 2025 08:30
Copy link
Member

@simo5 simo5 left a comment

Choose a reason for hiding this comment

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

LGTM

%if %{with check}
%check
%if 0%{?rhel}
%cargo_test -f dynamic,nssdb,standard -- -- --exact --skip tests::signatures::test_rsa_signatures
Copy link
Member

Choose a reason for hiding this comment

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

I deally we should fix the sha1 tests instead of skipping them all, but that would take more time, tagging this comment to issue #181 I filed to fix this in general so that we do not forget to change this back.

@simo5 simo5 merged commit 9f5965f into latchset:main Mar 13, 2025
24 checks passed
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.

2 participants