Skip to content

Conversation

@bhufmann
Copy link
Contributor

@bhufmann bhufmann commented Feb 24, 2025

What it does

Sign 3rd party artifacts with Trace Compass's PGP key. Both Trace Compass update site and RCP update site will be signed.

Use tycho-gpg-plugin:
https://tycho.eclipseprojects.io/doc/4.0.11/tycho-gpg-plugin/sign-p2-artifacts-mojo.html

Upgrade to Tycho 4.0.11 because signing of RCP update site requires Tycho 4.0.10 or later. The new Tycho required to change unnecessary cast settings to warning o.e.tc.segmentstore.core to to avoid build failure.

How to test

I tested using a test Jenkins job and made sure that gpg signing is executed when using profile -Psign-update-site. Then I downloaded the artifact.xml.xz file and checked that the GPG signing is included for the trace-event-logger artifact. Note that, this test Jenkins job and result will be only temporary available.

To test it one the own laptop is more tricky. You would have to move the gpg signing in a own profile (releng-site and rcp.product pom.xml), generate a local GPG signature, update the keyname in the root pom.xml and run maven with that profile and -Dgpg-passphrase=....

Follow-ups

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed the instructions in this template

Signed-off-by: Bernd Hufmann [email protected]

This will get the latest code for gpg signing.

Change unnecessary cast settings to warning o.e.tc.segmentstore.core to
to avoid build failure.

Signed-off-by: Bernd Hufmann <[email protected]>
Both Trace Compass update site and RCP update site will be signed.

Signing of RCP update site requires Tycho 4.0.10 or later.

Use tycho-gpg-plugin:
https://tycho.eclipseprojects.io/doc/4.0.11/tycho-gpg-plugin/sign-p2-artifacts-mojo.html

Signed-off-by: Bernd Hufmann <[email protected]>
@bhufmann bhufmann merged commit f7b29e0 into master Feb 25, 2025
4 checks passed
@bhufmann bhufmann deleted the gpg-signing branch February 25, 2025 14:35
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.

3 participants