[Buildroot] new <pkg>_CONFIG_FIXUP variable

["Stefan Fröberg" <stefan.froberg@petroprogram.com>]

Hi Arnout

9.1.2013 22:31, Arnout Vandecappelle kirjoitti:
> On 01/09/13 16:21, Stefan Fr?berg wrote:
>> Hi Thomas
>>
>> I tried that empty variable checking example you gave but it won't work.
>> If I remove DIVINE_CONFIG_FIXUP from divine.mk it will still spit the
>> "Fixing package configuration files" message
>>
>> This is what it looks so far. Added your variable checking and removed
>> redundant '\ from sed line as Arnout said.
>>
>>      if test -n $($(PKG)_CONFIG_FIXUP) ; then \
>
>  There should be quotes around "$($(PKG)_CONFIG_FIXUP)". test -n
> without arguments is true, test -n "" is false.
>
Ok

>>          $(call MESSAGE,"Fixing package configuration files") ;\
>>          for file in $($(PKG)_CONFIG_FIXUP); do \
>>              if [ -e $(STAGING_DIR)/usr/bin/$${file} ]; then \
>
>  This condition should be removed, as Thomas said.
>
>
>  Regards,
>  Arnout
>

Okay.

Here is the new version

    $(Q)if test -n "$($(PKG)_CONFIG_FIXUP)" ; then \
        $(call MESSAGE,"Fixing package configuration files") ;\
            $(SED)  "s,^prefix=.*,prefix=$(STAGING_DIR)/usr,g" \
                -e "s,^exec_prefix=.*,exec_prefix=$(STAGING_DIR)/usr,g" \
                $(addprefix
$(STAGING_DIR)/usr/bin/,$($(PKG)_CONFIG_FIXUP)) ;\
    fi

If nothing else comes up then I will make a new patch of this sometime
tomorrow

Regards
Stefan