doc: Specify how to set the value of TORGROUP #22420

pull lsilva01 wants to merge 1 commits into bitcoin:master from lsilva01:patch-1 changing 1 files +1 −1

lsilva01 commented at 3:10 PM on July 9, 2021: contributor

This change just makes it more explicit how to assign the value to the TORGROUP variable.
DrahtBot added the label Docs on Jul 9, 2021
tryphe commented at 4:49 AM on July 31, 2021: contributor

ACK

This change seems to make sense as ${TORGROUP} is already referenced in the docs.

Specifies how to set the value of TORGROUP

This change just makes it more explicit how to assign the value to the TORGROUP variable.

5984084610

in doc/tor.md:138 in e0d84e90f0 outdated
```
 136 | @@ -137,6 +137,12 @@ auth cookie will usually be `/run/tor/control.authcookie`:
 137 |  stat -c '%G' /run/tor/control.authcookie
 138 |  ```
 139 |  
 140 | +You can assign the Tor group to a variable called `TORGROUP`:
 141 | +
 142 | +```
 143 | +TORGROUP=$(stat -c '%G' /run/tor/control.authcookie)
 144 | +```
```
laanwj commented at 10:58 AM on August 2, 2021:

Wouldn't it make more sense for this to replace the block above? Most people using a shell will be aware how to assign values to environment variables, but I do think it's a small improvement to add the assignment. But it doesn't need to be mentioned two times.

lsilva01 commented at 3:03 AM on August 8, 2021:

Done.
lsilva01 force-pushed on Aug 8, 2021
Zero-1729 commented at 8:06 PM on August 8, 2021: contributor

Concept ACK 59840846104306febecf185f03a56c821ebb8642
laanwj renamed this:
~~Specifies how to set the value of TORGROUP~~
doc: Specifiy how to set the value of TORGROUP
on Sep 16, 2021
laanwj commented at 6:15 PM on September 16, 2021: member

ACK 59840846104306febecf185f03a56c821ebb8642
laanwj merged this on Sep 16, 2021
laanwj closed this on Sep 16, 2021
laanwj renamed this:
~~doc: Specifiy how to set the value of TORGROUP~~
doc: Specify how to set the value of TORGROUP
on Sep 16, 2021
sidhujag referenced this in commit 2321f94edc on Sep 19, 2021
DrahtBot locked this on Oct 30, 2022

Contributors

Labels