Page MenuHomeFreeBSD

[New Port] sysutils/mping
ClosedPublic

Authored by dteske on Jun 17 2018, 11:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 8:00 PM
Unknown Object (File)
Sun, Mar 17, 7:59 PM
Unknown Object (File)
Sun, Mar 17, 7:48 PM
Subscribers

Details

Summary

mping is a utility for pinging multiple hosts.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bapt requested changes to this revision.Jun 18 2018, 4:25 AM
bapt added a subscriber: bapt.
bapt added inline comments.
sysutils/mping/Makefile
23 ↗(On Diff #43981)

Why this line, I don't think it is actually needed

26 ↗(On Diff #43981)

This is not needed as well, automatically created by the mtree file when creating the stage directory

This revision now requires changes to proceed.Jun 18 2018, 4:25 AM
sysutils/mping/pkg-descr
1 ↗(On Diff #43990)

pkg-descr should be more descriptive than COMMENT.

Make pkg-descr more descriptive

mat added inline comments.
sysutils/mping/pkg-descr
4–5 ↗(On Diff #44040)

I do not think the FreeBSD ports tree is used on MacOSX or Linux, so the end may not be of any interest here.

sysutils/mping/pkg-descr
4–5 ↗(On Diff #44040)

At work we just pick a FreeBSD box and copy it to our mac from that machine. So we use FreeBSD as a means to an end. I don't see anything wrong with that, but if you are intent on removing that bit, I will.

sysutils/mping/pkg-descr
4–5 ↗(On Diff #44040)

I don't see anything wrong, I approved the review.

I just added a small comment about a bit I felt was not needed.

Trim pkg-descr for relevancy

Add sysutils/Makefile modification

To prevent the "This revision was not accepted when it landed; it landed in state Needs Review." moniker being attached to this review, can you re-approve?

I wouldn't worry about it...

This revision was not accepted when it landed; it landed in state Needs Review.Jun 27 2018, 6:35 PM
Closed by commit rP473453: [New Port] sysutils/mping (authored by dteske). · Explain Why
This revision was automatically updated to reflect the committed changes.
In D15875#339613, @imp wrote:

I wouldn't worry about it...

Derp, you're right. It ended up getting that tag anyway because of bapt's request for changes (which were addressed but bapt didn't come back around to approve). I consider that a short-coming of phabricator that the blocking status of a request for changes doesn't get released when the action is taken. Blah.

There is a PKGBASE collision between sysutils/mping and net/mping.
PKGNAMEPREFIX or PKGNAMESUFFIX has to be added.

Been thinking about this for the past day. I can't think of any suitable value for either prefix or suffix. I'm open to suggestions -- perhaps you've delt with this in the past and have a suggestion for suitable value when this occurs.

It could be PKGNAMESUFFIX= -shell or PKGNAMESUFFIX= -fraubsd

It could be PKGNAMESUFFIX= -shell or PKGNAMESUFFIX= -fraubsd

https://reviews.freebsd.org/D16106

@antoine can you accept so I can close?

I think this is a bug in phabricator. The review was in "closed" state and committed. Then a comment caused it to go back into "Needs Review". The person that added the comment (antoine) even came back and accepted and it still shows as "Needs Review". Can't close this ticket until it is in approved. I'd rather not use the sledgehammer of removing all the reviewers.

Let me reach out to bapt and, imp, and mat to see if their approval can get it out of the "Needs Review" state (which it shouldn't be in because it was previously in "Closed; committed" state prior to comment.

imp removed a reviewer: imp.

Steeling

this has been committed and got into a weird state.

imp accepted this revision.
dteske removed a reviewer: dteske.
This revision is now accepted and ready to land.Jul 19 2018, 6:25 PM

Removed bapt and closed (was already committed but not closed because phab bugette)