All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Egger <Christoph.Egger@amd.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH][TOOLS] libfsimage: portability fixes
Date: Thu, 27 Mar 2008 16:16:51 +0100	[thread overview]
Message-ID: <200803271616.52383.Christoph.Egger@amd.com> (raw)
In-Reply-To: <18411.32634.344747.246537@mariner.uk.xensource.com>

On Thursday 27 March 2008 12:05:30 Ian Jackson wrote:
> Christoph Egger writes ("[Xen-devel] [PATCH][TOOLS] libfsimage: portability 
fixes"):
> > -${CC:-gcc} -o ext2-test ext2-test.c -lext2fs >/dev/null 2>&1
> > +if test -z ${CC}; then CC="gcc"; fi
> > +${CC} -o ext2-test ext2-test.c -lext2fs >/dev/null 2>&1
> > +
>
> This also seems to serve no purpose as far as I can tell.

This is for the ancient Solaris /bin/sh.  If John Levon confirms that
this and that the $(SHELL)  mentioned in your other mail 
are NOT needed/required to not break Solaris, you can undo these two changes.

Christoph


-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy

  reply	other threads:[~2008-03-27 15:16 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-26 14:14 [PATCH][TOOLS] libfsimage: portability fixes Christoph Egger
2008-03-27 10:54 ` Ian Jackson
2008-03-27 15:13   ` Christoph Egger
2008-03-27 15:36     ` Ian Jackson
2008-03-27 16:23       ` John Levon
2008-03-28 10:08       ` Christoph Egger
2008-03-28 10:42         ` Ian Jackson
2008-03-28 12:40           ` Christoph Egger
2008-03-28 14:19             ` Ian Jackson
2008-03-27 11:05 ` Ian Jackson
2008-03-27 15:16   ` Christoph Egger [this message]
2008-03-27 16:26     ` John Levon
2008-03-27 15:05 ` Aron Griffis
2008-03-27 15:13   ` Ian Jackson
2008-03-27 15:56     ` Aron Griffis
  -- strict thread matches above, loose matches on Subject: below --
2008-03-27 15:36 Christoph Egger

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=200803271616.52383.Christoph.Egger@amd.com \
    --to=christoph.egger@amd.com \
    --cc=Ian.Jackson@eu.citrix.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.