[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] Config.mk: Update SEABIOS_UPSTREAM_TAG to rel-1.9.1



>>> On 24.02.16 at 16:18, <wei.liu2@xxxxxxxxxx> wrote:
> On Wed, Feb 24, 2016 at 08:14:09AM -0700, Jan Beulich wrote:
>> >>> On 24.02.16 at 15:41, <wei.liu2@xxxxxxxxxx> wrote:
>> > On Wed, Feb 24, 2016 at 08:34:31AM -0600, Doug Goldstein wrote:
>> >> On 2/16/16 7:52 AM, Wei Liu wrote:
>> >> > On Tue, Feb 16, 2016 at 01:43:26PM +0000, Ian Campbell wrote:
>> >> >> On Mon, 2016-02-15 at 16:13 +0000, Wei Liu wrote:
>> >> >>> On Mon, Feb 15, 2016 at 08:38:02AM -0600, Doug Goldstein wrote:
>> >> >>>> Switch from tracking a commit post 1.9.0 to the 1.9.1 release.
>> >> >>>>
>> >> >>>
>> >> >>> Thanks for this patch.
>> >> >>>
>> >> >>> We can't apply this patch just yet. We need to wait a bit until the
>> >> >>> change to upstream trickles down to our own tree [0] -- OSSTest will
>> >> >>> work on that shortly. See [1] for an example output.
>> >> >>
>> >> >> Our osstest branch only tests seabios.git#master whereas this is 
>> >> >> tracking
>> >> >> the 1.9.x stable branch.
>> >> >>
>> >> >> I don't think we really want/need the overhead of testing multiple 
> branches
>> >> >> of SeaBIOS, so IMHO it is sufficient to observe that all of the 
>> >> >> commits on
>> >> >> the stable branch have passed our master branch commit gate (even if 
>> >> >> that
>> >> >> means they did so in the context of other un-backported patches). I 
>> >> >> haven't
>> >> >> checked but given the timings and 
>> >> >> http://logs.test-lab.xenproject.org/osste 
> 
>> >> >> st/results/all-branch-statuses.txt indicating our seabios branch is up 
>> >> >> to
>> >> >> date I expect this is the case.
>> >> >>
>> >> >> There is some risk to this, but SeaBIOS gets a fairly small number of
>> >> >> changes so the stable branches do not stray too far from master in 
>> >> >> general
>> >> >> so I think this approach is fine.
>> >> >>
>> >> >> IOW it would be fine to push upstreams 1.9-stable to our repo along 
>> >> >> with
>> >> >> the rel-1.9.1 tag and to then apply this patch.
>> >> >>
>> >> > 
>> >> > Makes sense.
>> >> > 
>> >> 
>> >> ping?
>> >> 
>> > 
>> > With my recently acquired seabios maintainer hat on:
>> > 
>> >   Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
>> > 
>> > One of the committers will need to pick this up to do the actual push.
>> 
>> I can't see a rel-1.9.1 tag yet in the repo on xenbits, so - are you
>> sure?
>> 
> 
> That's what I meant -- a committer needs to fetch from upstream tree
> then push the tag to our own seabios tree before applying this patch.
> 
> The upstream tree is at git://git.seabios.org/seabios.git .
> 
> If you can handle that, it would be great. :-)

I'd rather leave this to someone being a better friend with git
than I am, so Ian, maybe you could do this?

Ultimately I think you, Wei, should really have commit rights
on that particular tree, to efficiently fulfill your role as its
maintainer.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.