qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X
@ 2015-08-06 15:35 Programmingkid
  2015-08-07 17:00 ` Peter Maydell
  0 siblings, 1 reply; 10+ messages in thread
From: Programmingkid @ 2015-08-06 15:35 UTC (permalink / raw)
  To: qemu-devel qemu-devel; +Cc: Peter Maydell

[-- Attachment #1: Type: text/plain, Size: 2254 bytes --]

Add information on compiling source code on Mac OS X
to the documentation.

Signed-off-by: John Arbuckle <programmingkidx@gmail.com>

---
This patch may depend on the other patch I sent in for the qemu-doc.texi file. 

 qemu-doc.texi |   41 ++++++++++++++++++++++++++++++++++++++---
 1 files changed, 38 insertions(+), 3 deletions(-)

diff --git a/qemu-doc.texi b/qemu-doc.texi
index 9a8f353..eb4ce8d 100644
--- a/qemu-doc.texi
+++ b/qemu-doc.texi
@@ -3024,9 +3024,44 @@ and all other qemu-system-@var{target}.exe compiled for Win32.
 @node Mac OS X
 @section Mac OS X
 
-The Mac OS X patches are not fully merged in QEMU, so you should look
-at the QEMU mailing list archive to have all the necessary
-information.
+After downloading the QEMU source code, double-click it to expand it.
+
+Then configure QEMU to build the emulator you want using the --target-list option:
+@example
+./configure --target-list=ppc-softmmu,i386-softmmu --disable-sdl --disable-gtk
+@end example
+
+@smalldisplay
+*Disabling SDL and GTK support is suggested because QEMU runs poorly with them.
+@end smalldisplay
+
+Start building QEMU using the make command:
+@example
+make
+@end example
+
+Note: Apple supplied versions of gcc may not work with QEMU. If after the configure
+step you see a message like this:
+@example
+ERROR: Your compiler does not support the __thread specifier for
+       Thread-Local Storage (TLS). Please upgrade to a version that does.
+@end example
+You may have to build your own version of gcc from source. Expect that to take
+several hours. GCC 4.3 or above would work. More information can be found here:
+https://gcc.gnu.org/install/ @*
+
+These are some of the third party binaries of gcc available for download:
+@itemize
+@item Homebrew http://brew.sh/
+@item https://www.litebeam.net/gcc/gcc_472.pkg
+@item http://www.macports.org/ports.php?by=name&substr=gcc
+@end itemize
+
+You can have several versions of GCC on your system. To specify a certain version,
+use the --cc and --cxx options.
+@example
+./configure --cxx=<path of your c++ compiler> --cc=<path of your c compiler> <other options>
+@end example
 
 @node Make targets
 @section Make targets
-- 
1.7.5.4


[-- Attachment #2: Type: text/html, Size: 11921 bytes --]

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-06 15:35 [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X Programmingkid
@ 2015-08-07 17:00 ` Peter Maydell
  2015-08-07 20:32   ` Programmingkid
  2015-08-07 20:57   ` [Qemu-devel] [PATCH v2] " Programmingkid
  0 siblings, 2 replies; 10+ messages in thread
From: Peter Maydell @ 2015-08-07 17:00 UTC (permalink / raw)
  To: Programmingkid; +Cc: qemu-devel qemu-devel

On 6 August 2015 at 16:35, Programmingkid <programmingkidx@gmail.com> wrote:
> Add information on compiling source code on Mac OS X
> to the documentation.
>
> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
>
> ---
> This patch may depend on the other patch I sent in for the qemu-doc.texi
> file.
>
>  qemu-doc.texi |   41 ++++++++++++++++++++++++++++++++++++++---
>  1 files changed, 38 insertions(+), 3 deletions(-)
>
> diff --git a/qemu-doc.texi b/qemu-doc.texi
> index 9a8f353..eb4ce8d 100644
> --- a/qemu-doc.texi
> +++ b/qemu-doc.texi
> @@ -3024,9 +3024,44 @@ and all other qemu-system-@var{target}.exe compiled
> for Win32.
>  @node Mac OS X
>  @section Mac OS X
>
>
>
> -The Mac OS X patches are not fully merged in QEMU, so you should look
> -at the QEMU mailing list archive to have all the necessary
> -information.

Heh, that's laughably ancient. Thanks for writing a fix for this.

