public inbox for elfutils@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mjw@redhat.com>
To: elfutils-devel@lists.fedorahosted.org
Subject: Re: [PATCH] Remove eu-ld and unused code.
Date: Thu, 07 Jul 2016 09:55:43 +0200	[thread overview]
Message-ID: <1467878143.8663.14.camel@redhat.com> (raw)
In-Reply-To: CAOPLpQcMXFYzYK4fD3X+XWyCNBwLSFtbucZQnHam7TeYsPbqjQ@mail.gmail.com

[-- Attachment #1: Type: text/plain, Size: 1617 bytes --]

Hi Ulrich,

On Wed, 2016-07-06 at 18:04 -0400, Ulrich Drepper wrote:
> On Wed, Jul 6, 2016 at 5:10 PM, Mark Wielaard <mjw@redhat.com> wrote:
> > Nobody has hacked on eu-ld in a very long time. It didn't really work.
> > And we didn't install it by default in the spec file. Remove sources,
> > the build rules and any (now) unused code.
> 
> Wrong.  I have worked on it and this is my code after all.  You just,
> without asking, claimed ownership anyway.  This is the only linker,
> I'm aware of, which is aimed to be used in library form which was one
> of the design decisions.

Yes, you did a lot of work on it. And yes, it does have an unique
design. Thanks. But your last commit to it was 8 years ago. Since then
people have only patched it up to fix build issues. There are no
testcases for it. And we do get reports from people who do install it
and find it not working. While it does have great potential, if nobody
is working on it or using it, then it is just a drag on maintenance.

The direct reason for removing it now is because various projects found
some use for the string table code which I wanted to rework. This code
is of course used a lot in the linker. But given that nobody actively
uses or works on the linker and we have no tests for it, it seemed
better to remove the (unused) code now than to pretend we keep what is
there working.

The code is of course not really gone. It will be in the git repository
history and can be resurrected once people actively work on it again.
All elfutils development is done openly through this mailinglist.

Cheers,

Mark

             reply	other threads:[~2016-07-07  7:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-07  7:55 Mark Wielaard [this message]
  -- strict thread matches above, loose matches on Subject: below --
2016-08-03 16:26 Mark Wielaard
2016-07-08 13:09 Mark Wielaard
2016-07-06 22:04 Ulrich Drepper
2016-07-06 21:10 Mark Wielaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1467878143.8663.14.camel@redhat.com \
    --to=mjw@redhat.com \
    --cc=elfutils-devel@lists.fedorahosted.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).