Skip to content

Commit da5fae3

Browse files
committed
docs: Fix the chown command in macvtap-bridge.md
When invoking the script chown shows a warning. chown: warning: '.' should be ':': ‘1000.1000’ From `info coreutils 'chown invocation'`. Some older scripts may still use ‘.’ in place of the ‘:’ separator. POSIX 1003.1-2001 (*note Standards conformance::) does not require support for that, but for backward compatibility GNU ‘chown’ supports ‘.’ so long as no ambiguity results, although it issues a warning and support may be removed in future versions. New scripts should avoid the use of ‘.’ because it is not portable, and because it has undesirable results if the entire OWNER‘.’GROUP happens to identify a user whose name contains ‘.’. Signed-off-by: Wei Liu <liuwe@microsoft.com>
1 parent 8a78043 commit da5fae3

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

docs/macvtap-bridge.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ tapindex=$(< /sys/class/net/macvtap0/ifindex)
1818
tapdevice="/dev/tap$tapindex"
1919

2020
# Ensure that we can access this device
21-
sudo chown "$UID.$UID" "$tapdevice"
21+
sudo chown "$UID:$UID" "$tapdevice"
2222

2323
# Use --net fd=3 to point to fd 3 which the shell has opened to point to the /dev/tapN device
2424
target/debug/cloud-hypervisor \

0 commit comments

Comments
 (0)