At the top of this section is probably a good place to say

# QEMU should run on Mac OS X versions 10.5 and later.

> +After downloading the QEMU source code, double-click it to expand it.
> +
> +Then configure QEMU to build the emulator you want using the --target-list
> option:
> +@example
> +./configure --target-list=ppc-softmmu,i386-softmmu --disable-sdl
> --disable-gtk
> +@end example

I think it would be better to just make configure disable
SDL and GTK automatically when building the OSX binary.
I'll write a patch for this...

Suggestions like --target-list= belong in a host-OS-agnostic
section of the documentation; they're not OSX specific.

> +
> +@smalldisplay
> +*Disabling SDL and GTK support is suggested because QEMU runs poorly with
> them.
> +@end smalldisplay
> +
> +Start building QEMU using the make command:
> +@example
> +make
> +@end example
> +
> +Note: Apple supplied versions of gcc may not work with QEMU. If after the
> configure

I think the first and most important thing we should say is:

# If you have a recent version of Mac OS X (OSX 10.7 or better
# with Xcode 4.2 or better) we recommend building QEMU with the
# default compiler provided by Apple for your version of OSX as
# part of Xcode (which will be 'clang'). The configure script will
# automatically pick this.

That will cover almost everybody who wants to build QEMU
(on the assumption that most people are running a new OSX).

Then after that you can have the information about older
OSX which don't have a working compiler:

# Apple supplied versions of GCC do not work with QEMU.
# If after the configure step ....

(https://trac.macports.org/wiki/UsingTheRightCompiler
suggests that XCode 3.2 for OSX 10.6 provided a non-default
'clang'; I don't have a 10.6 + 3.2 system to check whether that
clang will build QEMU though. If it does then we should
recommend using it rather than building gcc. 10.5 users
are definitely stuck with finding or building a non-Apple
compiler, though.)

> +step you see a message like this:
> +@example
> +ERROR: Your compiler does not support the __thread specifier for
> +       Thread-Local Storage (TLS). Please upgrade to a version that does.
> +@end example
> +You may have to build your own version of gcc from source. Expect that to
> take
> +several hours. GCC 4.3 or above would work. More information can be found
> here:
> +https://gcc.gnu.org/install/ @*
> +
> +These are some of the third party binaries of gcc available for download:
> +@itemize
> +@item Homebrew http://brew.sh/
> +@item https://www.litebeam.net/gcc/gcc_472.pkg
> +@item http://www.macports.org/ports.php?by=name&substr=gcc
> +@end itemize
> +
> +You can have several versions of GCC on your system. To specify a certain
> version,
> +use the --cc and --cxx options.
> +@example
> +./configure --cxx=<path of your c++ compiler> --cc=<path of your c
> compiler> <other options>
> +@end example
>

thanks
-- PMM

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-07 17:00 ` Peter Maydell
@ 2015-08-07 20:32   ` Programmingkid
  2015-08-07 23:13     ` Peter Maydell
  2015-08-07 20:57   ` [Qemu-devel] [PATCH v2] " Programmingkid
  1 sibling, 1 reply; 10+ messages in thread
From: Programmingkid @ 2015-08-07 20:32 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel qemu-devel


On Aug 7, 2015, at 1:00 PM, Peter Maydell wrote:

>> -The Mac OS X patches are not fully merged in QEMU, so you should look
>> -at the QEMU mailing list archive to have all the necessary
>> -information.
> 
> Heh, that's laughably ancient. Thanks for writing a fix for this.
Your welcome. 

> 
> At the top of this section is probably a good place to say
> 
> # QEMU should run on Mac OS X versions 10.5 and later.
Good suggestion. Will include a system requirement section.

> 
>> +After downloading the QEMU source code, double-click it to expand it.
>> +
>> +Then configure QEMU to build the emulator you want using the --target-list
>> option:
>> +@example
>> +./configure --target-list=ppc-softmmu,i386-softmmu --disable-sdl
>> --disable-gtk
>> +@end example
> 
> I think it would be better to just make configure disable
> SDL and GTK automatically when building the OSX binary.
> I'll write a patch for this...

If you are certain this patch will fix the problem, then removing the SDL and GTK info would be fine.

> 
> Suggestions like --target-list= belong in a host-OS-agnostic
> section of the documentation; they're not OSX specific.
OK.

