macOS 运行install.sh出错

[MJFcoNaN]

remote: Total 18 (delta 1), reused 10 (delta 0)
Unpacking objects: 100% (18/18), done.
Removing old configurations...
Installing new configurations...
sed: 1: "/etc/dnsmasq.d/accelera ...": extra characters at the end of d command

Google该错误后，似乎是因为Unix中sed的用法有些不同，根据一些建议，第37行加一个空字符后可用，即：
sed -i "" "s|^(server.)/[^/]$|\1/$_server|" /etc/dnsmasq.d/*."$_server".conf

[erickyim]

@MJFcoNaN

sed -i "" "s|^(server.)/[^/]$|\1/$_server|" /etc/dnsmasq.d/*."$_server".conf
seems not work on macOS 10.14.5 ...

Removing old configurations...
Installing new configurations...
sed: 1: "s|^(server.)/[^/]$|\1/1 ...": \1 not defined in the RE

[MJFcoNaN]

@erickyim
Curiously...
It works for me (macOS Mojave 10.14.5)

remote: Total 18 (delta 1), reused 10 (delta 0) Unpacking objects: 100% (18/18), done. Removing old configurations... Installing new configurations... Restarting dnsmasq service... Now please restart dnsmasq since I don't know how to do it. Cleaning up...

[erickyim]

OK, it's a copy-paste issue.

# macOS
sed -i "" "s|^\(server.*\)/[^/]*$|\1/$_server|" /etc/dnsmasq.d/*."$_server".conf

Now works...

[felixonmars]

sed -i "" seems to be a macos only solution as it doesn't work in linux version of sed. Someone on stackoverflow suggests that sed -i="" works cross-platform and I have tried it on Linux, could you confirm that it works on macos too?

[MJFcoNaN]

@felixonmars
yes, sed -i="" works on macOS.

[erickyim]

seems not work.

[MJFcoNaN]

"=" is considered as the extension of backup files...
But I think this does not imperil the function, does it?
-i extension Edit files in-place, saving backups with the specified extension. If a zero- length extension is given, no backup will be saved. It is not recommended to give a zero-length extension when in-place editing files, as you risk corrup- tion or partial content in situations where disk space is exhausted, etc.