qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Cody <jcody@redhat.com>
To: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
	"Stefan Hajnoczi" <stefanha@gmail.com>,
	"Alexander Graf" <agraf@suse.de>,
	qemu-devel <qemu-devel@nongnu.org>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] [RFC PATCH 1/1] OpenBIOS: Switch over to official OpenBIOS git repo
Date: Thu, 7 Jul 2016 16:53:44 -0400	[thread overview]
Message-ID: <20160707205344.GB21682@localhost.localdomain> (raw)
In-Reply-To: <1b11eefb-ed51-417a-7391-53fdab96fe95@ilande.co.uk>

On Thu, Jul 07, 2016 at 06:22:37PM +0100, Mark Cave-Ayland wrote:
> On 07/07/16 04:12, Jeff Cody wrote:
> 
> > Alright, here is what I think will work to switch over relatively
> > painlessly, and move master over to the github hashes for the submodule.  
> > 
> > I haven't done it yet on the server, because I want to get another set of
> > eyes double checking it (Stefan, Mark).  However, I have mocked it up and
> > tested it locally, so I think it should work fine, unless there are
> > scenarios I missed.  Here are the step-by-step details:
> > 
> > 
> > Server side actions
> > =====================
> > 
> > 1) In bare openbios.git repo (git://git.qemu.org/openbios.git):
> > 
> >     git tag svn-last-official 422b916
> >     git branch svn-head 422b916
> >     git branch -f master 36785d7
> > 
> > 2) openbios update repo (updated daily via cron):
> > 
> >     # update script contents
> >     # continue fetch svn commits for now as well
> >     # 'official' branch tracks github/master
> >     git svn fetch svn 
> >     git merge git-svn
> >     git fetch github   # official openbios github repo
> >     git push /pub/git/openbios.git master:svn-head
> >     git push /pub/git/openbios.git official:master  
> > 
> > 
> > Non-server side actions
> > ======================
> > 
> > * QEMU new patch to switch over
> > 
> >     # normal submodule stuff:
> > 
> >     git submodule init
> >     git submodule update roms/openbios
> > 
> >     # create patch to switch over:
> >     cd roms/openbios
> >     git checkout master
> >     cd ../
> >     git commit -a  # assuming you made no other changes
> > 
> > 
> > * Existing users of git://git.qemu.org/openbios.git as a standalone work
> >   repo:
> > 
> >     git fetch
> > 
> >     # if there are local commits / work they want to keep
> >     git rebase origin/master
> >    
> >     # alternatively:
> >     git reset --hard origin/master                         
> > 
> > 
> > 
> > The above all seems to work in my testing.  Stefan, Mark: do you see any
> > glaring issues with the above?
> 
> Hi Jeff,
> 
> I've just worked through this locally and it looks pretty good - I've
> documented a few minor alterations below where I've made them:
> 

Great, thanks.

> 
> # Get qemu-project.org repository, fetch remote master
> git clone --bare git://git.qemu.org/openbios.git
> cd openbios.git
> git remote add -f openbios-github https://github.com/openbios/openbios.git
> 
> # Switch master to github
> git tag svn-last-official 422b916
> git branch svn-head 422b916
> git branch -f master 36785d7
> 
> # official branch is no longer needed
> git branch -D official
> 

Yep, we should be able to get rid of the 'official' branch, as it won't be
needed going forward.

> 
> I see that there are still references to SVN in the nightly update, and
> since the SVN repository will disappear as soon as the new git
> repository is up and running and master now tracks master, I figured
> this could be simplified right down to this:
> 
> 
> # Nightly update
> git fetch openbios-github
> git push /pub/git/openbios.git master

I wasn't sure if there was going to be an overlap period - if the SVN server
is going offline, however, then I will go ahead and get rid of the svn
fetch, merge, and push actions.

> 
> 
> For the submodule switchover, the use of "git submodule init" seemed a
> bit strange - I would have thought we'd be just fine starting with the
> "git submodule update" line? Also I have a script for the remainder of
> the switchover process that I use when updating the OpenBIOS roms, so as
> soon as openbios.git is updated, I can send a pull request to switch the
> submodule hash really easily.
> 
> Finally for people who have work in branches from the old checkout, I'm
> happy to suggest that they just do "git fetch" followed by "git rebase
> origin/master" on any local branches of interest.
> 
>

Thanks!
Jeff

  reply	other threads:[~2016-07-07 20:53 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-21  5:40 [Qemu-devel] [RFC PATCH 1/1] OpenBIOS: Switch over to official OpenBIOS git repo Jeff Cody
2016-06-21  5:42 ` Jeff Cody
2016-06-21 10:28 ` Stefan Hajnoczi
2016-06-21 13:48   ` Mark Cave-Ayland
2016-06-27  6:48     ` Mark Cave-Ayland
2016-06-28  6:11       ` Jeff Cody
2016-06-28 13:44         ` Stefan Hajnoczi
2016-06-28 20:43           ` Mark Cave-Ayland
2016-07-06  7:26             ` Mark Cave-Ayland
2016-07-06 13:29               ` Jeff Cody
2016-07-06 15:19                 ` Mark Cave-Ayland
2016-07-07  3:12                   ` Jeff Cody
2016-07-07 17:22                     ` Mark Cave-Ayland
2016-07-07 20:53                       ` Jeff Cody [this message]
2016-07-07 21:25                         ` Jeff Cody
2016-07-08 15:48                           ` Mark Cave-Ayland
     [not found] <mailman.852.1467157487.26856.qemu-devel@nongnu.org>
2016-06-28 23:55 ` G 3

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160707205344.GB21682@localhost.localdomain \
    --to=jcody@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=mark.cave-ayland@ilande.co.uk \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).