> 
>> +
>> +@smalldisplay
>> +*Disabling SDL and GTK support is suggested because QEMU runs poorly with
>> them.
>> +@end smalldisplay
>> +
>> +Start building QEMU using the make command:
>> +@example
>> +make
>> +@end example
>> +
>> +Note: Apple supplied versions of gcc may not work with QEMU. If after the
>> configure
> 
> I think the first and most important thing we should say is:
> 
> # If you have a recent version of Mac OS X (OSX 10.7 or better
> # with Xcode 4.2 or better) we recommend building QEMU with the
> # default compiler provided by Apple for your version of OSX as
> # part of Xcode (which will be 'clang'). The configure script will
> # automatically pick this.
> 
> That will cover almost everybody who wants to build QEMU
> (on the assumption that most people are running a new OSX).
> 
> Then after that you can have the information about older
> OSX which don't have a working compiler:
> 
> # Apple supplied versions of GCC do not work with QEMU.
> # If after the configure step ....
> 
> (https://trac.macports.org/wiki/UsingTheRightCompiler
> suggests that XCode 3.2 for OSX 10.6 provided a non-default
> 'clang'; I don't have a 10.6 + 3.2 system to check whether that
> clang will build QEMU though. If it does then we should
> recommend using it rather than building gcc. 10.5 users
> are definitely stuck with finding or building a non-Apple
> compiler, though.)

I just checked and my system does include clang. Didn't even know about it. Too bad it didn't work to compile QEMU. It doesn't support Thread-Local Storage. It is probably too old (version 3.0). 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-07 17:00 ` Peter Maydell
  2015-08-07 20:32   ` Programmingkid
@ 2015-08-07 20:57   ` Programmingkid
  2015-08-14 15:16     ` Peter Maydell
  1 sibling, 1 reply; 10+ messages in thread
From: Programmingkid @ 2015-08-07 20:57 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel qemu-devel

[-- Attachment #1: Type: text/plain, Size: 2186 bytes --]

Add information on compiling source code on Mac OS X
to the documentation.

Signed-off-by: John Arbuckle <programmingkidx@gmail.com>

---
 qemu-doc.texi |   44 +++++++++++++++++++++++++++++++++++++++++---
 1 files changed, 41 insertions(+), 3 deletions(-)

diff --git a/qemu-doc.texi b/qemu-doc.texi
index 0125bc7..c059c49 100644
--- a/qemu-doc.texi
+++ b/qemu-doc.texi
@@ -3021,9 +3021,47 @@ and all other qemu-system-@var{target}.exe compiled for Win32.
 @node Mac OS X
 @section Mac OS X
 
-The Mac OS X patches are not fully merged in QEMU, so you should look
-at the QEMU mailing list archive to have all the necessary
-information.
+System Requirements:
+@itemize
+@item Mac OS 10.5 or higher
+@item GCC 4.3 or higher
+@end itemize
+
+After downloading the QEMU source code, double-click it to expand it.
+
+Then configure and make QEMU:
+@example
+./configure
+make
+@end example
+
+If you have a recent version of Mac OS X (OSX 10.7 or better
+with Xcode 4.2 or better) we recommend building QEMU with the
+default compiler provided by Apple, for your version of Mac OS X
+(which will be 'clang'). The configure script will
+automatically pick this.
+
+Note: If after the configure step you see a message like this:
+@example
+ERROR: Your compiler does not support the __thread specifier for
+       Thread-Local Storage (TLS). Please upgrade to a version that does.
+@end example
+You may have to build your own version of gcc from source. Expect that to take
+several hours. More information can be found here:
+@uref{https://gcc.gnu.org/install/} @*
+
+These are some of the third party binaries of gcc available for download:
+@itemize
+@item Homebrew: @uref{http://brew.sh/}
+@item @uref{https://www.litebeam.net/gcc/gcc_472.pkg}
+@item @uref{http://www.macports.org/ports.php?by=name&substr=gcc}
+@end itemize
+
+You can have several versions of GCC on your system. To specify a certain version,
+use the --cc and --cxx options.
+@example
+./configure --cxx=<path of your c++ compiler> --cc=<path of your c compiler> <other options>
+@end example
 
 @node Make targets
 @section Make targets
-- 
1.7.5.4


[-- Attachment #2: Type: text/html, Size: 11977 bytes --]

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-07 20:32   ` Programmingkid
@ 2015-08-07 23:13     ` Peter Maydell
  2015-08-08  1:06       ` Programmingkid
  0 siblings, 1 reply; 10+ messages in thread
