From: Ian Jackson <ian.jackson@citrix.com>
To: Roger Pau Monne <roger.pau@citrix.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [Xen-devel] [PATCH 6/6] osstest: use a locally built pkg repository for FreeBSD
Date: Fri, 24 May 2019 18:26:44 +0100 [thread overview]
Message-ID: <23784.10580.1233.680380@mariner.uk.xensource.com> (raw)
Message-ID: <20190524172644.yt4xi1e2c_wb75XS4wM9z-exH6MXlXptczEB6ZKMaCI@z> (raw)
In-Reply-To: <20190524104545.7s7hi722qw7fobkv@Air-de-Roger>
Roger Pau Monne writes ("Re: [Xen-devel] [PATCH 6/6] osstest: use a locally built pkg repository for FreeBSD"):
> On Thu, May 23, 2019 at 12:06:29PM +0100, Ian Jackson wrote:
> > I realise this is a bit late to be saying this, but had you
> > considered making the packages build a different step in the same
> > job ? That might make a lot of this go away...
>
> Do you mean to build the packages in build-prep instead of relying on
> having a custom binary repository?
No. Maybe I am confused. I thought your usual flight was
1 install anointed freebsd
2 build this freebsd
3+ build this package repo
4 install this freebsd (from step 2)
5 rebuild this freebsd (for testing that the build didn't break)
6 rebuild this package repo (")
7+ install this package repoo
8+ build xen
My question is why 2/3 and 5/6 are different jobs. If you made 2+3 a
single job (with 2 and 3 being separate steps) then there would only
need to be a single anointment.
> IIRC the package building job takes a non-trivial amount of time (2-3h
> IIRC), because it has to build gcc (for SeaBIOS) and python, perl...
You could make the package building job optional if you only want to
do it some of the time.
> > > + # refkey: freebsd job: build-<arch>-freebsd
> > > + # refkey: freebsd-packages job: build-<arch>-freebsd-packages
> > > + anoint="$anoint \"$anointed\" $flight \
> > > + build-$freebsd_arch-$freebsd_name"
> >
> > Maybe use an array variable for anount, and then you can avoid the
> > shell \" quoting.
>
> Please bear with me, but can you elaborate on this?
Roughly,
anoint=()
....
anoint+=("$anointed" $flight build-..)
...
./mg-anoint "${anoint[@]}"
Note that the \" \" construct has gone, because there is now no
additional layer of shell dequoting.
> > There seems like a lot of repetition here. For example, FREEBSD_DIST
> > overrides FreeBSDDist but /$arch is appended in two places. Maybe
> > ${FREEBSD_DIST- ... something ... } would be better ?
>
> OK, let me try to remove some of the duplication here.
Thanks.
Another suggestion I mentioned IRL which I wanted to write down was:
maybe have mg-anoint have a reporting mode where it prints something
suitable for shell `eval'.
> > This feels very similar to the code above, although it lacks the
> > special handling for the version.
>
> Maybe I can see about factoring some of this into a helper, but there
> are slight differences in both if branches that I'm not sure can be
> factored out.
Mmmm.
> Why I don't start by fixing the repetition of:
> freebsd_runvars="$freebsd_runvars \... and we take it from there?
Sure, let's see what you come up with.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2019-05-24 17:27 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-20 16:59 [PATCH 0/6] osstest: create a local binary FreeBSD package repository Roger Pau Monne
2019-02-20 16:59 ` [PATCH 1/6] osstest: introduce a helper to stash a whole directory Roger Pau Monne
2019-05-23 9:48 ` Ian Jackson
2019-05-23 9:48 ` [Xen-devel] " Ian Jackson
2019-05-24 9:42 ` Roger Pau Monné
2019-05-24 9:42 ` [Xen-devel] " Roger Pau Monné
2019-02-20 16:59 ` [PATCH 2/6] osstest: introduce a helper to create a weblink to a directory Roger Pau Monne
2019-05-23 9:57 ` Ian Jackson
2019-05-23 9:57 ` [Xen-devel] " Ian Jackson
2019-02-20 16:59 ` [PATCH 3/6] osstest: allow to perform multiple anoints in the same transaction Roger Pau Monne
2019-05-23 10:00 ` Ian Jackson
2019-05-23 10:00 ` [Xen-devel] " Ian Jackson
2019-02-20 16:59 ` [PATCH 4/6] osstest: introduce a helper to get the svn revision of a git commit Roger Pau Monne
2019-05-23 10:03 ` Ian Jackson
2019-05-23 10:03 ` [Xen-devel] " Ian Jackson
2019-05-24 9:57 ` Roger Pau Monné
2019-05-24 9:57 ` [Xen-devel] " Roger Pau Monné
2019-05-24 10:35 ` Ian Jackson
2019-05-24 10:35 ` [Xen-devel] " Ian Jackson
2019-02-20 17:00 ` [PATCH 5/6] osstest: introduce a script to build a FreeBSD package repository Roger Pau Monne
2019-05-23 10:19 ` Ian Jackson
2019-05-23 10:19 ` [Xen-devel] " Ian Jackson
2019-05-23 10:38 ` Ian Jackson
2019-05-23 10:38 ` [Xen-devel] " Ian Jackson
2019-05-24 10:13 ` Roger Pau Monné
2019-05-24 10:13 ` [Xen-devel] " Roger Pau Monné
2019-05-24 11:21 ` Ian Jackson
2019-05-24 11:21 ` [Xen-devel] " Ian Jackson
2019-02-20 17:00 ` [PATCH 6/6] osstest: use a locally built pkg repository for FreeBSD Roger Pau Monne
2019-05-23 11:06 ` Ian Jackson
2019-05-23 11:06 ` [Xen-devel] " Ian Jackson
2019-05-24 10:45 ` Roger Pau Monné
2019-05-24 10:45 ` [Xen-devel] " Roger Pau Monné
2019-05-24 17:26 ` Ian Jackson [this message]
2019-05-24 17:26 ` Ian Jackson
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=23784.10580.1233.680380@mariner.uk.xensource.com \
--to=ian.jackson@citrix.com \
--cc=roger.pau@citrix.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).