[PATCH] dependency list for static libraries
Howard Chu
hyc@symas.com
Wed Oct 28 14:56:58 GMT 2020
Nick Clifton wrote:
> Hi Howard,
>
>> Sorry for the noise, this attached is the same as previous but with fixes
>> to whitespace / indentation.
>
> Thanks - the patch looks good now, although there are still a couple of minor
> issues - and one major issue:
>
> * The new test fails for the alpha-vms target. This is not serious however
> as almost all of the ar tests fail for this target. One day I will track
> down what is going wrong and either fix it, or arrange to skip these tests
> for alpha-vms.
>
> * There are a couple of minor formatting issues. Specifically: comments should
> end in a period followed by two spaces before the closing marker. /* Like this. */
> Plus function calls should have a space between the function name and the
> opening parenthesis of the argument list. like (this)
Whitespace fixes done, attached.
>
> The major issue is that I would really like for this extension to be supported
> by the LLVM community as well. It would be a shame to add it to the binutils
> only to have a different solution implemented there. I might have misread the
> emails but I believe that Fangrui still has some misgivings about this approach.
> Is that correct ? If so, then I would very much like to see them resolved
> before we commit the patch.
I haven't seen any further response from that side. Still seems obvious that a solution
that revolves around plaintext is easier to maintain than one that requires access to
the original source code+build environment and a recompile in order to modify/update.
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-ar-support-for-dependencies.patch
Type: text/x-patch
Size: 17173 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20201028/b4ea5ae7/attachment-0001.bin>
More information about the Binutils
mailing list