[PATCH] defaults.mc: Remove any checks for objdump and objcopy

[Chris Johns]

On 1/7/21 12:31 am, Joel Sherrill wrote:
> On Wed, Jun 30, 2021 at 3:28 AM <chrisj at rtems.org> wrote:
>>
>> From: Chris Johns <chrisj at rtems.org>
>>
>> - FreeBSD is removing any dependence on binutils and release 13
>>   has remove objdump. This is fine as we build our own version.
> 
> removed. :)

Thanks.

> 
>> ---
>>  source-builder/defaults.mc | 5 ++---
>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/source-builder/defaults.mc b/source-builder/defaults.mc
>> index 8ed7003..98775e8 100644
>> --- a/source-builder/defaults.mc
>> +++ b/source-builder/defaults.mc
>> @@ -184,13 +184,12 @@ __mkdir:             exe,     required, '/bin/mkdir'
>>  __mkdir_p:           exe,     none,     '/bin/mkdir -p'
>>  __mv:                exe,     required, '/bin/mv'
>>  __nm:                exe,     required, '/usr/bin/nm'
>> -__objcopy:           exe,     optional, '/usr/bin/objcopy'
>> -__objdump:           exe,     optional, '/usr/bin/objdump'
>> +__objcopy:           exe,     none,     '/usr/bin/objcopy'
>> +__objdump:           exe,     none,     '/usr/bin/objdump'
> 
> Looks like you are also removing objcopy.
> 
> Why would we have ever needed the native versions of either?

We should not. It is hang over from the very start of the RSB when I translated
across the spec file defaults and that is focused on native build.

> And what's the alternative on FreeBSD if someone did need that
> functionality?

The maintainer doing the work put in the 13 release notes you use readelf and/or
llvm-objdump. I should point out this is removed from base so you can still
install binutils as a package using pkg.

Chris