Skip to content

[(z_)vcpkg_prettify_command(_line)] Scripts Tree Audit#16130

Merged
strega-nil merged 7 commits intomicrosoft:masterfrom
strega-nil:scripts-audit/vcpkg_prettify_command
Feb 9, 2021
Merged

[(z_)vcpkg_prettify_command(_line)] Scripts Tree Audit#16130
strega-nil merged 7 commits intomicrosoft:masterfrom
strega-nil:scripts-audit/vcpkg_prettify_command

Conversation

@strega-nil
Copy link
Copy Markdown
Contributor

Audit vcpkg_prettify_command to be in line with #16055

  • Since it's internal, rename to z_vcpkg_prettify_command_line
  • Change it from a macro to a function
  • Change how the function takes arguments, from taking two variable names, to taking a variable name output variable and the command line as the remaining arguments

Additionally, this adds the z_vcpkg_escape_argv helper macro from #16061, except that it's been renamed and modified slightly; #16061 should depend on this PR and use this new macro.

@strega-nil strega-nil changed the title [vcpkg_prettify_command] Scripts Tree Audit [(z_)vcpkg_prettify_command(_line)] Scripts Tree Audit Feb 8, 2021
@JackBoosY JackBoosY added category:documentation To resolve the issue, documentation will need to be updated category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:internal labels Feb 9, 2021
@strega-nil
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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

Labels

category:documentation To resolve the issue, documentation will need to be updated category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants