All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeremy Katz <katzj@redhat.com>
To: xen-devel@lists.xensource.com
Subject: Re: [PATCH] tools: Fix network-bridge to work with Gentoo (version 2)
Date: Wed, 05 Oct 2005 11:05:09 -0400	[thread overview]
Message-ID: <1128524710.21137.8.camel@bree.local.net> (raw)
In-Reply-To: <20051004235436.GB7576@us.ibm.com>

On Tue, 2005-10-04 at 18:54 -0500, Jon Mason wrote:
> This patch fixes domU networking in Gentoo.  The change to ifup/ifdown
> (which isn't availabe in Gentoo) prevents the bridge from being setup
> correctly.  The patch below checks if ifup is avaiable, and if not
> starts the Gentoo networking scripts.
> 
> This new version includes shell functions where distro specific options
> may be handled (per Ian's suggestion).  

Doing this all in the same script feels like it's going to spiral out of
control into spaghetti-land kind of quickly and also not allow for some
of the real variation that's going to be desired for different
distributions.

Why try to do a one-size-fits-all instead of having scripts which really
can be made to best fit the distro?  With a more traditional build
process, it would be easy enough to have distro specific scripts dropped
into subdirs of the examples directory and use them as
overrides/replacements.  The thing that makes that tricky for Xen is the
way make dist works.  

Thinking aloud -- maybe just setting a variable so we know we're in the
'make dist' case and using that to install all the distro specific stuff
into DESTDIR and then letting install.sh also pick out the right distro
would work.  Then for the more normal install case, just the proper
things for the distro would get installed.  Does this seem reasonable?
I'll try to cook up something today

Jeremy

      parent reply	other threads:[~2005-10-05 15:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-29 22:20 [PATCH] tools: Fix network-bridge to work with Gentoo Jon Mason
2005-09-29 23:11 ` Dan Smith
2005-09-29 23:31   ` Nivedita Singhvi
2005-09-29 23:23 ` Robb Romans
2005-10-04 23:54 ` [PATCH] tools: Fix network-bridge to work with Gentoo (version 2) Jon Mason
2005-10-05  4:03   ` Anthony Liguori
2005-10-05 15:05   ` Jeremy Katz [this message]

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=1128524710.21137.8.camel@bree.local.net \
    --to=katzj@redhat.com \
    --cc=xen-devel@lists.xensource.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.