[PATCH] dependency list for static libraries

Nick Clifton nickc@redhat.com
Wed Nov 25 11:17:55 GMT 2020


Hi Howard,

>> This linker plugin works with both ld and gold.

> Ping: anything else needed for this, docs, etc.?

Sorry, sorry - my mind has been a mile away.

So - the code in the patch looks good.  I am running regression tests
now but I am not expecting to encounter any problems.  There are some
things missing however, the first of which is documentation as you
guessed:

   * A entry in ld/NEWS describing the new plugin.

   * Documentation in ld/ld.texi describing how to use the plugin,
     preferably with examples.

   * One or more new tests in linker's testsuite to check the
     functionality of the plugin.  Obviously these would have to be
     native only tests, but there are places in the testsuite
     where this is checked.  (Eg ld/testsuite/ld-bootstrap/bootstrap.exp)


One other thing - is the plugin installed into the correct location
if "make install" is run ?  (Ie:  ${libdir}/bfd-plugins).  I have not
actually checked this yet, so maybe it works, but I would have guessed
that some extra work would need to be done on ld/Makefile.am to manage
the installation.

Cheers
   Nick




More information about the Binutils mailing list