Thread (8 messages) 8 messages, 4 authors, 2024-03-20

Re: [PATCH v3 0/2] Add hostname condition to includeIf

From: Eric Sunshine <hidden>
Date: 2024-03-20 02:49:42

On Tue, Mar 19, 2024 at 8:19 PM Jeff King [off-list ref] wrote:
On Tue, Mar 19, 2024 at 05:13:47PM -0400, Eric Sunshine wrote:
quoted
The other possibility which came to mind was adding a GIT_HOSTNAME
variable to the output of `git var -l`.
That strikes me as a more appropriate spot than an option to git-config.
Even if config is the only thing _now_ which cares about the hostname,
it may be something that other parts of the system care about in the
future.
Also, taking into consideration Patrick's proposed revamp[1] of
git-config to give it a subcommand API, then git-config becomes an
even less welcome place for a standalone --show-hostname option which,
by itself, doesn't really fit into the subcommand paradigm, and it
probably doesn't make sense to add a new subcommand ("info" or
whatever) just for that.

[1]: https://lore.kernel.org/git/cover.1710198711.git.ps@pks.im/T/#u (local)
Some care may need to be taken for error handling, though. For "git var
GIT_HOSTNAME" it is OK to exit non-zero, but "git var -l" should not
bail on a system where gethostname() doesn't work (it is still not clear
to me if that is a real case to worry about or not).
Ports to oddball platforms should probably be providing a
gethostname() in "compat/" anyhow, just as is done for Windows in
"compat/mingw.c".
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help