Re: [Qemu-devel] [PATCH] sun4m: Add FCode ROM for TCX framebuffer

[Peter Maydell]

On 21 August 2013 15:44, Mark Cave-Ayland <address@hidden> wrote:
> On 20/08/13 23:41, Peter Maydell wrote:
>> Shouldn't this blob come in the same patch as an update to some
>> git module, so that we keep track of the sources used to build
>> the blob?

> Unfortunately the OpenBIOS repository is still based in SVN :(  There is a
> git-svn mirror on git.qemu.org, but currently it needs to be manually
> updated and so is generally not particularly helpful. For the 1.6 release I
> got Anthony to manually update the repository on git.qemu.org so that the
> git submodule reference was updated as part of the pull request.
>
> The main reason for not updating the git submodule in this particular commit
> is because this is actually a precursor to another larger sun4m framebuffer
> patch, and once both these patches are hopefully accepted then my plan is to
> send a single pull request to update all 3 of the OpenBIOS images at the
> same time rather than to split architectures across different OpenBIOS
> versions.

In that case we should defer adding this fcode rom blob until then.
I really don't like the idea of having random binary blobs in our git
repo (and thus in our release tarballs) which aren't tied exactly to
the sources you need to rebuild them.

(It's the pain of managing this which is why I don't like binary blobs
at all, in fact. But they're a fact of life given that mostly we don't have
easy cross-compilation setups  :-( )

-- PMM