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

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

Expand Messages
  • Kirill A. Shutemov
    ... Bram, any feedback? -- Kirill A. Shutemov -- You received this message from the vim_dev maillist. Do not top-post! Type your reply below the text you are
    Message 1 of 9 , Oct 1, 2010
      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?

      --
      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 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 2 of 9 , Oct 1, 2010
        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 3 of 9 , Oct 1, 2010
          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 4 of 9 , Oct 2, 2010
            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.