xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Juergen Gross <jgross@suse.com>
To: "Géza Gémes" <geza.gemes@gmail.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>
Subject: Re: raisin and minios stubdom
Date: Mon, 3 Apr 2017 07:20:15 +0200	[thread overview]
Message-ID: <ca6b6ab7-88bc-33e7-a428-e374d971643f@suse.com> (raw)
In-Reply-To: <CAAzT_yRjTKOwm_tc=4cQfnyio=2MrX-=As9gtCK4bXGB_wSuJw@mail.gmail.com>

On 01/04/17 08:19, Géza Gémes wrote:
> 
> 
> 2017. márc. 31. 16:15 ezt írta ("Juergen Gross" <jgross@suse.com
> <mailto:jgross@suse.com>>):
> 
>     On 31/03/17 16:05, Konrad Rzeszutek Wilk wrote:
>     > On Thu, Mar 30, 2017 at 07:42:48PM +0200, Gémes Géza wrote:
>     >>
>     >>> On Mon, Mar 27, 2017 at 09:28:14PM +0200, Gémes Géza wrote:
>     >>>> Hi,
>     >>>>
>     >>>> Currently the xen build system has optional support for
>     building a minios
>     >>>> (+needed libraries and tools) based stubdom.
>     >>>>
>     >>>> What is your opinion about moving support for building this
>     into raisin and
>     >>>> once that is stable drop support in the xen build system?
>     >>> Why? I do like doing 'make' and 'make install' and it doing
>     everything
>     >>> for me.
>     >>>
>     >>>> Cheers,
>     >>>>
>     >>>> Geza
>     >>>>
>     >>>>
>     >>>> _______________________________________________
>     >>>> Xen-devel mailing list
>     >>>> Xen-devel@lists.xen.org <mailto:Xen-devel@lists.xen.org>
>     >>>> https://lists.xen.org/xen-devel <https://lists.xen.org/xen-devel>
>     >>
>     >> Because it means that xen build needs to download and build a lot
>     of 3PP
>     >> components. Raisin is already designed to do so (it already
>     builds qemu-xen,
>     >
>     > If you do 'make src-tarball' it will do that for you - and you can
>     package
>     > all of that in a tarball.
>     >
>     >> qemu-traditional, libvirt and a few others). I think building
>     anything
>     >> besides xen proper would fit its scope better.
>     >
>     > OK, but that does not square well with RPM build systems. Those
>     are interested
>     > in building just one component (xen+toolstack+its extra pieces). Using
>     > raisin to build everything is not going to fly.
>     >
>     > (Also distros like to seperate componets out - so they build
>     qemu-upstream
>     > seperate - which is used by Xen - and they could also do it for MiniOS
>     > if they were spec files for it and such).
> 
>     There are only few stubdoms you can build without the Xen tree. How
>     would you do so for e.g. xenstore-stubdom needing the Xenstore sources
>     to be built? Several stubdoms need libxc built for stubdom included.
>     And you want to have a build error if e.g. a libxc modification is
>     breaking stubdom build.
> 
> 
>     Juergen
> 
> Hi,
> 
> Raisin already builds xen too, so it has all the dependencies ready.
> Regarding the problem of breaking stubdom build by libxc changes I think
> those can be prevented if we introduce osstests for raisin build. Maybe
> we should start with that, adding raisin to the osstest framework.
> Opinions?

osstest is too late. I want to see a build error _before_ sending a
patch.

So how is raisin working exactly? Is it possible to do incremental
buils or is the build always complete? Can I start builds of only a
subtree? Is it possible to use a private version of some sub-component?

I'm not opposed to use raisin e.g. in osstest. I'm opposed to a change
in the developer workflow requiring to spend either much more time for
testing the build or to add additional steps for it. One-time changes
are fine, changes requiring the developer not to forget an additional
command are not.


Juergen

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

  parent reply	other threads:[~2017-04-03  5:20 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-27 19:28 raisin and minios stubdom Gémes Géza
2017-03-27 19:38 ` Konrad Rzeszutek Wilk
2017-03-30 17:42   ` Gémes Géza
2017-03-31 14:05     ` Konrad Rzeszutek Wilk
2017-03-31 14:15       ` Juergen Gross
2017-04-01  6:19         ` Géza Gémes
2017-04-01 18:00           ` Gémes Géza
2017-04-03  5:20           ` Juergen Gross [this message]
2017-04-07 18:54             ` Géza Gémes
2017-04-10  5:05               ` Juergen Gross
2017-04-19 23:33                 ` Stefano Stabellini
2017-04-03 11:17 ` George Dunlap
2017-04-03 12:01   ` Wei Liu
2017-04-07 18:29     ` Gémes Géza

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=ca6b6ab7-88bc-33e7-a428-e374d971643f@suse.com \
    --to=jgross@suse.com \
    --cc=geza.gemes@gmail.com \
    --cc=xen-devel@lists.xenproject.org \
    /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).