[pacman-dev] [PATCH] makepkg: make strip options configurable

[Cedric Staniewski]

On 12/14/2009 02:42 AM, Allan McRae wrote:
> However, what is this change?
> 
>> diff --git a/doc/Makefile.am b/doc/Makefile.am
>> index fcbcab8..ce1adab 100644
>> --- a/doc/Makefile.am
>> +++ b/doc/Makefile.am
> ...
> 
>> @@ -93,8 +99,12 @@ A2X_OPTS = \
>>      --xsltproc-opts='-param man.endnotes.list.enabled 0 -param
>> man.endnotes.are.numbered 0'
>>  
>>  # These rules are due to the includes and files of the asciidoc text
>> -$(ASCIIDOC_MANS): asciidoc.conf footer.txt
>> -    a2x $(A2X_OPTS) --asciidoc-opts="$(ASCIIDOC_OPTS)" $@.txt
>> +$(ASCIIDOC_MANS): asciidoc.conf footer.txt Makefile
>> +    @echo GEN $@;
>> +    @rm -f $@.tmp
>> +    @test -f $(srcdir)/$@.txt && $(edit) $(srcdir)/$@.txt >$@.tmp ||
>> true
>> +    a2x $(A2X_OPTS) --asciidoc-opts="$(ASCIIDOC_OPTS)" $@.tmp
>> +    @rm -f $@.tmp
>>  
>>  %.html: %.txt
>>      asciidoc $(ASCIIDOC_OPTS) -a linkcss $*.txt

It is needed to get the correct default values in makepkg.conf man page:

> +**STRIP_BINARIES=**"@STRIP_BINARIES@"::
> [...]
> +**STRIP_SHARED=**"@STRIP_SHARED@"::
> [...]
> +**STRIP_STATIC=**"@STRIP_STATIC@"::

There are no os-dependent values in the documentation yet, so I had to
add these lines to be able to replace strings with actual values. It
only makes a difference on darwin, though.