[coreboot] [PATCH] Use a tag for SeaBIOS stable checkouts.
Kevin O'Connor
kevin at koconnor.net
Tue Jan 25 03:58:03 CET 2011
On Tue, Jan 25, 2011 at 02:37:03AM +0100, Stefan Reinauer wrote:
> * Peter Stuge <peter at stuge.se> [110124 00:35]:
> > > +TAG-$(CONFIG_SEABIOS_MASTER)=origin/master
> > > TAG-$(CONFIG_SEABIOS_STABLE)=rel-0.6.1.3
> > ..
> > > checkout:
> > > echo "Checking out SeaBIOS $(TAG-y)"
> > > - test -d seabios && ( cd seabios; git pull ) || \
> > > + test -d seabios && ( cd seabios; git fetch ) || \
> >
> > I agree that git fetch should be here, certainly not git pull.
>
> A fetch alone does not seem to make much sense. It will not update the
> tree when working with the master tag, as far as I can tell. The idea of
> master is that you will always get the latest and greatest SeaBIOS, so
> the tree should be updated accordingly, eh, fetched and merged.
The fetch should be okay, because the patch also changes the tag from
"master" to "origin/master".
The real fix is to move SeaBIOS to Kconfig.
It's okay if you wish to backout my previous patch. I apologize for
not catching the build failure on subsequent builds.
-Kevin
More information about the coreboot
mailing list