gitdist: Pass in '-c color.status=always' when --dist-no-color is not added #588
Conversation
|
@jmgate, not sure if you are still using gitdist or not but I don't know who else to ask to review a PR like this. |
I also updated documentation to mention that the user should also pass in --color=never as well for non-color output.
Calling --color=never to the right commands. These purpose of these commands is to allow for easy archiving of the repo state in files, etc, and not for the shell.
bartlettroscoe
force-pushed
the
color-always
branch
from
0cb0109
to
549162d
Compare
I haven't used it in years, but I was about to pick it up again because I was just put on a project with dozens upon dozens of repositories. I'll take a quick look. |
There was a problem hiding this comment.
Might be worthwhile to add some smarts at some point to automatically add --color=always/never for the commands that accept it, so the user doesn't need to double-specify whether they want colors.
Also, do the -c foo=bar and --color=foo options require a certain version of git? Probably doesn't matter, but thought it might be worth checking.
|
@jmgate, thanks for taking a look! |
…ribits-update This pulls in the new TriBITS PRs: * TriBITSPub/TriBITS#583 * TriBITSPub/TriBITS#585 * TriBITSPub/TriBITS#586 * TriBITSPub/TriBITS#587 * TriBITSPub/TriBITS#588 * TriBITSPub/TriBITS#590
…bits This brings in the following TriBITS PRs: * TriBITSPub/TriBITS#591: Allow 100% raw CMake to be used in a TriBITS-compliant package (TriBITSPub/TriBITS#582) * TriBITSPub/TriBITS#588: gitdist: Pass in '-c color.status=always' when --dist-no-color is not added * TriBITSPub/TriBITS#590: Fix: gitdist: dist-repo-status: Display tag or SHA1 instead of 'HEAD' * TriBITSPub/TriBITS#587: gitdist: dist-repo-status: Display tag or SHA1 instead of 'HEAD' * TriBITSPub/TriBITS#586: More cleanup of packaging support (trilinos#11976) This goes back to TriBITS commits from June 2023.
Have
--dist-no-coloradd-c color.status=neverbefore very git repo command and set-c color.status=alwaysotherwise.