Differential D11450

Create default dirs and symlink to dnsapi scripts
ClosedPublic
Actions

Authored by dvl on Jul 2 2017, 4:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 20, 6:05 PM

	Unknown Object (File)
	Sun, Sep 20, 5:37 PM

	Unknown Object (File)
	Sun, Sep 20, 4:43 PM

	Unknown Object (File)
	Sun, Sep 20, 4:40 PM

	Unknown Object (File)
	Sun, Sep 20, 4:31 PM

	Unknown Object (File)
	Sat, Sep 19, 7:29 PM

	Unknown Object (File)
	Fri, Sep 18, 10:26 PM

	Unknown Object (File)
	Wed, Sep 9, 1:21 PM

Subscribers

Details

Reviewers: None
Commits: rP444990: Add some sane defaults in account.conf to get you started.

Summary

~/certs and ~/.acme.sh for the acme user and provide a sample account.conf

Test Plan

https://services.unixathome.org/poudriere/build.html?mastername=110amd64-working&build=2017-07-02_16h18m43s

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dvl created this revision.Jul 2 2017, 4:19 PM

Herald added a subscriber: mat. · View Herald TranscriptJul 2 2017, 4:19 PM

Harbormaster completed remote builds in B10255: Diff 30323.Jul 2 2017, 4:19 PM

Remove wrong code from files/pkg-install.in

Harbormaster completed remote builds in B10256: Diff 30324.Jul 2 2017, 4:29 PM

mat added inline comments.Jul 3 2017, 1:00 PM

security/acme.sh/Makefile
45 ↗	(On Diff #30324)	You cannot put sample files in there. You have to move them somewhere else, like EXAMPLESDIR. newsyslog.conf has: <include> /usr/local/etc/newsyslog.conf.d/* So all the files in there are always used. If you put a .sample file, it will be used. Also, the file should probably be named acme.sh, and not the very long name you have now.
security/acme.sh/files/pkg-install.in
9–11 ↗	(On Diff #30324)	/usr/bin/install -d -o acme -g acme -m 750 ~/acme/.acme.sh
15–17 ↗	(On Diff #30324)	/usr/bin/install -d -o acme -g acme -m 750 ~/acme/certs
21 ↗	(On Diff #30324)	I would use %%EXAMPLESDIR%% directly here, no real need to create a variable.

Updates as suggested

Harbormaster completed remote builds in B10281: Diff 30376.Jul 3 2017, 9:41 PM

Closed by commit rP444990: Add some sane defaults in account.conf to get you started. (authored by dvl). · Explain WhyJul 3 2017, 10:22 PM

This revision was automatically updated to reflect the committed changes.

dvl marked 4 inline comments as done.

Revision Contents
Changeset List

Path

Size

head/

security/

acme.sh/

5 lines

files/

6 lines

29 lines

19 lines

1 line

Diff 30383

head/security/acme.sh/Makefile

Loading...

head/security/acme.sh/files/acme.sh

Loading...

head/security/acme.sh/files/pkg-install.in

Loading...

head/security/acme.sh/files/pkg-message.in

Loading...

head/security/acme.sh/pkg-plist

Loading...