gtristan commented on code in PR #1994:
URL: https://github.com/apache/buildstream/pull/1994#discussion_r2072581909
##########
man/bst-show.1:
##########
@@ -45,6 +45,7 @@ Show elements in the pipeline
%{deps} A list of all dependencies
%{build-deps} A list of build dependencies
%{runtime-deps} A list of runtime dependencies
+ %{artifact-cas-digest} The CAS digest of the built artifact
Review Comment:
This has already been changed in docstring of the `show()` command in
`_frontend/cli.py`.
Updating the man pages is [documented
here](https://docs.buildstream.build/master/hacking/writing_documentation.html#man-pages)
and is done as a single action in a separate commit. (I know, its a bit of an
obscure location and the contributing guide is a bit too much and comes from
days when we had way too many contributors).
Also the man page generation will not work right now, this is fixed in the
`tristan/sboms` branch in 0c2a92d2667f7f88938bea8779d50484ed6b6d94
Actually you may have run into this brokenness of man page generation and
I'm ambivalent if this lands in advance of `tristan/sboms`, if this lands after
`tristan/sboms`, it will be better for you to remove this change, follow the
updated documentation, and add a "man pages update" commit to this branch.
--
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]
