[PATCH] Fix r188763 - CLANG_ORDER_FILE defined as a BOOL instead as a STRING

[Edoardo P.]

Ok, updated the patch with FILEPATH.


2013/8/26 Jordan Rose <jordan_rose at apple.com>

> Most of our path variables point to directories or can be relative paths,
> so they have type PATH. I'm not sure CLANG_ORDER_FILE can be relative, and
> it has to point to a file that already exists, so using FILEPATH makes
> sense.
>
> On Aug 23, 2013, at 17:46 , "Edoardo P." <ed0.88.prez at gmail.com> wrote:
>
> I was not sure about it, because with a quick search through the cmake
> files I couldn't find another variable with said type. Should I still
> change it?
>
>
> 2013/8/24 Jordan Rose <jordan_rose at apple.com>
>
>> Actually, it should be a FILEPATH, shouldn't it?
>>
>> On Aug 23, 2013, at 16:00 , Edoardo P. <ed0.88.prez at gmail.com> wrote:
>>
>> > Hi,
>> >
>> > I noticed that in tools/clang/CMakeLists.txt CLANG_ORDER_FILE was
>> added, but it was defined with the cmake function 'option', which makes the
>> variable a BOOL, which is wrong. I noticed it when I got: 'error: cannot
>> open ON: No such file or directory' when linking clang executable. This
>> happened because I accidentally checked the 'option' CLANG_ORDER_FILE with
>> cmake-gui. Since it is a filename, it should be defined as a STRING with
>> 'set'. The attached patch should fix this.
>> >
>> > Regards,
>> > Edward-san
>> > --
>> > Mathematics is the language with which God has written the universe.
>> (Galilei)
>>
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>>
>>
>
>
> --
> Mathematics is the language with which God has written the universe.
> (Galilei)
>
>
>


-- 
Mathematics is the language with which God has written the universe.
(Galilei)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130826/c22806d0/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cmake_clang_order_file_fix.diff
Type: application/octet-stream
Size: 487 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130826/c22806d0/attachment.obj>