[bug#28973] [PATCH 1/1] gnu: Add strongswan.

[Ludovic Courtès]

Hi,

Adam Van Ymeren <address@hidden> skribis:

> Okay updated patch below.  I pulled all the copyright information from
> the debian package.  Added configure arguments to avoid compiling source
> files which are 4-clause BSD licensed.
>
> There is one file who's license I am not sure what to name.
>
> src/libstrongswan/plugins/pkcs11.h
>
> The text of the license is:
>
> "This file is free software; as a special exception the author gives
> unlimited permission to copy and/or distribute it, with or without
> modifications, as long as this notice is preserved."
>
> Debian listed this as "MIT" but I don't believe that is correct.  I
> can't find text of any "MIT" license like this, and it doesn't match the
> expat license either.

Like you wrote it’s like what short GNU helper files have, so you could
use ‘non-copyleft’ like Tobias mentions.

To me the only remaining issue is:

> --- /dev/null
> +++ b/gnu/packages/patches/strongswan-test_process-disable-all.patch
> @@ -0,0 +1,48 @@
> +

For both patches please add two or three lines explaining what the patch
does and why.

The first one can probably be called ‘strongswan-skip-tests.patch’.

> +diff -Naur 
> strongswan-5.6.0-original/src/libstrongswan/tests/suites/test_process.c 
> strongswan-5.6.0/src/libstrongswan/tests/suites/test_process.c
> +--- strongswan-5.6.0-original/src/libstrongswan/tests/suites/test_process.c  
> 2016-04-22 16:01:35.000000000 -0400
> ++++ strongswan-5.6.0/src/libstrongswan/tests/suites/test_process.c   
> 2017-10-23 15:10:12.753837342 -0400
> +@@ -201,27 +201,27 @@
> +
> +     s = suite_create("process");
> +
> +-    tc = tcase_create("return values");
> +-    tcase_add_test(tc, test_retval_true);
> +-    tcase_add_test(tc, test_retval_false);
> +-    suite_add_tcase(s, tc);
> ++    /* tc = tcase_create("return values"); */
> ++    /* tcase_add_test(tc, test_retval_true); */
> ++    /* tcase_add_test(tc, test_retval_false); */
> ++    /* suite_add_tcase(s, tc); */

As a bonus, it’s better if you remove the lines instead of commenting
them out: it makes the patch shorter and easier to read.

Could you send an updated patch?  We’re pretty much done.  :-)

Thank you!

Ludo’.