net-vpn/vtun: Fix shell script compatibility issue in rc script (bashism).

[leakingmemory]

Newline into the separator (IFS) apparently isn't very easy to accomplish. This seems to be the best way of doing that without relying on bash features.

I guess this fixes the last remaining reported bug for net-vpn/vtun at the moment, so hopefully the scheduled removal can be averted?

#849446

[gentoo-bot]

Pull Request assignment

Submitter: @leakingmemory
Areas affected: ebuilds
Packages affected: net-vpn/vtun

net-vpn/vtun: @gentoo/proxy-maint (maintainer needed)

Linked bugs

Bugs linked: 849446

---

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide

[gentoo-repo-qa-bot]

Pull request CI report

Report generated at: 2022-12-16 20:43 UTC
Newest commit scanned: 67316b0
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/391b646484/output.html

[thesamesam]

Nice work, thanks! I'll unmask it.

[thesamesam]

Would you like to adopt it?

[leakingmemory]

Hi,

Yes, sure, I think this is something I can take on. I'm quite new to writing ebuilds and Gentoo procedures so I'll appreciate greatly some guidance, and obviously thanks to all of you who have done so already. :)

(Btw. sorry if the message got duplicated. It appears replying by email didn't work properly)

[thesamesam]

Sounds great! I normally may not encourage someone to do it so early but I think given:

1. vtun was at risk of removal;
2. you managed to fix all the bugs properly
   it's no problem.

See https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers/User_Guide#Proxied_maintainer_in_metadata.xml for how to do it (new PR). Come ask on IRC in #gentoo-proxy-maint or #gentoo-dev-help if get stuck.

[leakingmemory]

Added https://bugs.gentoo.org/886595