kotborealis opened a new pull request, #2137: URL: https://github.com/apache/buildstream/pull/2137
When using `bst workspace reset --all a.bst b.bst c.bst`, list of elements is ignored and it resets all workspaces, which could be confusing. This patch adds a check that either `--all` is specified or list of elementsis provided. This also fixes the following strange problem: Running `bst workspace reset --all --soft` (with en-dash in `--soft`) causes buildstream to parse it as: reset with `all` flag and with list of elements `--soft`, as it could not parse it as a flag due to en-dash. This leads to bst hard-resetting workspace when for user it seems like he requested the `--soft`-one. Now bst would throw error that either `--all` or element list needs to be provided, and would not reset user changes to workspace. Ref. https://github.com/apache/buildstream/pull/2136 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
