On 23/1/2024 5:55 pm, Sebastian Huber wrote:
> ----- Am 22. Jan 2024 um 22:56 schrieb Chris Johns chr...@rtems.org:
> 
>> On 22/1/2024 6:18 pm, Sebastian Huber wrote:
>>> On 22.01.24 00:47, Chris Johns wrote:
>>>> On 22/1/2024 3:42 am, Sebastian Huber wrote:
>>>>> Does XCode ship a Symbolize.h and llvm-config?
>>>>
>>>> No ...
>>>>
>>>> % uname -a
>>>> Darwin weka.contemporary.net.au 23.2.0 Darwin Kernel Version 23.2.0: Wed 
>>>> Nov 15
>>>> 21:55:06 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T6020 arm64
>>>> % c++ -v
>>>> Apple clang version 15.0.0 (clang-1500.1.0.2.5)
>>>> Target: arm64-apple-darwin23.2.0
>>>> Thread model: posix
>>>> InstalledDir:
>>>> /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
>>>> % type llvm-config
>>>> llvm-config not found
>>>> % find
>>>> /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain
>>>> -name Symbolize.h
>>>> %
>>>
>>> Ok, thanks for checking. It seems that using the LLVM from brew is then the
>>> right approach.
>>
>> I have stated for years now we will not depend on brew or macports for 
>> building
>> RTEMS tools. Those distro packagers are rolling releases. Also what about
>> MacPorts?
>>
>> FYI when I first moved to Macs and I starting building native tools with
>> macports then brew and gave up. At the time our tool versions exposed issues 
>> in
>> their tools and updates broke working things and exposed new issues till the
>> point I found myself attempting to fix issue is those distros. Xcode tools 
>> just
>> worked and have done so for 20+ years.
> 
> What I meant with right approach is that using the brew provided LLVM for a 
> Github job is the right approach to test to RTEMS Tools build with LLVM. You 
> can of course build the RTEMS Tools with XCode only.

Thanks for explaining this. Sorry, I had misunderstood the comment.

> 
>>
>>>
>>> Can I check in the patch now?
>>>
>>
>> If the changes does not build for me without brew or macports then I am a no.
> 
> The usage of LLVM for the RTEMS Tools is optional. You can build them without 
> LLVM. However, this patch is relevant to a host which has LLVM installed. So, 
> to test this patch you need an LLVM installation.

Great and thanks. The change is fine to push however I like to understand some
things ...

Does this mean that tool does not work on some operating systems?

Does the user get any indication?

Chris
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to