Skip to content

Meeting Minutes for Planning Council 2026-11-04 - #100

Open
HeikoKlare wants to merge 1 commit into
eclipse-simrel:mainfrom
HeikoKlare:planning-council-2026-11-04
Open

HeikoKlare wants to merge 1 commit into
eclipse-simrel:mainfrom
HeikoKlare:planning-council-2026-11-04

Conversation

@HeikoKlare

@HeikoKlare HeikoKlare commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

@HeikoKlare
HeikoKlare force-pushed the planning-council-2026-11-04 branch from 8ec62ff to 36c600b Compare November 6, 2026 08:32
@HeikoKlare HeikoKlare changed the title Agenda for Planning Council 2026-11-04 Meeting Minutes for Planning Council 2026-11-04 Nov 6, 2026
@HeikoKlare
HeikoKlare marked this pull request as ready for review November 6, 2026 08:38
- Building a product signs the executable again, so a product's executable has one broken signature from Equinox and one valid signature from product build
- Resulting requirement:
- When building products with branding and shipping the artifacts to p2, signing in the build must happen early enough to have the signed artifact in the p2 repository as well
- Currently not possible with Tycho out of the box

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I just want to note that this is not true (but also not very convenient) see documentation here:

https://tycho.eclipseprojects.io/doc/latest/SignProducts.html

It also contains a note that signing the executable itself is not really giving any security, it is more silencing maybe a warning.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

With regard to signing, it's true it can be made to work without modifying Tycho because @sratz fixed it like this in Oomph and BIRT:

It's quite convoluted mind you, but no one will argue with success. 😁

On Windows, the executable in the repository that is "corrupted" by the branding icon (and is only fixed by signing it after branding has stuffed in an icon) can and does lead to Windows refusing to launch the executable, so it's much worse than a warning. That's what @jonahgraham noticed last week and mentioned at the planning council as well.

But now the secondary problem is completely fixed because the latest signing service can even remove the corrupted signature so that the executable only has a single valid signature:

eclipse-oomph/oomph@629cbbf

🥳

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It's quite convoluted mind you, but no one will argue with success. 😁

Yes and it uses ant or bash ... I just wanted to note that there is a plain maven solution documented at Tycho so saying "Currently not possible with Tycho out of the box" seems questionable to me (and contradicting given @sratz fixed it now for oomph+birt) - depending on how one defines OOTB of course, given that there is no standard solution to sign any-ways (Eclipse Signing Service is only available to Eclipse of course).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I thought I already agreed with you. 🤓

When someone contributes a working solution, I don’t “look the gift horse in the mouth” as the saying goes. Maybe there’s a nicer way, probably given your comment. The build of the installer is not so pretty to begin with because we generate several variations of the product on the fly. Maybe there’s a nicer way for that too. I’m not as skilled as you. Pull requests are welcome as we like to say in our trade. 🤗

We all think Tycho is awesome, and as you say, there are no standard signing services that could just be built in. If there were and there were sufficient funding, we could probably provide something more streamlined.

I’m just happy that what comes out at the end is good and happy it works purely by making changes in my build steps, i.e., it works out of the box.

Comment on lines +32 to +37
- Raised questions / concerns
- Previous implementations (Felix, p2, Equinox) were hard to get right (NP-complete problem); it is unclear how much effort a new approach would need beyond the common 90% case
- Why not improve existing resolver / Felix? Is the new approach so fundamentally different? (Incremental) improvements would be less risky than a completely new implementation
- Funding would require a commitment to fix follow-up issues over an extended period
- Follow-up
- Ed will bring it to the PMC meeting, also to get Thomas Watson's opinion

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If there is anything unclear from the proposal I would be happy to answer open questions, maybe the gitlab issue is a better place than the meeting minutes, so just let me know if additional information is desired.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The proposal was well written and well received. Mostly we wanted opinions from other "experts" because none of us can claim to be an expert...

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

As it seems there are some open question I just wanted to offer to clarify / follow up on them if wanted (but as mentioned maybe better in the gitlab ticket) - before or after getting in touch with other stakeholders - so just let me know then.

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.

4 participants