svn commit: r358666 - head/Mk

[Eitan Adler]

On 20 June 2014 23:53, olli hauer <ohauer at gmx.de> wrote:
> On 2014-06-21 08:39, Eitan Adler wrote:> Author: eadler
>> Date: Sat Jun 21 06:39:32 2014
>> New Revision: 358666
>> URL: http://svnweb.freebsd.org/changeset/ports/358666
>> QAT: https://qat.redports.org/buildarchive/r358666/
>>
>> Log:
>>   Expand upon option descriptions whose only text are 'option support'.  Such
>>   descriptions provide no value for the user and should be expanded upon.
>>
>>   While here make one description fit in space limit
>>
>>   Reviewed by:        marino
>>
>> Modified:
>>   head/Mk/bsd.options.desc.mk
>>
>> Modified: head/Mk/bsd.options.desc.mk
>> ==============================================================================
>> --- head/Mk/bsd.options.desc.mk       Sat Jun 21 06:21:14 2014        (r358665)
>> +++ head/Mk/bsd.options.desc.mk       Sat Jun 21 06:39:32 2014        (r358666)
>
>> -LDAP_DESC?=          LDAP support
>> +LCMS2_DESC?=         Little CMS 2.x support
>> +LDAP_DESC?=          LDAP authentication support
>>  LDAPS_DESC?=         LDAPS support
>
>
> Why the change for LDAP?

The same reason for all of them: the format "OPTION_DESC?= OPTION
support" provides no value to the user.  The option description should
provide additional context beyond the name.  If the option is
overloaded it should be split apart.

> The old DESC was about the protocol where the new one suggest a functionality.

This was based on a quick grep-and-check of the relevent ports.  It
was not exhaustive.  Maintainers that dislike the current text can
override it locally in their port.

> LDAP can be used as simple database for a dozen of different things then authentication.

Feel free to suggest (or just commit) alternatives.

-- 
Eitan Adler
Source, Ports, Doc committer
Bugmeister, Ports Security teams