Bug 223120

Summary: net-im/libpurple Makefile suggestion for radio option for SSL, and Bonjour to be treated as a protocol
Product: Ports & Packages Reporter: sid
Component: Individual Port(s)Assignee: Joe Marcus Clarke <marcus>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (marcus)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   

Description sid 2017-10-19 22:10:13 UTC 
In this suggestion GNUTLS and NSS are moved to a radio (either or) option under SSL. Also, Bonjour behaves like a protocol, as it can be selected from the drop down menu, when running Pidgin, so I suggest moving it to the (net) protocols section. Thank you.
Here is the diff file:


--- Makefile	2017-10-19 17:00:26.306552000 -0500
+++ Makefile.edit	2017-10-19 16:59:30.248201000 -0500
@@ -41,13 +41,15 @@
 
 CPE_VENDOR=	pidgin
 
-OPTIONS_DEFINE=	BONJOUR DBUS GNUTLS NSS SASL GSTREAMER VV IDN PERL TCLTK
+OPTIONS_DEFINE=		DBUS SASL GSTREAMER VV IDN PERL TCLTK
+OPTIONS_RADIO=		SSL
+OPTIONS_RADIO_SSL=	NSS GNUTLS
 OPTIONS_GROUP=		PROTOCOLS
-OPTIONS_GROUP_PROTOCOLS=GG SAMETIME SILC IRC JABBER \
+OPTIONS_GROUP_PROTOCOLS=BONJOUR GG SAMETIME SILC IRC JABBER \
 			NOVELL OSCAR QQ SIMPLE ZEPHYR
 OPTIONS_DEFAULT=	BONJOUR DBUS NSS GSTREAMER VV IDN GG IRC JABBER \
 			NOVELL OSCAR QQ SIMPLE ZEPHYR
-BONJOUR_DESC=	mDNS support
+BONJOUR_DESC=	mDNS support and Bonjour protocol
 GNUTLS_DESC=	Use GNUTLS for encryption support
 NSS_DESC=	Use Mozilla NSS for encryption support
 SASL_DESC=	Cyrus SASL support (for jabberd)
Comment 1 sid 2017-10-20 02:13:02 UTC 
+OPTIONS_RADIO=		SSL
+OPTIONS_RADIO_SSL=	NSS GNUTLS

In this Makefile, both "SSL" should be replaced with "TLS", because SSL is a subset of TLS:

+OPTIONS_RADIO=		TLS
+OPTIONS_RADIO_TLS=	NSS GNUTLS
Comment 2 Joe Marcus Clarke freebsd_committer freebsd_triage 2017-12-15 13:24:43 UTC 
Committed, thanks!
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-12-15 13:24:52 UTC 
A commit references this bug:

Author: marcus
Date: Fri Dec 15 13:24:31 UTC 2017
New revision: 456394
URL: https://svnweb.freebsd.org/changeset/ports/456394

Log:
  Clean up the Makefile a bit.

  * Make BONJOUR a protocol option since it behaves like one
  * Add a radio button for TLS options NSS and GNUTLS

  PR:		223120
  Submitted by:	sid@bsdmail.com

Changes:
  head/net-im/libpurple/Makefile