search

garywill / linux-router

Set Linux as router in one command. Support Internet sharing, redsocks, Wifi hotspot, IPv6. Can also be used for routing VM/containers 🛰️ (也欢迎关注B站 https://space.bilibili.com/2123686105 )
GNU Lesser General Public License v2.1
1.71k stars 152 forks source link

mktemp: Invalid argument #21

Closed jrabausch closed 3 years ago

jrabausch commented 3 years ago

Hi, im using Alpine Linux 3.12 with BusyBox 1.31.1 and mktemp fails with mktemp: Invalid argument. I had to change line 1238 to CONFDIR="$(mktemp -d $TMPDIR/lnxrouter.${TARGET_IFACE}.conf.XXXXXX)".

mktemp requires 6 times X for the file/dir template.

Awesome script, best regards.

garywill commented 3 years ago

Hey thank you for feedback! We should increase it to 6. Do you want to make a PR to be a contributor? Or I can do it myself.

jrabausch commented 3 years ago

You can apply this small fix yourself. I'll close the issue.

Best regards.