Skip to content

Make conan install command copyable#730

Merged
memsharded merged 1 commit into
conan-io:develop2from
ChrisThrasher:copyable_install_command
May 13, 2026
Merged

Make conan install command copyable#730
memsharded merged 1 commit into
conan-io:develop2from
ChrisThrasher:copyable_install_command

Conversation

@ChrisThrasher
Copy link
Copy Markdown
Contributor

@ChrisThrasher ChrisThrasher commented May 12, 2026

Closes #729

When ARGN contains CMake lists, those lists get expanded within quotes. This entails that the semicolons that delimit list items are printed in the log message. To make this log message reflect the actual shell command being ran, we want to strip out those semicolons. We do this by joining each list element with spaces to create a new string.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 12, 2026

CLA assistant check
All committers have signed the CLA.

When ARGN contains CMake lists, those lists get expanded within quotes.
This entails that the semicolons that delimit list items are printed in
the log message. To make this log message reflect the actual shell
command being ran, we want to strip out those semicolons.
@ChrisThrasher ChrisThrasher force-pushed the copyable_install_command branch from 50e0816 to 0d09307 Compare May 12, 2026 17:27
@JakeSwartwoutTrimble
Copy link
Copy Markdown

Just tested and this looks to solve my issue, thanks 👍

@memsharded
Copy link
Copy Markdown
Member

Looking good, thanks for the contribution!

@memsharded memsharded merged commit 5ac0500 into conan-io:develop2 May 13, 2026
5 checks passed
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.

Please make the "conan install" message copy-paste able

4 participants