[Pki-devel] [PATCH] 35 Added option to build without Javadoc.
Matthew Harmsen
mharmsen at redhat.com
Mon Mar 26 16:28:50 UTC 2012
On 03/26/12 08:41, Endi Sukma Dewata wrote:
> On 3/23/2012 8:48 PM, Matthew Harmsen wrote:
>> On 03/23/12 12:53, Endi Sukma Dewata wrote:
>>> The build scripts have been modified to provide an option to build
>>> without Javadoc to speed up development builds. The option can be
>>> used as follows:
>>>
>>> compose_pki_core_packages --without-javadoc hybrid_rpms
>>>
>>> Ticket #111
>
>> CONDITIONAL ACK with CAVEATS:
>>
>> The following invocations succeeded building the RPMS without building
>> the javadoc RPMS (it should have failed with a Usage message):
>>
>> * pki/scripts/compose_pki_core_packages *--w* hybrid_rpms
>> * pki/scripts/compose_pki_core_packages *--without* hybrid_rpms (note
>> that '*--without*' was used instead of '*--without-javadoc*')
>>
>> while the following invocations failed as expected and issued a Usage
>> message:
>>
>> * pki/scripts/compose_pki_core_packages *--foo* hybrid_rpms
>> * pki/scripts/compose_pki_core_packages *--without-foo* hybrid_rpms
>> * pki/scripts/compose_pki_core_packages *--without-javadocs*
>> hybrid_rpms (note that '*--without-javadocs*' (plural) was used
>> instead of '*--without-javadoc*')
>>
>> *Please feel free to check-in this patch once this issue has been
>> corrected.*
>
> It seems to be a feature of the getopt. According to getopt man page:
>
> Long options may be abbreviated, as long as the abbreviation is
> not ambiguous.
>
> There's already a request to allow disabling abbreviation, but it
> won't be fixed:
>
> http://sourceware.org/bugzilla/show_bug.cgi?id=6863
>
> I've tested adding another option (e.g. --without-test). If you
> specify an abbreviated option (e.g. --without) it will find the first
> defined option that matches it (e.g. --without-javadoc) even thought
> it could be considered ambiguous.
>
okay -- looks like there is no intention to fix this.
Go ahead and check this in.
-- Matt
More information about the Pki-devel
mailing list