[llvm-commits] [PROPOSAL] Add --program-prefix support to build

[Rick Foos]

Here are patches to llvm, and test-suite to add --program-prefix support.

A few small changes are needed to pass the program prefix value and add 
the prefix to binary files that are installed.

test-suite has one binary install step, and a similar change is done there.

Polly has install steps, but it does not install binary files and 
doesn't need this change.

-rick



On 09/24/2012 05:54 PM, Rick Foos wrote:
> I'll pull config.guess from the patch. There are currently three 
> different versions of config.guess in the tree.
>
>
> On 09/24/2012 03:13 PM, Jim Grosbach wrote:
>> Updating config.guess should probably be a separate patch, as it's 
>> unrelated to the core change you're wanting. I gather the new version 
>> is still OK license-wise?
>>
> Yes, the GPL license exception is still in place.
>
> # As a special exception to the GNU General Public License, if you
> # distribute this file as part of a program that contains a
> # configuration script generated by Autoconf, you may include it under
> # the same distribution terms that you use for the rest of that program.
>
>> No comment on the guts of the patch (I'm not familiar w/ how/why it 
>> failed before, so not really qualified to weigh in there).
>>
>> -Jim
>>
>> On Sep 24, 2012, at 1:03 PM, Sebastian Pop<spop at codeaurora.org>  wrote:
>>
>>> Hi,
>>>
>>> I would like to submit this patch again: the first time we had to 
>>> revert it
>>> because some buildbots started failing.  Rick has then set up a 
>>> dragonegg bot to
>>> test and fix the problems.  I think that we can try to push the 
>>> patch once more,
>>> now that the problem is fixed.  If there are no objections, I will 
>>> be pushing
>>> this and follow the build bots activity on IRC.
>>>
>>> Sebastian
>>>
>>> Rick Foos wrote:
>>>> On 09/20/2012 10:52 PM, Rick Foos wrote:
>>>>> This patch adds --program-prefix support to the autoconf build.
>>>>>
>>>>> The cmake build isn't changed.
>>>>>
>>>>> makefile.rules now has program_prefix from configure. Some install
>>>>> rules need it with Destdir.
>>>>>
>>>>> In configure.ac, A trap for the program_prefix = NONE before
>>>>> AC_SUBST is needed. Configure ususally doesn't export
>>>>> program-<prefix|suffix|transform>  variables, and the NONE default
>>>>> value can escape w/o a trap.
>>>>>
>>>>> config.guess is updated to 2012-08-14. No platforms are removed.
>>>>> Hexagon is added.
>>>>>
>>>>> --program-suffix wasn't included as it might interfere with the
>>>>> EXEEXT variables, and goes beyond the scope of this patch.
>>>>>
>>>>> Thank you,
>>>>> -rick
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> llvm-commits mailing list
>>>>> llvm-commits at cs.uiuc.edu
>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>
>


-- 
Rick Foos
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-program-prefix-support-to-build.patch
Type: text/x-patch
Size: 5514 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120925/365f2623/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-TEST-SUITE-Add-program-prefix-support-to-build.patch
Type: text/x-patch
Size: 13806 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120925/365f2623/attachment-0001.bin>