Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2016-03-04 Thread Roland McGrath
We want things to continue to work on ELF files with e_machine codes for which we have no ebl port. So if you do that, you need to adapt all the users so that they do something useful when openbackend fails. It might be simpler to stick with the generic backend but add an ebl hook for "warning to

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-27 Thread Ben Gamari
Mark Wielaard writes: > The default openbackend should handle ebl == NULL just fine. > Then we just have to adjust the callers to either error out or just warn > about the unknown backend. > That would be one option for the particular problem that lead to this patch. However, I really think the

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-27 Thread Mark Wielaard
On Wed, 2015-11-25 at 23:51 +0100, Ben Gamari wrote: > Mark Wielaard writes: > > Yes, could you describe what kind of calls you were doing that made this > > really messy? > > > Well, much of the time I spent debugging wasn't even against GHC. I > spent quite a while trying to figure out why `read

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-25 Thread Ben Gamari
Mark Wielaard writes: > On Wed, 2015-11-25 at 20:03 +0100, Ben Gamari wrote: >> Josh Stone writes: >> >> > On 11/25/2015 04:45 AM, Ben Gamari wrote: >> >> This could have easily saved me three hours. > > Urgh, that is terrible. I am really sorry about this. > It's okay. This sort of thing comes

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-25 Thread Mark Wielaard
On Wed, 2015-11-25 at 20:03 +0100, Ben Gamari wrote: > Josh Stone writes: > > > On 11/25/2015 04:45 AM, Ben Gamari wrote: > >> This could have easily saved me three hours. Urgh, that is terrible. I am really sorry about this. We really should do a better job. One of the problems is that libebl

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-25 Thread Ben Gamari
Josh Stone writes: > On 11/25/2015 04:45 AM, Ben Gamari wrote: >> This could have easily saved me three hours. >> --- >> libebl/eblopenbackend.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/libebl/eblopenbackend.c b/libebl/eblopenbackend.c >> index b301400..6628

Re: [PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-25 Thread Josh Stone
On 11/25/2015 04:45 AM, Ben Gamari wrote: > This could have easily saved me three hours. > --- > libebl/eblopenbackend.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libebl/eblopenbackend.c b/libebl/eblopenbackend.c > index b301400..662842d 100644 > --- a/libebl/eblo

[PATCH] eblopenback: Warn when unable to find ebl backend

2015-11-25 Thread Ben Gamari
This could have easily saved me three hours. --- libebl/eblopenbackend.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libebl/eblopenbackend.c b/libebl/eblopenbackend.c index b301400..662842d 100644 --- a/libebl/eblopenbackend.c +++ b/libebl/eblopenbackend.c @@ -376,7 +376,