AmneziaWG VPN packages - #62535
AmneziaWG VPN packages#62535xolophreny wants to merge 3 commits into
Conversation
why restricted? is there a licensing concern? |
Ah, no, pardon, I misunderstood the implications of failing build checks. That is the only issue with the thing, though I'm not familiar with Go software at all to provide more details. Just the "Unformatted code" in many Go files being the reason for failing, as mentioned in the template And would appreciate more input on this as well, thank you |
309ba38 to
951e00f
Compare
| distfiles="${homepage}/archive/v${version}.tar.gz" | ||
| checksum=24c656cfb80ff6855702710eaf2e3729fa710bf6bfdbbbdfba01984ccd17de95 | ||
|
|
||
| make_check=no # "Unformatted code" in many go files |
There was a problem hiding this comment.
Skipping checks is only acceptable if you can explain why the tests are expected not to work in the build environment. Skipping them to avoid solving the issue is not.
There was a problem hiding this comment.
Understood, thank you! Will work on that
951e00f to
06ac282
Compare
Testing the changes
New package
Local build testing
Alternative Wireguard implementation.
The DKMS module should be used whenever possible, the Go variant is slower and doesn't pass build checking, though otherwise works. I should be able to bring the issue up to the devs, and would like to ask if it could be included as a restricted package until the problem is resolved, or if it should be dropped entirely