Skip to content

Update to latest EsrpClientTool task and allow version override for testing#916

Merged
mjcheetham merged 2 commits into
vfs-2.53.0from
esrp-update
May 15, 2026
Merged

Update to latest EsrpClientTool task and allow version override for testing#916
mjcheetham merged 2 commits into
vfs-2.53.0from
esrp-update

Conversation

@mjcheetham
Copy link
Copy Markdown
Member

@mjcheetham mjcheetham commented May 15, 2026

Task version 4 was using an out of date Node.JS version. Version 5 is the latest.
Test run of the pipeline works.

Whilst we're at it, let's add a way to override the Git version so we can test without pushing an annotated tag. Untagged builds cannot publish to GitHub.

Builds with the override show like this:
image
image
image

Update the EsrpClientTool installer task from v4 to v5.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
@mjcheetham mjcheetham requested a review from dscho May 15, 2026 08:51
Allow Git version override at queue time. This skips the tag resolution
to determine the Git version.

Since this produces a build from an untagged commit, GitHub publishing
will be skipped if set.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
displayName: 'Enable GitHub release publishing'
- name: 'versionOverride'
type: string
default: '-'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We cannot use empty string '' because Azure Pipelines treats this as not being set, and then without a default this becomes a required property! Great... Use '-' as a sentinel value for 'not set'.


steps:
- task: EsrpClientTool@4
- task: EsrpClientTool@5
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This installs the latest 2.x of the esrpclient.exe which is about 25% faster when submitting files for signing!

@mjcheetham mjcheetham changed the title Update to latest EsrpClientTool task Update to latest EsrpClientTool task and allow version override for testing May 15, 2026
@mjcheetham mjcheetham enabled auto-merge May 15, 2026 09:55
@mjcheetham mjcheetham disabled auto-merge May 15, 2026 10:11
@mjcheetham mjcheetham merged commit 07669ec into vfs-2.53.0 May 15, 2026
172 of 178 checks passed
@mjcheetham mjcheetham deleted the esrp-update branch May 15, 2026 10:11
@mjcheetham
Copy link
Copy Markdown
Member Author

Bypassing CI checks - the failures were flakes in the GitHub workflows; totally unrelated to this change.

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