Re: [Qemu-devel] [PATCH v2] qemu-doc.texi: Add information on compiling source code on Mac OS X

[Peter Maydell]

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

> ---
>  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 address@hidden 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:
> address@hidden
> address@hidden Mac OS 10.5 or higher
> address@hidden 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.)

> address@hidden itemize
> +
> +After downloading the QEMU source code, double-click it to expand it.
> +
> +Then configure and make QEMU:
> address@hidden
> +./configure
> +make
> address@hidden 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:
> address@hidden
> +ERROR: Your compiler does not support the __thread specifier for
> +       Thread-Local Storage (TLS). Please upgrade to a version that does.
> address@hidden 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:
> address@hidden://gcc.gnu.org/install/} @*
> +
> +These are some of the third party binaries of gcc available for download:
> address@hidden
> address@hidden Homebrew: @uref{http://brew.sh/}
> address@hidden @uref{https://www.litebeam.net/gcc/gcc_472.pkg}
> address@hidden @uref{http://www.macports.org/ports.php?by=name&substr=gcc}
> address@hidden itemize
> +
> +You can have several versions of GCC on your system. To specify a certain
> version,
> +use the --cc and --cxx options.
> address@hidden
> +./configure --cxx=<path of your c++ compiler> --cc=<path of your c
> compiler> <other options>
> address@hidden example

Otherwise this looks good.

thanks
-- PMM