Pass PYTHON=$(PYTHON) to gmake when building seabios. This fixes seabios build error ''python not found'' along with the patches from Kevin O''Connor. Signed-off-by: Christoph Egger <Christoph.Egger@amd.com> -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote:> Pass PYTHON=$(PYTHON) to gmake when building seabios. > This fixes seabios build error > ''python not found'' > along with the patches from Kevin O''Connor. > > Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>Acked-by: Ian Campbell <ian.campbell@citrix.com>
On 02/09/12 12:59, Ian Campbell wrote:> On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote: >> Pass PYTHON=$(PYTHON) to gmake when building seabios. >> This fixes seabios build error >> ''python not found'' >> along with the patches from Kevin O''Connor. >> >> Signed-off-by: Christoph Egger<Christoph.Egger@amd.com> > > Acked-by: Ian Campbell<ian.campbell@citrix.com> >Please apply this patch and update SeaBIOS. Keven O''Connors patches went upstream on Feb 8th. Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override
$(PYTHON)"):> On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote:
> > Pass PYTHON=$(PYTHON) to gmake when building seabios.
> > This fixes seabios build error
> > ''python not found''
> > along with the patches from Kevin O''Connor.
> >
> > Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
Thanks for this. I''m going to hold off on this though for the moment
in case it conflicts with Roger''s autoconf patch. Also I''m
afraid it
needs a refresh anyway.
Thanks,
Ian.
On Mon, 2012-02-20 at 10:37 +0000, Christoph Egger wrote:> On 02/09/12 12:59, Ian Campbell wrote: > > On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote: > >> Pass PYTHON=$(PYTHON) to gmake when building seabios. > >> This fixes seabios build error > >> ''python not found'' > >> along with the patches from Kevin O''Connor. > >> > >> Signed-off-by: Christoph Egger<Christoph.Egger@amd.com> > > > > Acked-by: Ian Campbell<ian.campbell@citrix.com> > > > > Please apply this patch and update SeaBIOS. > Keven O''Connors patches went upstream on Feb 8th.I was hoping to track SeaBIOS stable branches so I am reluctant to simply update to a random commit on the development branch. Currently we are tracking the upstream 1.6.3-stable branch in the xen-unstable branch of our seabios.git. In hindsight this might have been an error since it means that our branch will only be non-rebasing until we switch to 1.6.4 or 1.7.0 (whichever comes next). I''m not sure how best to approach this, obviously I could create a 1.6.3-stable-xen branch and backport your fix to it. I''d like to decide what approach I should take when the upstream latest-stable branch changes first though. My immediate thought is that I should remove the "xen-unstable" branch from our tree and instead push upstream''s 1.6.3-stable branch and then push my 1.6.3-stable-xen branch and switch to that. When new stable branch X.Y.Z-stable happens upstream I will simply create a new X.Y.Z-stable-xen based on it. However I am not sure what to do with the "master" branch of our tree. Previously I just omitted it since it has no real meaning but that caused confusion and people wanted me to put something there. I don''t want to track the upstream devel branch since I don''t want users who clone our tree to think we support that as is. I don''t want to push the currently supported stable branch because that would necessarily make master a rebasing branch. The xen-unstable build system itself uses explicit commit numbers or tags so isn''t really effected. IanJ -- what do you think? Ian.
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override
$(PYTHON)"):> However I am not sure what to do with the "master" branch of our
tree.
> Previously I just omitted it since it has no real meaning but that
> caused confusion and people wanted me to put something there.
>
> I don''t want to track the upstream devel branch since I
don''t want users
> who clone our tree to think we support that as is. I don''t want to
push
> the currently supported stable branch because that would necessarily
> make master a rebasing branch.
>
> The xen-unstable build system itself uses explicit commit numbers or
> tags so isn''t really effected.
>
> IanJ -- what do you think?
Why not make "master" be equal to the SEABIOS_UPSTREAM_TAG in
xen-unstable''s Config.mk ? That''s what I do with
qemu-xen-unstable.git.
Ian.
On Tue, 21 Feb 2012, Ian Campbell wrote:> On Mon, 2012-02-20 at 10:37 +0000, Christoph Egger wrote: > > On 02/09/12 12:59, Ian Campbell wrote: > > > On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote: > > >> Pass PYTHON=$(PYTHON) to gmake when building seabios. > > >> This fixes seabios build error > > >> ''python not found'' > > >> along with the patches from Kevin O''Connor. > > >> > > >> Signed-off-by: Christoph Egger<Christoph.Egger@amd.com> > > > > > > Acked-by: Ian Campbell<ian.campbell@citrix.com> > > > > > > > Please apply this patch and update SeaBIOS. > > Keven O''Connors patches went upstream on Feb 8th. > > I was hoping to track SeaBIOS stable branches so I am reluctant to > simply update to a random commit on the development branch. > > Currently we are tracking the upstream 1.6.3-stable branch in the > xen-unstable branch of our seabios.git. In hindsight this might have > been an error since it means that our branch will only be non-rebasing > until we switch to 1.6.4 or 1.7.0 (whichever comes next). > > I''m not sure how best to approach this, obviously I could create a > 1.6.3-stable-xen branch and backport your fix to it. I''d like to decide > what approach I should take when the upstream latest-stable branch > changes first though.Have you tried to ask Kevin to backport the commit to stable?
On Tue, 2012-02-21 at 12:34 +0000, Ian Jackson wrote:> Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override $(PYTHON)"): > > However I am not sure what to do with the "master" branch of our tree. > > Previously I just omitted it since it has no real meaning but that > > caused confusion and people wanted me to put something there. > > > > I don''t want to track the upstream devel branch since I don''t want users > > who clone our tree to think we support that as is. I don''t want to push > > the currently supported stable branch because that would necessarily > > make master a rebasing branch. > > > > The xen-unstable build system itself uses explicit commit numbers or > > tags so isn''t really effected. > > > > IanJ -- what do you think? > > Why not make "master" be equal to the SEABIOS_UPSTREAM_TAG in > xen-unstable''s Config.mk ? That''s what I do with > qemu-xen-unstable.git.That would make master rebasing. I''m happy to do that if there is consensus that it is ok. Ian.
On Tue, 2012-02-21 at 13:30 +0000, Stefano Stabellini wrote:> On Tue, 21 Feb 2012, Ian Campbell wrote: > > On Mon, 2012-02-20 at 10:37 +0000, Christoph Egger wrote: > > > On 02/09/12 12:59, Ian Campbell wrote: > > > > On Thu, 2012-02-09 at 11:27 +0000, Christoph Egger wrote: > > > >> Pass PYTHON=$(PYTHON) to gmake when building seabios. > > > >> This fixes seabios build error > > > >> ''python not found'' > > > >> along with the patches from Kevin O''Connor. > > > >> > > > >> Signed-off-by: Christoph Egger<Christoph.Egger@amd.com> > > > > > > > > Acked-by: Ian Campbell<ian.campbell@citrix.com> > > > > > > > > > > Please apply this patch and update SeaBIOS. > > > Keven O''Connors patches went upstream on Feb 8th. > > > > I was hoping to track SeaBIOS stable branches so I am reluctant to > > simply update to a random commit on the development branch. > > > > Currently we are tracking the upstream 1.6.3-stable branch in the > > xen-unstable branch of our seabios.git. In hindsight this might have > > been an error since it means that our branch will only be non-rebasing > > until we switch to 1.6.4 or 1.7.0 (whichever comes next). > > > > I''m not sure how best to approach this, obviously I could create a > > 1.6.3-stable-xen branch and backport your fix to it. I''d like to decide > > what approach I should take when the upstream latest-stable branch > > changes first though. > > Have you tried to ask Kevin to backport the commit to stable?Not yet, I was more interested in the general problem than this specific commit right at the moment. Ian.
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override
$(PYTHON)"):> On Tue, 2012-02-21 at 12:34 +0000, Ian Jackson wrote:
> > Why not make "master" be equal to the SEABIOS_UPSTREAM_TAG
in
> > xen-unstable''s Config.mk ? That''s what I do with
> > qemu-xen-unstable.git.
>
> That would make master rebasing. I''m happy to do that if there is
> consensus that it is ok.
Wait, you''re rewinding SEABIOS_UPSTREAM_TAG ? That''s quite
exciting.
Ian.
On Tue, 2012-02-21 at 15:14 +0000, Ian Jackson wrote:> Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override $(PYTHON)"): > > On Tue, 2012-02-21 at 12:34 +0000, Ian Jackson wrote: > > > Why not make "master" be equal to the SEABIOS_UPSTREAM_TAG in > > > xen-unstable''s Config.mk ? That''s what I do with > > > qemu-xen-unstable.git. > > > > That would make master rebasing. I''m happy to do that if there is > > consensus that it is ok. > > Wait, you''re rewinding SEABIOS_UPSTREAM_TAG ? That''s quite exciting.SEABIOS_UPSTREAM_TAG is tracking an upstream stable branch. When they release a new stable release we will want to switch to that, but that will not be a fast-forward. Ian.
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override
$(PYTHON)"):> On Tue, 2012-02-21 at 15:14 +0000, Ian Jackson wrote:
> > Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios:
override $(PYTHON)"):
> > > On Tue, 2012-02-21 at 12:34 +0000, Ian Jackson wrote:
> > > > Why not make "master" be equal to the
SEABIOS_UPSTREAM_TAG in
> > > > xen-unstable''s Config.mk ? That''s what I
do with
> > > > qemu-xen-unstable.git.
> > >
> > > That would make master rebasing. I''m happy to do that if
there is
> > > consensus that it is ok.
> >
> > Wait, you''re rewinding SEABIOS_UPSTREAM_TAG ?
That''s quite exciting.
>
> SEABIOS_UPSTREAM_TAG is tracking an upstream stable branch. When they
> release a new stable release we will want to switch to that, but that
> will not be a fast-forward.
Hmm. Well as to the question of what "master" should be:
If it''s anything useful at all, it needs to be what you would need to
edit to change the result of the Xen build. So either it must be
.._TAG or it must be some fast-forwarding branch constructed from the
series of _TAG values. (Ie, a branch whose commits correspond 1:1
with _TAG values, each commit on the master branch having the previous
master commit as its left parent and the _TAG, which determines the
contents, as the right parent.)
Ian.
On Tue, 2012-02-21 at 15:31 +0000, Ian Jackson wrote:> Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override $(PYTHON)"): > > On Tue, 2012-02-21 at 15:14 +0000, Ian Jackson wrote: > > > Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override $(PYTHON)"): > > > > On Tue, 2012-02-21 at 12:34 +0000, Ian Jackson wrote: > > > > > Why not make "master" be equal to the SEABIOS_UPSTREAM_TAG in > > > > > xen-unstable''s Config.mk ? That''s what I do with > > > > > qemu-xen-unstable.git. > > > > > > > > That would make master rebasing. I''m happy to do that if there is > > > > consensus that it is ok. > > > > > > Wait, you''re rewinding SEABIOS_UPSTREAM_TAG ? That''s quite exciting. > > > > SEABIOS_UPSTREAM_TAG is tracking an upstream stable branch. When they > > release a new stable release we will want to switch to that, but that > > will not be a fast-forward. > > Hmm. Well as to the question of what "master" should be: > > If it''s anything useful at all,(which I''m not personally convinced of, but I seem to be in the minority so I shall try and make it mean something)> it needs to be what you would need to > edit to change the result of the Xen build. So either it must be > .._TAGI''ll choose this option. I''ll also edit the repo description to make it clear that all branches are rebasing.> or it must be some fast-forwarding branch constructed from the > series of _TAG values. (Ie, a branch whose commits correspond 1:1 > with _TAG values, each commit on the master branch having the previous > master commit as its left parent and the _TAG, which determines the > contents, as the right parent.)I think I''d inevitably end up cocking this scheme up, especially given the expected frequency of seabios updates (e.g. v. low). Ian.
On Tue, 2012-02-21 at 16:17 +0000, Ian Campbell wrote:> > it needs to be what you would need to > > edit to change the result of the Xen build. So either it must be > > .._TAG > > I''ll choose this option. I''ll also edit the repo description to make it > clear that all branches are rebasing. > > > or it must be some fast-forwarding branch constructed from the > > series of _TAG values. (Ie, a branch whose commits correspond 1:1 > > with _TAG values, each commit on the master branch having the previous > > master commit as its left parent and the _TAG, which determines the > > contents, as the right parent.) > > I think I''d inevitably end up cocking this scheme up, especially given > the expected frequency of seabios updates (e.g. v. low).... actually I don''t need to pick between the two until this actually happens. When it does happen I''ll play with the second option and determine if I am capable of not making a mess of it etc. For now I have cherry-picked the upstream fix onto a new branch "1.6.3-stable-xen" and pushed that branch to our repo. I have also updated the "xen-unstable" branch (the default branch) with the same. IanJ -- when you are ready to push the xen-unstable.hg side please include an update of SEABIOS_UPSTREAM_TAG to 002d30b5f4f48ee203be3bad9d68dc8b538ee35b (instead of rel-1.6.3.1 as it is currently). Ian.
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override
$(PYTHON)"):> IanJ -- when you are ready to push the xen-unstable.hg side please
> include an update of SEABIOS_UPSTREAM_TAG to
> 002d30b5f4f48ee203be3bad9d68dc8b538ee35b (instead of rel-1.6.3.1 as it
> is currently).
I think each side of this change is harmless independently, so you can
go ahead and do that whenever you think it right. But perhaps we
should wait with this and the $(PYTHON)-passing patch until we get the
tree building again.
Ian.
On Wed, 2012-02-22 at 11:58 +0000, Ian Jackson wrote:> Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools/seabios: override $(PYTHON)"): > > IanJ -- when you are ready to push the xen-unstable.hg side please > > include an update of SEABIOS_UPSTREAM_TAG to > > 002d30b5f4f48ee203be3bad9d68dc8b538ee35b (instead of rel-1.6.3.1 as it > > is currently). > > I think each side of this change is harmless independently, so you can > go ahead and do that whenever you think it right. But perhaps we > should wait with this and the $(PYTHON)-passing patch until we get the > tree building again.Ack on waiting. Ian.
On 02/22/12 12:53, Ian Campbell wrote:> On Tue, 2012-02-21 at 16:17 +0000, Ian Campbell wrote: >>> it needs to be what you would need to >>> edit to change the result of the Xen build. So either it must be >>> .._TAG >> >> I''ll choose this option. I''ll also edit the repo description to make it >> clear that all branches are rebasing. >> >>> or it must be some fast-forwarding branch constructed from the >>> series of _TAG values. (Ie, a branch whose commits correspond 1:1 >>> with _TAG values, each commit on the master branch having the previous >>> master commit as its left parent and the _TAG, which determines the >>> contents, as the right parent.) >> >> I think I''d inevitably end up cocking this scheme up, especially given >> the expected frequency of seabios updates (e.g. v. low). > > ... actually I don''t need to pick between the two until this actually > happens. When it does happen I''ll play with the second option and > determine if I am capable of not making a mess of it etc. > > For now I have cherry-picked the upstream fix onto a new branch > "1.6.3-stable-xen" and pushed that branch to our repo. I have also > updated the "xen-unstable" branch (the default branch) with the same. > > IanJ -- when you are ready to push the xen-unstable.hg side please > include an update of SEABIOS_UPSTREAM_TAG to > 002d30b5f4f48ee203be3bad9d68dc8b538ee35b (instead of rel-1.6.3.1 as it > is currently).There is still one fix missing from upstream, namely: 805ede2bd35243a4298bc64bd81be6db7cf57f58 Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632
On Wed, 2012-02-22 at 12:33 +0000, Christoph Egger wrote:> On 02/22/12 12:53, Ian Campbell wrote: > > IanJ -- when you are ready to push the xen-unstable.hg side please > > include an update of SEABIOS_UPSTREAM_TAG to > > 002d30b5f4f48ee203be3bad9d68dc8b538ee35b (instead of rel-1.6.3.1 as it > > is currently). > > There is still one fix missing from upstream, namely: > 805ede2bd35243a4298bc64bd81be6db7cf57f58I''ve applied this one as well and updated SEABIOS_UPSTREAM_TAG to pull in both. Thanks, Ian.
Seemingly Similar Threads
- [PATCH v10 0/7] build upstream qemu and seabios by default
- [PATCH v2 0/3] hvmloader: Make ROM dependencies optional
- [PATCH v3 0/5] hvmloader: Make ROM dependencies optional
- [PATCH] seabios: update to 1.6.3.1 release
- libxl: error: libxl_dm.c:1212:device_model_spawn_outcome: domain 1 device model: spawn failed (rc=-3) when creating VM using upstream qemu on Xen 4.2.