Page MenuHomeFreeBSD
Differential D13700

New port: net/nncp: Utilities for secure store-and-forward files and mail exchanging
ClosedPublic
Actions

Authored by yuri on Dec 31 2017, 12:23 AM.
Tags
None
Referenced Files
F134090131: D13700.id37407.diff
Thu, Oct 30, 4:48 PM
Unknown Object (File)
Wed, Oct 29, 2:22 PM
Unknown Object (File)
Sun, Oct 26, 4:55 AM
Unknown Object (File)
Sat, Oct 25, 1:38 AM
Unknown Object (File)
Fri, Oct 24, 1:38 AM
Unknown Object (File)
Mon, Oct 20, 6:53 PM
Unknown Object (File)
Mon, Oct 20, 6:53 PM
Unknown Object (File)
Mon, Oct 20, 6:53 PM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
adamw
mat
Commits
rP460314: New port: net/nncp: Utilities for secure store-and-forward files and mail…
Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=215973

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14033
Build 14223: arc lint + arc unit

Event Timeline

yuri created this revision.Dec 31 2017, 12:23 AM
Herald added a subscriber: mat. · View Herald TranscriptDec 31 2017, 12:24 AM
Harbormaster completed remote builds in B13968: Diff 37266.Dec 31 2017, 12:24 AM
mat requested changes to this revision.Dec 31 2017, 9:41 AM
mat added inline comments.
net/nncp/Makefile
31

You cannot install sample newsyslog files in the newsyslog.conf.d directory, as all the files in it are used.

This revision now requires changes to proceed.Dec 31 2017, 9:41 AM
tcberner added inline comments.Dec 31 2017, 9:50 AM
net/nncp/Makefile
5

^DISTVERSION

yuri updated this revision to Diff 37302.Dec 31 2017, 9:59 AM

.

Harbormaster completed remote builds in B13992: Diff 37302.Dec 31 2017, 9:59 AM
yuri marked an inline comment as done.Dec 31 2017, 10:03 AM
yuri added inline comments.
net/nncp/Makefile
5

The portlint maintainer doesn't see this stated in the PHB: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224002
He closed my request.

yuri updated this revision to Diff 37303.Dec 31 2017, 10:28 AM
yuri marked an inline comment as done.

Rearranged conf files to avoid .sample being in etc/newsyslog.conf.d/

yuri marked 2 inline comments as done.Dec 31 2017, 10:29 AM
mat added inline comments.Dec 31 2017, 10:25 PM
net/nncp/Makefile
32

I do not understand why this is so complicated.

Please have a look at 8.3. Configuration Files.

yuri marked an inline comment as done.Dec 31 2017, 11:56 PM
yuri added inline comments.
net/nncp/Makefile
32

It actually follows PHB section 8.3.
It creates a .conf file in etc/ using @sample, and links the real config file etc/newsyslog.conf.d/nncp.conf to it.

Could you please explain what's wrong with this?

yuri updated this revision to Diff 37356.Jan 1 2018, 5:22 PM
yuri marked an inline comment as done.

The submitter verified that it works, and corrected one typo.

@mat, if you don't have any objections, could you please unblock this?

Harbormaster completed remote builds in B14033: Diff 37356.Jan 1 2018, 5:22 PM
mat added inline comments.Jan 2 2018, 11:52 AM
net/nncp/Makefile
32

Yes, I could.

@sample etc/nncp.conf.sample ​etc/newsyslog.conf.d/nncp.conf
yuri updated this revision to Diff 37407.Jan 2 2018, 11:58 AM
yuri marked 2 inline comments as done.

.

Harbormaster completed remote builds in B14062: Diff 37407.Jan 2 2018, 11:58 AM
tcberner accepted this revision.Jan 9 2018, 9:52 PM

As long as mat is fine with it too now, this is good to go.

yuri retitled this revision from New port: net/nncp: Utilities for secure store-and-forward files and mail exchanging to New port: net/nncp: Utilities for secure store-and-forward files and mail exchanging {PENDING portmgr@ approval}.Jan 9 2018, 10:28 PM
yuri added a comment.Jan 29 2018, 12:33 AM

Can this be unblocked by portmgr@?

mat accepted this revision.Jan 29 2018, 11:47 AM

It never was blocked by portmgr.

This revision is now accepted and ready to land.Jan 29 2018, 11:47 AM
yuri retitled this revision from New port: net/nncp: Utilities for secure store-and-forward files and mail exchanging {PENDING portmgr@ approval} to New port: net/nncp: Utilities for secure store-and-forward files and mail exchanging.Jan 29 2018, 4:07 PM
Closed by commit rP460314: New port: net/nncp: Utilities for secure store-and-forward files and mail… (authored by yuri). · Explain WhyJan 29 2018, 4:18 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 37356

View Options

net/Makefile

Loading...
View Options

net/nncp/

Loading...
View Options

net/nncp/Makefile

Loading...
View Options

net/nncp/distinfo

Loading...
View Options

net/nncp/files/

Loading...
View Options

net/nncp/files/nncp-caller.in

Loading...
View Options

net/nncp/files/nncp-daemon.in

Loading...
View Options

net/nncp/files/nncp-toss.in

Loading...
View Options

net/nncp/files/nncp.newsyslog.conf.sample

Loading...
View Options

net/nncp/files/pkg-deinstall.in

Loading...
View Options

net/nncp/files/pkg-install.in

Loading...
View Options

net/nncp/files/pkg-message.in

Loading...
View Options

net/nncp/pkg-descr

Loading...
View Options

net/nncp/pkg-plist

Loading...