depot/tools/gerrit-cli.nix, branch refs/r/4512 monorepo for the virus lounge http://code.tvl.fyi/depot/atom?h=refs%2Fr%2F4512 2020-06-21T23:49:30+00:00 fix(gerrit-cli): don't output the path to SSH if it's on the path 2020-06-21T23:49:30+00:00 Luke Granger-Brown hg@lukegb.com 2020-06-21T23:29:09+00:00 urn:sha1:68779da59612dc467425a63f78fb34f5e53af8e0 Change-Id: Ic5ef3e9ae7984cc6c84ef99553bc94a7d44c3929 Reviewed-on: https://cl.tvl.fyi/c/depot/+/551 Reviewed-by: tazjin <mail@tazj.in> fix(tools/gerrit-cli): Don't wrap ssh sniffing in a subshell 2020-06-16T15:43:28+00:00 Griffin Smith grfn@gws.fyi 2020-06-16T13:59:47+00:00 urn:sha1:18994b9c588977375826ea09b090b97851fb1c8e This was actually executing ssh, which was kinda silly - we really just want to check the return code of the `which` invocation Change-Id: I8a4b277a2be3b0c6a43d2da830cb82a32f9ee51e Reviewed-on: https://cl.tvl.fyi/c/depot/+/428 Reviewed-by: tazjin <mail@tazj.in> feat(gerrit-cli): Add gerrit CLI script 2020-06-16T01:54:39+00:00 Griffin Smith grfn@gws.fyi 2020-06-16T00:54:26+00:00 urn:sha1:406a66bed11c273d279bc1470d44e80c354f8d98 Add a script to run gerrit commands on the depot host, which reads the gerrit username from the TVL_USERNAME env var or $(whoami) by default. At some point this may want to solve the configuration problem in a more general fashion, but for now this seems relatively obvious. Change-Id: Ied91c1d26daf4770aef74b2e755d1760c486bb7b Reviewed-on: https://cl.tvl.fyi/c/depot/+/396 Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: glittershark <grfn@gws.fyi>
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<feed xmlns="http://www.w3.org/2005/Atom">
<title>depot/tools/gerrit-cli.nix, branch refs/r/4512</title>
<subtitle>monorepo for the virus lounge</subtitle>
<id>http://code.tvl.fyi/depot/atom?h=refs%2Fr%2F4512</id>
<link rel="self" href="http://code.tvl.fyi/depot/atom?h=refs%2Fr%2F4512"/>
<link rel="alternate" type="text/html" href="http://code.tvl.fyi/"/>
<updated>2020-06-21T23:49:30+00:00</updated>
<entry>
<title>fix(gerrit-cli): don't output the path to SSH if it's on the path</title>
<updated>2020-06-21T23:49:30+00:00</updated>
<author>
<name>Luke Granger-Brown</name>
<email>hg@lukegb.com</email>
</author>
<published>2020-06-21T23:29:09+00:00</published>
<link rel="alternate" type="text/html" href="http://code.tvl.fyi/commit/?id=68779da59612dc467425a63f78fb34f5e53af8e0"/>
<id>urn:sha1:68779da59612dc467425a63f78fb34f5e53af8e0</id>
<content type="text"> Change-Id: Ic5ef3e9ae7984cc6c84ef99553bc94a7d44c3929 Reviewed-on: https://cl.tvl.fyi/c/depot/+/551 Reviewed-by: tazjin <mail@tazj.in> </content>
</entry>
<entry>
<title>fix(tools/gerrit-cli): Don't wrap ssh sniffing in a subshell</title>
<updated>2020-06-16T15:43:28+00:00</updated>
<author>
<name>Griffin Smith</name>
<email>grfn@gws.fyi</email>
</author>
<published>2020-06-16T13:59:47+00:00</published>
<link rel="alternate" type="text/html" href="http://code.tvl.fyi/commit/?id=18994b9c588977375826ea09b090b97851fb1c8e"/>
<id>urn:sha1:18994b9c588977375826ea09b090b97851fb1c8e</id>
<content type="text"> This was actually executing ssh, which was kinda silly - we really just want to check the return code of the `which` invocation Change-Id: I8a4b277a2be3b0c6a43d2da830cb82a32f9ee51e Reviewed-on: https://cl.tvl.fyi/c/depot/+/428 Reviewed-by: tazjin <mail@tazj.in> </content>
</entry>
<entry>
<title>feat(gerrit-cli): Add gerrit CLI script</title>
<updated>2020-06-16T01:54:39+00:00</updated>
<author>
<name>Griffin Smith</name>
<email>grfn@gws.fyi</email>
</author>
<published>2020-06-16T00:54:26+00:00</published>
<link rel="alternate" type="text/html" href="http://code.tvl.fyi/commit/?id=406a66bed11c273d279bc1470d44e80c354f8d98"/>
<id>urn:sha1:406a66bed11c273d279bc1470d44e80c354f8d98</id>
<content type="text"> Add a script to run gerrit commands on the depot host, which reads the gerrit username from the TVL_USERNAME env var or $(whoami) by default. At some point this may want to solve the configuration problem in a more general fashion, but for now this seems relatively obvious. Change-Id: Ied91c1d26daf4770aef74b2e755d1760c486bb7b Reviewed-on: https://cl.tvl.fyi/c/depot/+/396 Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: glittershark <grfn@gws.fyi> </content>
</entry>
</feed>