From: Peter Maydell @ 2015-08-07 23:13 UTC (permalink / raw)
  To: Programmingkid; +Cc: qemu-devel qemu-devel

On 7 August 2015 at 21:32, Programmingkid <programmingkidx@gmail.com> wrote:
>
> On Aug 7, 2015, at 1:00 PM, Peter Maydell wrote:
>> (https://trac.macports.org/wiki/UsingTheRightCompiler
>> suggests that XCode 3.2 for OSX 10.6 provided a non-default
>> 'clang'; I don't have a 10.6 + 3.2 system to check whether that
>> clang will build QEMU though. If it does then we should
>> recommend using it rather than building gcc. 10.5 users
>> are definitely stuck with finding or building a non-Apple
>> compiler, though.)
>
> I just checked and my system does include clang. Didn't even know
> about it. Too bad it didn't work to compile QEMU. It doesn't support
> Thread-Local Storage. It is probably too old (version 3.0).

Thanks for testing that -- you're running OSX 10.6, right?
I guess the TLS stuff got added to clang for the version
that's in Xcode 4.2. So it looks like we have a fairly
straightforward
 * 10.5 and 10.6 -> compile a gcc or download a third-party build
 * 10.7 and above -> use Xcode's clang
division.

-- PMM

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-07 23:13     ` Peter Maydell
@ 2015-08-08  1:06       ` Programmingkid
  0 siblings, 0 replies; 10+ messages in thread
From: Programmingkid @ 2015-08-08  1:06 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel qemu-devel


On Aug 7, 2015, at 7:13 PM, Peter Maydell wrote:

> On 7 August 2015 at 21:32, Programmingkid <programmingkidx@gmail.com> wrote:
>> 
>> On Aug 7, 2015, at 1:00 PM, Peter Maydell wrote:
>>> (https://trac.macports.org/wiki/UsingTheRightCompiler
>>> suggests that XCode 3.2 for OSX 10.6 provided a non-default
>>> 'clang'; I don't have a 10.6 + 3.2 system to check whether that
>>> clang will build QEMU though. If it does then we should
>>> recommend using it rather than building gcc. 10.5 users
>>> are definitely stuck with finding or building a non-Apple
>>> compiler, though.)
>> 
>> I just checked and my system does include clang. Didn't even know
>> about it. Too bad it didn't work to compile QEMU. It doesn't support
>> Thread-Local Storage. It is probably too old (version 3.0).
> 
> Thanks for testing that -- you're running OSX 10.6, right?
Yes.

> I guess the TLS stuff got added to clang for the version
> that's in Xcode 4.2. So it looks like we have a fairly
> straightforward
> * 10.5 and 10.6 -> compile a gcc or download a third-party build
> * 10.7 and above -> use Xcode's clang
> division.

