All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Pfaff <blp@cs.stanford.edu>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: PATCH: Solaris/Sparc patch against yesterdays' BGR CVS update
Date: Tue, 16 May 2006 20:56:05 -0700	[thread overview]
Message-ID: <87fyj9uwx6.fsf@benpfaff.org> (raw)
In-Reply-To: 20987482.1147827980269.JavaMail.root@eastrmwml01.mgt.cox.net

Ben Taylor <sol10x86@cox.net> writes:

> ---- Ben Pfaff <blp@cs.stanford.edu> wrote: 
>> Ben Taylor <sol10x86@cox.net> writes:
>> 
>> > Enclosed is the latest version of the qemu solaris/sparc patch, applied
>> > against yesterday's CVS (post BGR update).
>> 
>> I'm happy to say that this patch makes qemu compile and work on
>> Solaris quite nicely for me.  I did have to invoke configure with
>> "--extra-ldflags=-lsocket" (and fix configure to put $LDFLAGS in
>> the libSDL probing commands).
>
> Odd. It may have something to do with your localized copy of libSDL.
> The one from blastwave doesn't require -lsocket.  I've been using
> a version of these patches for a year, and never ran into that problem.

For what it's worth, I installed libSDL 1.2.9 from the upstream
.tar.gz without any special configuration options.

>>  I also had to override the
>> configure script's desire to use /usr/ucb/install instead of GNU
>> install.  
>
> The configure script wants to use ginstall, assuming you've got it in your
> path.  I did not use "install" because there are several variations of install
> on Solaris, and configure is coded to make sure you don't use them.
>
> It recommends /usr/ucb/install if you don't have GNU install (or it can't find
> it because it's looking for ginstall).  I'll have to check the logic to make sure

Ah, I see.  I thought that it was complaining that the default
"install" was not the UCB install.  In the future I'll know to
just give it the GNU version up front.

>> The former won't let me install without being root,
>> because it wants to install everything as a "staff" account; the
>> latter will.
>
> That's a localized configuration requirement you have, and that configure should
> probably not try to take into account.

I'm not sure that installing as non-root is all that localized a
requirement, but I'm easily able to work around it so it doesn't
bother me that much.
-- 
"Then, I came to my senses, and slunk away, hoping no one overheard my
 thinking."
--Steve McAndrewSmith in the Monastery

  reply	other threads:[~2006-05-17  3:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-17  1:06 [Qemu-devel] Re: PATCH: Solaris/Sparc patch against yesterdays' BGR CVS update Ben Taylor
2006-05-17  3:56 ` Ben Pfaff [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-05-11 14:45 [Qemu-devel] " Ben Taylor
2006-05-16 23:28 ` [Qemu-devel] " Ben Pfaff

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=87fyj9uwx6.fsf@benpfaff.org \
    --to=blp@cs.stanford.edu \
    --cc=qemu-devel@nongnu.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 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.