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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean up docs on and add tab completion for buildcache command #5125
Conversation
|
||
Build caches are created via: | ||
|
||
.. code-block:: sh | ||
.. code-block:: console |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a command run at the console, not a shell script.
need to be adjusted for better re-locatability. | ||
|
||
.. _cmd-spack-buildcache: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This adds a link in the Command Index.
Looks OK to me. |
-k <key> : the key to sign package with. In the case where multiple keys exist, the package will be unsigned unless -k is used. | ||
-r : make paths in binaries relative before creating tarball | ||
-y : answer yes to all create unsigned "build_cache" questions | ||
<> : list of package specs or package hashes with leading / |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously, this entire section was on a single line: http://spack.readthedocs.io/en/latest/binary_caches.html#spack-buildcache-create
Reformatted to use a table.
command_index | ||
package_list | ||
binary_caches |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The last 2 sections here are lists of packages and links to other docs, not the kind of documentation that people read. I moved Build Caches above these, as people are likely to skip Command Index and Package List when reading the docs.
@adamjstewart Thanks! I never did get time to build the docs. |
The
spack buildcache
command was added a couple days ago in #4854, but there were a couple things missing:This PR adds tab completion and updates the docs to fit better with the rest of our documentation.
@gartung Can you see if these changes look good to you?