Sounds right.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-07 20:57   ` [Qemu-devel] [PATCH v2] " Programmingkid
@ 2015-08-14 15:16     ` Peter Maydell
  2015-08-14 16:09       ` Programmingkid
  0 siblings, 1 reply; 10+ messages in thread
From: Peter Maydell @ 2015-08-14 15:16 UTC (permalink / raw)
  To: Programmingkid; +Cc: qemu-devel qemu-devel

On 7 August 2015 at 21:57, Programmingkid <programmingkidx@gmail.com> wrote:
> Add information on compiling source code on Mac OS X
> to the documentation.
>
> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>

> ---
>  qemu-doc.texi |   44 +++++++++++++++++++++++++++++++++++++++++---
>  1 files changed, 41 insertions(+), 3 deletions(-)
>
> diff --git a/qemu-doc.texi b/qemu-doc.texi
> index 0125bc7..c059c49 100644
> --- a/qemu-doc.texi
> +++ b/qemu-doc.texi
> @@ -3021,9 +3021,47 @@ and all other qemu-system-@var{target}.exe compiled
> for Win32.
>  @node Mac OS X
>  @section Mac OS X
>
>
>
> -The Mac OS X patches are not fully merged in QEMU, so you should look
> -at the QEMU mailing list archive to have all the necessary
> -information.
> +System Requirements:
> +@itemize
> +@item Mac OS 10.5 or higher
> +@item GCC 4.3 or higher

I think this should be "The clang compiler shipped with Xcode 4.2 or higher,
or GCC 4.3 or higher".

We should have an @item about any library dependencies we need that
aren't provided by Apple. I think that's just glib, but it's been
so long since I installed dependencies for QEMU I forget if we
need anything else third-party. (We don't need to go into detail
about where to get the third-party libs, but mentioning the
dependency is helpful I think.)

> +@end itemize
> +
> +After downloading the QEMU source code, double-click it to expand it.
> +
> +Then configure and make QEMU:
> +@example
> +./configure
> +make
> +@end example
> +
> +If you have a recent version of Mac OS X (OSX 10.7 or better
> +with Xcode 4.2 or better) we recommend building QEMU with the
> +default compiler provided by Apple, for your version of Mac OS X
> +(which will be 'clang'). The configure script will
> +automatically pick this.
> +
> +Note: If after the configure step you see a message like this:
> +@example
> +ERROR: Your compiler does not support the __thread specifier for
> +       Thread-Local Storage (TLS). Please upgrade to a version that does.
> +@end example
> +You may have to build your own version of gcc from source. Expect that to
> take

this should be lower case "you"

> +several hours. More information can be found here:
> +@uref{https://gcc.gnu.org/install/} @*
> +
> +These are some of the third party binaries of gcc available for download:
> +@itemize
> +@item Homebrew: @uref{http://brew.sh/}
> +@item @uref{https://www.litebeam.net/gcc/gcc_472.pkg}
> +@item @uref{http://www.macports.org/ports.php?by=name&substr=gcc}
> +@end itemize
> +
> +You can have several versions of GCC on your system. To specify a certain
> version,
> +use the --cc and --cxx options.
> +@example
> +./configure --cxx=<path of your c++ compiler> --cc=<path of your c
> compiler> <other options>
> +@end example

Otherwise this looks good.

thanks
-- PMM

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-14 15:16     ` Peter Maydell
@ 2015-08-14 16:09       ` Programmingkid
  2015-08-14 16:28         ` Peter Maydell
  0 siblings, 1 reply; 10+ messages in thread
From: Programmingkid @ 2015-08-14 16:09 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel qemu-devel


On Aug 14, 2015, at 11:16 AM, Peter Maydell wrote:

> On 7 August 2015 at 21:57, Programmingkid <programmingkidx@gmail.com> wrote:
>> Add information on compiling source code on Mac OS X
>> to the documentation.
>> 
>> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
> 
>> ---
>> qemu-doc.texi |   44 +++++++++++++++++++++++++++++++++++++++++---
>> 1 files changed, 41 insertions(+), 3 deletions(-)
>> 
>> diff --git a/qemu-doc.texi b/qemu-doc.texi
>> index 0125bc7..c059c49 100644
>> --- a/qemu-doc.texi
>> +++ b/qemu-doc.texi
>> @@ -3021,9 +3021,47 @@ and all other qemu-system-@var{target}.exe compiled
>> for Win32.
>> @node Mac OS X
>> @section Mac OS X
>> 
>> 
>> 
>> -The Mac OS X patches are not fully merged in QEMU, so you should look
>> -at the QEMU mailing list archive to have all the necessary
>> -information.
>> +System Requirements:
>> +@itemize
>> +@item Mac OS 10.5 or higher
>> +@item GCC 4.3 or higher
> 
> I think this should be "The clang compiler shipped with Xcode 4.2 or higher,
> or GCC 4.3 or higher".
Sounds good.

> 
> We should have an @item about any library dependencies we need that
> aren't provided by Apple. I think that's just glib, but it's been
> so long since I installed dependencies for QEMU I forget if we
> need anything else third-party. (We don't need to go into detail
> about where to get the third-party libs, but mentioning the
> dependency is helpful I think.)
That is a really good idea. This is probably what a mac user will have to install 
when installing QEMU for the first time:
- libffi
- gettext
- glib
- pgk-config
- autoconfig
- automake
- libtool
- pixman

This is all I had to install when installing QEMU onto Mac OS 10.9 for the first time. 

I could put this information in its own section like this:

Additional requirements (install in order)
1. libffi
2. gettext
3. glib
4. pgk-config
5. autoconfig
6. automake
7. libtool
8. pixman

Each item will be a hyperlink to each item's web site. Does this sound right?

> 
>> +@end itemize
>> +
>> +After downloading the QEMU source code, double-click it to expand it.
>> +
>> +Then configure and make QEMU:
>> +@example
>> +./configure
>> +make
>> +@end example
>> +
>> +If you have a recent version of Mac OS X (OSX 10.7 or better
>> +with Xcode 4.2 or better) we recommend building QEMU with the
>> +default compiler provided by Apple, for your version of Mac OS X
>> +(which will be 'clang'). The configure script will
>> +automatically pick this.
>> +
>> +Note: If after the configure step you see a message like this:
>> +@example
>> +ERROR: Your compiler does not support the __thread specifier for
>> +       Thread-Local Storage (TLS). Please upgrade to a version that does.
>> +@end example
>> +You may have to build your own version of gcc from source. Expect that to
>> take
> 
> this should be lower case "you"
> 
>> +several hours. More information can be found here:
>> +@uref{https://gcc.gnu.org/install/} @*
>> +
>> +These are some of the third party binaries of gcc available for download:
>> +@itemize
>> +@item Homebrew: @uref{http://brew.sh/}
>> +@item @uref{https://www.litebeam.net/gcc/gcc_472.pkg}
>> +@item @uref{http://www.macports.org/ports.php?by=name&substr=gcc}
>> +@end itemize
>> +
>> +You can have several versions of GCC on your system. To specify a certain
>> version,
>> +use the --cc and --cxx options.
>> +@example
>> +./configure --cxx=<path of your c++ compiler> --cc=<path of your c
>> compiler> <other options>
>> +@end example
> 
> Otherwise this looks good.
> 
> thanks
> -- PMM

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-14 16:09       ` Programmingkid
@ 2015-08-14 16:28         ` Peter Maydell
  2015-08-14 16:45           ` Programmingkid
  0 siblings, 1 reply; 10+ messages in thread
From: Peter Maydell @ 2015-08-14 16:28 UTC (permalink / raw)
  To: Programmingkid; +Cc: qemu-devel qemu-devel

On 14 August 2015 at 17:09, Programmingkid <programmingkidx@gmail.com> wrote:
>
> On Aug 14, 2015, at 11:16 AM, Peter Maydell wrote:
>
>> On 7 August 2015 at 21:57, Programmingkid <programmingkidx@gmail.com> wrote:
>>> Add information on compiling source code on Mac OS X
>>> to the documentation.
>>>
>>> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
>>
>>> ---
>>> qemu-doc.texi |   44 +++++++++++++++++++++++++++++++++++++++++---
>>> 1 files changed, 41 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/qemu-doc.texi b/qemu-doc.texi
>>> index 0125bc7..c059c49 100644
>>> --- a/qemu-doc.texi
>>> +++ b/qemu-doc.texi
>>> @@ -3021,9 +3021,47 @@ and all other qemu-system-@var{target}.exe compiled
>>> for Win32.
>>> @node Mac OS X
>>> @section Mac OS X
>>>
>>>
>>>
>>> -The Mac OS X patches are not fully merged in QEMU, so you should look
>>> -at the QEMU mailing list archive to have all the necessary
>>> -information.
>>> +System Requirements:
>>> +@itemize
>>> +@item Mac OS 10.5 or higher
>>> +@item GCC 4.3 or higher
>>
>> I think this should be "The clang compiler shipped with Xcode 4.2 or higher,
>> or GCC 4.3 or higher".
> Sounds good.
>
>>
>> We should have an @item about any library dependencies we need that
>> aren't provided by Apple. I think that's just glib, but it's been
>> so long since I installed dependencies for QEMU I forget if we
>> need anything else third-party. (We don't need to go into detail
>> about where to get the third-party libs, but mentioning the
>> dependency is helpful I think.)
> That is a really good idea. This is probably what a mac user will have to install
> when installing QEMU for the first time:
> - libffi
> - gettext
> - glib
> - pgk-config

"pkg-config"

> - autoconfig

"autoconf"

> - automake
> - libtool
> - pixman

> This is all I had to install when installing QEMU onto Mac OS 10.9 for the first time.
>
> I could put this information in its own section like this:
>
> Additional requirements (install in order)
> 1. libffi
> 2. gettext
> 3. glib
> 4. pgk-config
> 5. autoconfig
> 6. automake
> 7. libtool
> 8. pixman
>
> Each item will be a hyperlink to each item's web site. Does this sound right?

Yes, that sounds good.

. We could add a note at the end too:
"You may find it easiest to get these from a third-party packager
such as Homebrew, Macports or Fink."

-- PMM

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X
  2015-08-14 16:28         ` Peter Maydell
@ 2015-08-14 16:45           ` Programmingkid
  0 siblings, 0 replies; 10+ messages in thread
From: Programmingkid @ 2015-08-14 16:45 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel qemu-devel


On Aug 14, 2015, at 12:28 PM, Peter Maydell wrote:

> On 14 August 2015 at 17:09, Programmingkid <programmingkidx@gmail.com> wrote:
>> 
>> On Aug 14, 2015, at 11:16 AM, Peter Maydell wrote:
>> 
>>> On 7 August 2015 at 21:57, Programmingkid <programmingkidx@gmail.com> wrote:
>>>> Add information on compiling source code on Mac OS X
>>>> to the documentation.
>>>> 
>>>> Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
>>> 
>>>> ---
>>>> qemu-doc.texi |   44 +++++++++++++++++++++++++++++++++++++++++---
>>>> 1 files changed, 41 insertions(+), 3 deletions(-)
>>>> 
>>>> diff --git a/qemu-doc.texi b/qemu-doc.texi
>>>> index 0125bc7..c059c49 100644
>>>> --- a/qemu-doc.texi
>>>> +++ b/qemu-doc.texi
>>>> @@ -3021,9 +3021,47 @@ and all other qemu-system-@var{target}.exe compiled
>>>> for Win32.
>>>> @node Mac OS X
>>>> @section Mac OS X
>>>> 
>>>> 
>>>> 
>>>> -The Mac OS X patches are not fully merged in QEMU, so you should look
>>>> -at the QEMU mailing list archive to have all the necessary
>>>> -information.
>>>> +System Requirements:
>>>> +@itemize
>>>> +@item Mac OS 10.5 or higher
>>>> +@item GCC 4.3 or higher
>>> 
>>> I think this should be "The clang compiler shipped with Xcode 4.2 or higher,
>>> or GCC 4.3 or higher".
>> Sounds good.
>> 
>>> 
>>> We should have an @item about any library dependencies we need that
>>> aren't provided by Apple. I think that's just glib, but it's been
>>> so long since I installed dependencies for QEMU I forget if we
>>> need anything else third-party. (We don't need to go into detail
>>> about where to get the third-party libs, but mentioning the
>>> dependency is helpful I think.)
>> That is a really good idea. This is probably what a mac user will have to install
>> when installing QEMU for the first time:
>> - libffi
>> - gettext
>> - glib
>> - pgk-config
> 
> "pkg-config"
> 
>> - autoconfig
> 
> "autoconf"
> 
>> - automake
>> - libtool
>> - pixman
> 
>> This is all I had to install when installing QEMU onto Mac OS 10.9 for the first time.
>> 
>> I could put this information in its own section like this:
>> 
>> Additional requirements (install in order)
>> 1. libffi
>> 2. gettext
>> 3. glib
>> 4. pgk-config
>> 5. autoconfig
>> 6. automake
>> 7. libtool
>> 8. pixman
>> 
>> Each item will be a hyperlink to each item's web site. Does this sound right?
> 
> Yes, that sounds good.
> 
> . We could add a note at the end too:
> "You may find it easiest to get these from a third-party packager
> such as Homebrew, Macports or Fink."
That might be a possibility for many of these packages. 

Will send in a new patch with the added suggestions above.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2015-08-14 16:46 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-06 15:35 [Qemu-devel] [PATCH] qemu-doc.texi: Add information on compiling source code on Mac OS X Programmingkid
2015-08-07 17:00 ` Peter Maydell
2015-08-07 20:32   ` Programmingkid
2015-08-07 23:13     ` Peter Maydell
2015-08-08  1:06       ` Programmingkid
2015-08-07 20:57   ` [Qemu-devel] [PATCH v2] " Programmingkid
2015-08-14 15:16     ` Peter Maydell
2015-08-14 16:09       ` Programmingkid
2015-08-14 16:28         ` Peter Maydell
2015-08-14 16:45           ` Programmingkid

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).