[LinuxBIOS] Allow user to specify pathnames for create and add

Stefan Reinauer stepan at coresystems.de
Mon Aug 20 16:40:30 CEST 2007


Dear Jordan,

* Jordan Crouse <jordan.crouse at amd.com> [070820 16:19]:
> On 20/08/07 02:01 +0200, Stefan Reinauer wrote:
> > jordan.crouse at amd.com wrote:
> > > Add another field to the filename specified for create and add operations
> > > to specify the intended pathname for the blob.
> > >
> > > Signed-off-by: Jordan Crouse <jordan.crouse at amd.com>
> > This patch breaks the produced image. Please provide a fix.
> 
> Not sure what you mean - but this code has suffered from too much bit rot,
> so I'm going to withdraw all of them.  Better luck next time, I guess. :)

No, it seems it's not a problem of bit rot, that code has not been
touched in the repository between the time you made the changes and the
time it got checked in. It just seems to be a corner case that is not
fully working yet with the new code base. I narrowed the problem down to
the util/lar/lib.c part of the patch which causes all modules to be
compressed, even those with a nocompress: tag.

It's too late for a withdraw now since your fine piece of work is
already in the v3 repository since yesterday night ;-) Please don't ask
me to remove it again. I believe this is a significant step forward on
our road to v3.

I want to apologize for the long delay since you provided the code - The
final version of the patches have been sitting in my inbox and on my
todo list since July 23rd. Time has been a very tight resource for me,
just like for many other core LinuxBIOS developers lately. We're trying
as good as we can to change that.

Let's try to get things going again so we don't lose the momentum we had
in Ottawa!

Best wishes,

Stefan

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/




More information about the coreboot mailing list