Loading ...
Sorry, an error occurred while loading the content.

Re: [PATCH] link.sh: do not reduce library if linker supports --as-needed

Expand Messages
  • Bram Moolenaar
    ... I don t know this link option, where it s supported or how well it works. If we want to use it we would need a configure check and somehow have the result
    Message 1 of 9 , Oct 1, 2010
    • 0 Attachment
      Kirill A. Shutemov wrote:

      > On Tue, Aug 31, 2010 at 09:29:57AM -0400, James Vega wrote:
      > > On Tue, Aug 31, 2010 at 8:30 AM, Kirill A. Shutemov
      > > <kirill@...> wrote:
      > > > On Wed, Aug 25, 2010 at 07:28:04AM -0400, James Vega wrote:
      > > >> On Wed, Aug 25, 2010 at 12:20:56PM +0300, Kirill A. Shutemov wrote:
      > > >> > On Mon, Aug 23, 2010 at 03:44:12PM +0300, Kirill A. Shutemov wrote:
      > > >> > > link.sh tries to avoid overlinking in a hackish way.
      > > >> > > At least GNU ld supports --as-needed which provides the same functionality
      > > >> > > at linker level. Let's use it.
      > > >> >
      > > >> > Any comments?
      > > >>
      > > >> Simply as a data point, I've been building Debian's Vim packages with
      > > >> -Wl,--as-needed for ~2.5 years now so I'd be interested in having this
      > > >> functionality done automatically when the proper tools are present.
      > > >
      > > > Seems, nobody cares. Sad.
      > >
      > > Just be patient. :) Bram appears to have been busy since the 7.3
      > > release. When he gets back to Vim stuff, you'll likely get a response
      > > then.
      >
      > Bram, any feedback?

      I don't know this link option, where it's supported or how well it
      works. If we want to use it we would need a configure check and somehow
      have the result used for link.sh. I haven't seen a patch for that.

      --
      hundred-and-one symptoms of being an internet addict:
      237. You tattoo your email address on your forehead.

      /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net \\\
      /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
      \\\ download, build and distribute -- http://www.A-A-P.org ///
      \\\ help me help AIDS victims -- http://ICCF-Holland.org ///

      --
      You received this message from the "vim_dev" maillist.
      Do not top-post! Type your reply below the text you are replying to.
      For more information, visit http://www.vim.org/maillist.php
    • Kirill A. Shutemov
      ... It works in both binutils linkers (bfd and gold). Some distributions use it by default for all packages. ALT Linux uses it since 2006. ... My patch checks
      Message 2 of 9 , Oct 1, 2010
      • 0 Attachment
        On Fri, Oct 01, 2010 at 09:21:23PM +0200, Bram Moolenaar wrote:
        >
        > Kirill A. Shutemov wrote:
        >
        > > On Tue, Aug 31, 2010 at 09:29:57AM -0400, James Vega wrote:
        > > > On Tue, Aug 31, 2010 at 8:30 AM, Kirill A. Shutemov
        > > > <kirill@...> wrote:
        > > > > On Wed, Aug 25, 2010 at 07:28:04AM -0400, James Vega wrote:
        > > > >> On Wed, Aug 25, 2010 at 12:20:56PM +0300, Kirill A. Shutemov wrote:
        > > > >> > On Mon, Aug 23, 2010 at 03:44:12PM +0300, Kirill A. Shutemov wrote:
        > > > >> > > link.sh tries to avoid overlinking in a hackish way.
        > > > >> > > At least GNU ld supports --as-needed which provides the same functionality
        > > > >> > > at linker level. Let's use it.
        > > > >> >
        > > > >> > Any comments?
        > > > >>
        > > > >> Simply as a data point, I've been building Debian's Vim packages with
        > > > >> -Wl,--as-needed for ~2.5 years now so I'd be interested in having this
        > > > >> functionality done automatically when the proper tools are present.
        > > > >
        > > > > Seems, nobody cares. Sad.
        > > >
        > > > Just be patient. :) Bram appears to have been busy since the 7.3
        > > > release. When he gets back to Vim stuff, you'll likely get a response
        > > > then.
        > >
        > > Bram, any feedback?
        >
        > I don't know this link option, where it's supported or how well it
        > works.

        It works in both binutils linkers (bfd and gold). Some distributions use
        it by default for all packages. ALT Linux uses it since 2006.

        > If we want to use it we would need a configure check and somehow
        > have the result used for link.sh. I haven't seen a patch for that.

        My patch checks if linker supports the option. Or do you want something
        else?

        --
        Kirill A. Shutemov

        --
        You received this message from the "vim_dev" maillist.
        Do not top-post! Type your reply below the text you are replying to.
        For more information, visit http://www.vim.org/maillist.php
      • Bram Moolenaar
        ... I missed the patch you originally sent. So it already includes a configure check. Seems to be OK, I ll have a closer look later. -- The amigos also
        Message 3 of 9 , Oct 2, 2010
        • 0 Attachment
          Kirill A. Shutemov wrote:

          > > > On Tue, Aug 31, 2010 at 09:29:57AM -0400, James Vega wrote:
          > > > > On Tue, Aug 31, 2010 at 8:30 AM, Kirill A. Shutemov
          > > > > <kirill@...> wrote:
          > > > > > On Wed, Aug 25, 2010 at 07:28:04AM -0400, James Vega wrote:
          > > > > >> On Wed, Aug 25, 2010 at 12:20:56PM +0300, Kirill A. Shutemov wrote:
          > > > > >> > On Mon, Aug 23, 2010 at 03:44:12PM +0300, Kirill A. Shutemov wrote:
          > > > > >> > > link.sh tries to avoid overlinking in a hackish way.
          > > > > >> > > At least GNU ld supports --as-needed which provides the same functionality
          > > > > >> > > at linker level. Let's use it.
          > > > > >> >
          > > > > >> > Any comments?
          > > > > >>
          > > > > >> Simply as a data point, I've been building Debian's Vim packages with
          > > > > >> -Wl,--as-needed for ~2.5 years now so I'd be interested in having this
          > > > > >> functionality done automatically when the proper tools are present.
          > > > > >
          > > > > > Seems, nobody cares. Sad.
          > > > >
          > > > > Just be patient. :) Bram appears to have been busy since the 7.3
          > > > > release. When he gets back to Vim stuff, you'll likely get a response
          > > > > then.
          > > >
          > > > Bram, any feedback?
          > >
          > > I don't know this link option, where it's supported or how well it
          > > works.
          >
          > It works in both binutils linkers (bfd and gold). Some distributions use
          > it by default for all packages. ALT Linux uses it since 2006.
          >
          > > If we want to use it we would need a configure check and somehow
          > > have the result used for link.sh. I haven't seen a patch for that.
          >
          > My patch checks if linker supports the option. Or do you want something
          > else?

          I missed the patch you originally sent. So it already includes a
          configure check. Seems to be OK, I'll have a closer look later.

          --
          "The amigos also appear to be guilty of not citing the work of others who had
          gone before them. Even worse, they have a chapter about modeling time and
          space without making a single reference to Star Trek!"
          (Scott Ambler, reviewing the UML User Guide)

          /// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net \\\
          /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
          \\\ download, build and distribute -- http://www.A-A-P.org ///
          \\\ help me help AIDS victims -- http://ICCF-Holland.org ///

          --
          You received this message from the "vim_dev" maillist.
          Do not top-post! Type your reply below the text you are replying to.
          For more information, visit http://www.vim.org/maillist.php
        Your message has been successfully submitted and would be delivered to recipients shortly.