Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

l18n,conf: isolate GOOS=windows code #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vbatts
Copy link

@vbatts vbatts commented Oct 9, 2020

Found when building golang.zx2c4.com/wireguard/windows/conf on linux.

(This logic would be nice to have not in a platform specific repo. Maybe in wireguard-go itself?)

Signed-off-by: Vincent Batts vbatts@hashbangbash.com

vbatts added 2 commits October 9, 2020 14:08
Found when looking at `golang.zx2c4.com/wireguard/windows/conf`, that it
fails on linux, do to this bit.

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
Found when building `golang.zx2c4.com/wireguard/windows/conf` on linux.

Pulled the resolve logic from the wgcfg .ToUAPI()

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts vbatts changed the title l18n: isolate GOOS=windows code l18n,conf: isolate GOOS=windows code Oct 9, 2020
@zx2c4-bot zx2c4-bot force-pushed the master branch 27 times, most recently from f490951 to 6315b61 Compare November 10, 2020 16:35
@zx2c4-bot zx2c4-bot force-pushed the master branch 2 times, most recently from 2c75584 to afe8594 Compare October 11, 2021 05:30
@zx2c4-bot zx2c4-bot force-pushed the master branch 13 times, most recently from dca9737 to 05d1072 Compare October 26, 2021 12:53
@zx2c4-bot zx2c4-bot force-pushed the master branch 7 times, most recently from 2522c0c to 4776166 Compare November 2, 2021 20:53
@zx2c4-bot zx2c4-bot force-pushed the master branch 8 times, most recently from cc96c93 to b1fc806 Compare November 8, 2021 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant