svn commit: r420150 - in head/security/letskencrypt: . files

[Adam Weinberger]

> On 12 Aug, 2016, at 17:17, Bernard Spil <brnrd at freebsd.org> wrote:
> 
> Author: brnrd
> Date: Fri Aug 12 23:17:58 2016
> New Revision: 420150
> URL: https://svnweb.freebsd.org/changeset/ports/420150
> 
> Log:
>  security/letskencrypt: Allow configurable license agreement
> 
>    - Backport upstream fixes for -a option (license agreement URL)
>    - Bump PORTREVISION

Hi Bernard,

I'm confused about what I'm supposed to do here.

It looks like the new license URL was added as default:

> Added: head/security/letskencrypt/files/patch-main.c
> ==============================================================================
> --- /dev/null	00:00:00 1970	(empty, because file is newly added)
> +++ head/security/letskencrypt/files/patch-main.c	Fri Aug 12 23:17:58 2016	(r420150)
> @@ -0,0 +1,55 @@
> + 
> ++#define AGREEMENT "https://letsencrypt.org" \
> ++		  "/documents/LE-SA-v1.1.1-August-1-2016.pdf"

... but the pkg-message change says that I have to add arguments to the invocation:

> Modified: head/security/letskencrypt/files/pkg-message.in
> ==============================================================================
> --- head/security/letskencrypt/files/pkg-message.in	Fri Aug 12 23:02:52 2016	(r420149)
> +++ head/security/letskencrypt/files/pkg-message.in	Fri Aug 12 23:17:58 2016	(r420150)
> @@ -1,4 +1,15 @@
> 
> +LetsEncrypt's ACME servers have changed the LE license
> +agreement. To successfully run renewal you have to add
> +
> +  -a https://letsencrypt.org/documents/LE-SA-v1.1.1-August-1-2016.pdf
> +
> +to the command invocation.

The files/letskencrypt.sh.in doesn't contain that argument.

So is it set automatically? Do I need to alter my letskencrypt.sh, or is the pkg-message inaccurate?

# Adam


-- 
Adam Weinberger
adamw at adamw.org
http://www.adamw.org