Page MenuHomeFreeBSD

Update: sysutils/getssl to 2.10_p1
ClosedPublic

Authored by rainbow_purlinux.org on Nov 11 2018, 10:25 PM.

Details

Summary

Bug 232918 reported a missing nslookup dependency. Added and incremented PORTREVISION by 1

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

meta added a subscriber: meta.Tue, Feb 5, 2:07 AM

Overall, LGTM. See also bug 235351 and add ftp/curl.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=235351

sysutils/getssl/distinfo
1 ↗(On Diff #50294)

You need not to update TIMESTAMP if there's no change on the distfile.

Update to include ftp/curl

meta added inline comments.Wed, Feb 6, 2:50 AM
sysutils/getssl/Makefile
15 ↗(On Diff #53614)

curl:ftp/curl

Otherwise, LGTM.

meta added a reviewer: meta.Wed, Feb 6, 2:52 AM
mat added a comment.Wed, Feb 6, 8:49 AM

depending on dns/bind-tools is probably a bad idea, the port will

sysutils/getssl/Makefile
15 ↗(On Diff #53614)

Please try to put only one dependency per line.

meta added a comment.Thu, Feb 7, 2:22 AM
In D17954#408241, @mat wrote:

depending on dns/bind-tools is probably a bad idea, the port will

will what?

meta added a comment.Thu, Feb 7, 2:35 AM

Even if it is a bad idea, this port actually and already depends on dns/bind-tools (nslookup) just not declared in ports Makefile.
This diff just tries to declare dependencies correctly. Does ports tree have drop-in replacement of bind-tools?

Fix missing curl binary from ftp/curl package dependency

rainbow_purlinux.org marked 2 inline comments as done.Sat, Feb 9, 3:38 AM
rainbow_purlinux.org added inline comments.
sysutils/getssl/Makefile
15 ↗(On Diff #53614)

My bad, mistyped! Thanks for catching that :)

15 ↗(On Diff #53614)

@mat I'd be happy to format this appropriately if you can point me to the right documentation. I didn't see anything in the porter's handbook, but it's been a while since I had cause to poke at my ports (no reviewer or mentor anymore) and I've been insanely busy, which means I could simply be forgetting.

sysutils/getssl/distinfo
1 ↗(On Diff #50294)

I'll remember that for the future. Thanks!

meta accepted this revision.Mon, Feb 11, 2:06 PM

@mat doesn't reply dns/bind-tools WILL WHAT but LGTM.

depending on dns/bind-tools is probably a bad idea, the port will

This revision is now accepted and ready to land.Mon, Feb 11, 2:06 PM
meta added inline comments.Mon, Feb 11, 2:10 PM
sysutils/getssl/Makefile
15 ↗(On Diff #53614)

Use tab after RUN_DEPENDS=. That's OK right now. I'll fix it.

This revision was automatically updated to reflect the committed changes.