Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add build to vPack Pipeline#25915

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
TravisEz13 merged 36 commits intomasterfromaddBuildTovPack
Sep 3, 2025
Merged

Add build to vPack Pipeline#25915

TravisEz13 merged 36 commits intomasterfromaddBuildTovPack
Sep 3, 2025

Conversation

@jshigetomi
Copy link
Collaborator

@jshigetomijshigetomi commentedAug 26, 2025
edited
Loading

PR Summary

This pull request introduces significant improvements to the PowerShell build pipeline by making the build and signing process more flexible and robust. The main changes allow the pipeline to distinguish between official and non-official builds, dynamically select templates, and provide more granular handling of code signing. These updates make it easier to manage different build scenarios and improve the reliability and clarity of signing checks.

Pipeline configuration and template selection:

  • Added theOfficialBuild boolean parameter andvPackName parameter to.pipelines/PowerShell-vPack-Official.yml, enabling selection between official and non-official build templates and scenarios.
  • Dynamically set the pipeline template via the newtemplateFile variable, allowing the pipeline to choose betweenMicrosoft.Official.yml andMicrosoft.NonOfficial.yml based on the build type.[1][2]

Build and artifact handling:

  • Refactored build steps to create a dedicated symbols folder per architecture, set thePowerShellRoot andRuntime variables, and improved output and artifact directory management for clarity and consistency.[1][2][3]

Code signing improvements:

  • Updated.pipelines/templates/obp-file-signing.yml andtools/packaging/packaging.psm1 to accept and propagate theOfficialBuild parameter, allowing signing logic to distinguish between official and non-official builds and adjust signature checks and warnings accordingly.[1][2][3][4][5][6]
  • Made the signing profile configurable via theSigningProfile parameter, supporting different signing requirements for official and non-official builds.

These changes collectively improve the maintainability and reliability of the pipeline, especially around build type selection and code signing.

PR Context

PR Checklist

@jshigetomijshigetomi requested a review froma team as acode ownerAugust 26, 2025 01:46
@jshigetomijshigetomi added CL-BuildPackagingIndicates that a PR should be marked as a build or packaging change in the Change Log BackPort-7.4.x-Consider BackPort-7.5.x-Consider labelsAug 26, 2025
Remove reference to packages pipelineUse repoRootset architecture variable to stringset and use variable architecutreGet the fullname of all artifacrtsLook in Symbols_Arch for all filesInitialize PowerShellRoot for signing templateSkip signature validation for nowrevert signature validationLook for tsa and credscan in pipeline.workspacerevert signature validationLook for tsa and credscan in pipeline.workspaceturn off sig valcheck for test cert sigining in pipeline and Update-PSSignedBuildFolderPass parameter to signing templateUse bool not stringcast to booleanAdd parameter to second call to methodlook for credscan in pipeline.workspacerevert windows-hosted build changeRevert credscandefine artifactPlatformRemove checkoutdefine artifactPlatformRemove checkoutmaintain checkout to only sAdd parameter for vPack nameRemove templateAdd parameter names
@jshigetomi
Copy link
CollaboratorAuthor

jshigetomi commentedAug 27, 2025
edited
Loading

/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI

@PowerShellPowerShell deleted a comment fromazure-pipelinesbotAug 27, 2025
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

jshigetomiand others added13 commitsAugust 27, 2025 16:32
Remove reference to packages pipelineUse repoRootset architecture variable to stringset and use variable architecutreGet the fullname of all artifacrtsLook in Symbols_Arch for all filesInitialize PowerShellRoot for signing templateSkip signature validation for nowrevert signature validationLook for tsa and credscan in pipeline.workspacerevert signature validationLook for tsa and credscan in pipeline.workspaceturn off sig valcheck for test cert sigining in pipeline and Update-PSSignedBuildFolderPass parameter to signing templateUse bool not stringcast to booleanAdd parameter to second call to methodlook for credscan in pipeline.workspacerevert windows-hosted build changeRevert credscandefine artifactPlatformRemove checkoutdefine artifactPlatformRemove checkoutmaintain checkout to only sAdd parameter for vPack nameRemove templateAdd parameter names
…lBuild and debug, and adjust vPackName settings
@lrpaula411-art

This comment was marked as spam.

@lrpaula411-art
Copy link

Instalar

lrpaula411-art

This comment was marked as spam.

@jshigetomi
Copy link
CollaboratorAuthor

/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI

@PowerShellPowerShell deleted a comment fromazure-pipelinesbotSep 3, 2025
@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@TravisEz13TravisEz13 merged commit499cedf intomasterSep 3, 2025
45 checks passed
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-servicebot commentedSep 3, 2025
edited by unfurl-linksbot
Loading

📣 Hey @@jshigetomi, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗https://aka.ms/PSRepoFeedback

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

Reviewers

@TravisEz13TravisEz13TravisEz13 approved these changes

@adityapatwardhanadityapatwardhanAwaiting requested review from adityapatwardhan

+1 more reviewer

@lrpaula411-artlrpaula411-artlrpaula411-art left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Backport-7.4.x-MigratedBackPort-7.5.x-DoneCL-BuildPackagingIndicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jshigetomi@lrpaula411-art@TravisEz13@adityapatwardhan

[8]ページ先頭

©2009-2025 Movatter.jp