* [PATCH v3] INSTALL: correct EXTRA_CFLAGS handling
@ 2014-11-27 9:26 Olaf Hering
2014-11-27 15:17 ` Konrad Rzeszutek Wilk
0 siblings, 1 reply; 3+ messages in thread
From: Olaf Hering @ 2014-11-27 9:26 UTC (permalink / raw)
To: xen-devel; +Cc: Olaf Hering, Ian Jackson, Ian Campbell
The already documented configure patch was not applied.
Adjust documentation to describe existing behaviour.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
---
v3: reword as suggested by Konrad, trim Cc list
v2: resend due to lack of Cc: tags
INSTALL | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/INSTALL b/INSTALL
index 6bb9d23..0bc67ea 100644
--- a/INSTALL
+++ b/INSTALL
@@ -128,13 +128,6 @@ original xenstored will be used. Valid names are xenstored and
oxenstored.
--with-xenstored=name
-Using additional CFLAGS to build tools running in dom0 is required when
-building distro packages. This is the option to pass things like
-RPM_OPT_FLAGS.
- --with-extra-cflags-tools=EXTRA_CFLAGS
- --with-extra-cflags-qemu-traditional=EXTRA_CFLAGS
- --with-extra-cflags-qemu-upstream=EXTRA_CFLAGS
-
Instead of starting the tools in dom0 with sysv runlevel scripts they
can also be started by systemd. If this option is enabled xenstored will
receive the communication socked directly from systemd. So starting it
@@ -241,6 +234,13 @@ QEMU_UPSTREAM_URL=
QEMU_TRADITIONAL_URL=
SEABIOS_UPSTREAM_URL=
+Using additional CFLAGS to build tools which will run in dom0 is
+required when building distro packages. These variables can be used to
+pass RPM_OPT_FLAGS.
+EXTRA_CFLAGS_XEN_TOOLS=
+EXTRA_CFLAGS_QEMU_TRADITIONAL=
+EXTRA_CFLAGS_QEMU_XEN=
+
This variable can be used to use DIR/include and DIR/lib during build.
This is the same as PREPEND_LIB and PREPEND_INCLUDES. APPEND_LIB and
APPEND_INCLUDES= will be appended to the CFLAGS/LDFLAGS variable.
@@ -310,10 +310,10 @@ sudo make install BOOT_DIR=/ood/path/boot EFI_DIR=/odd/path/efi
%build
export WGET=$(type -P false)
export GIT=$(type -P false)
+export EXTRA_CFLAGS_XEN_TOOLS="$RPM_OPT_FLAGS"
+export EXTRA_CFLAGS_QEMU_TRADITIONAL="$RPM_OPT_FLAGS"
+export EXTRA_CFLAGS_QEMU_XEN="$RPM_OPT_FLAGS"
%configure \
- --with-extra-cflags-tools="$RPM_OPT_FLAGS" \
- --with-extra-cflags-qemu-traditional="$RPM_OPT_FLAGS" \
- --with-extra-cflags-qemu-upstream="$RPM_OPT_FLAGS" \
--with-initddir=%{_initddir}
unset CFLAGS CXXFLAGS FFLAGS LDFLAGS
make
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] INSTALL: correct EXTRA_CFLAGS handling
2014-11-27 9:26 [PATCH v3] INSTALL: correct EXTRA_CFLAGS handling Olaf Hering
@ 2014-11-27 15:17 ` Konrad Rzeszutek Wilk
2014-11-28 12:10 ` Ian Campbell
0 siblings, 1 reply; 3+ messages in thread
From: Konrad Rzeszutek Wilk @ 2014-11-27 15:17 UTC (permalink / raw)
To: Olaf Hering, xen-devel; +Cc: Ian Jackson, Ian Campbell
On November 27, 2014 4:26:26 AM EST, Olaf Hering <olaf@aepfle.de> wrote:
>The already documented configure patch was not applied.
>Adjust documentation to describe existing behaviour.
>
>Signed-off-by: Olaf Hering <olaf@aepfle.de>
>Cc: Ian Campbell <ian.campbell@citrix.com>
>Cc: Ian Jackson <ian.jackson@eu.citrix.com>
>Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Reviewed-by: me.
Don't need an release ack for it.
>---
>v3: reword as suggested by Konrad, trim Cc list
>v2: resend due to lack of Cc: tags
>
> INSTALL | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
>diff --git a/INSTALL b/INSTALL
>index 6bb9d23..0bc67ea 100644
>--- a/INSTALL
>+++ b/INSTALL
>@@ -128,13 +128,6 @@ original xenstored will be used. Valid names are
>xenstored and
> oxenstored.
> --with-xenstored=name
>
>-Using additional CFLAGS to build tools running in dom0 is required
>when
>-building distro packages. This is the option to pass things like
>-RPM_OPT_FLAGS.
>- --with-extra-cflags-tools=EXTRA_CFLAGS
>- --with-extra-cflags-qemu-traditional=EXTRA_CFLAGS
>- --with-extra-cflags-qemu-upstream=EXTRA_CFLAGS
>-
> Instead of starting the tools in dom0 with sysv runlevel scripts they
>can also be started by systemd. If this option is enabled xenstored
>will
> receive the communication socked directly from systemd. So starting it
>@@ -241,6 +234,13 @@ QEMU_UPSTREAM_URL=
> QEMU_TRADITIONAL_URL=
> SEABIOS_UPSTREAM_URL=
>
>+Using additional CFLAGS to build tools which will run in dom0 is
>+required when building distro packages. These variables can be used to
>+pass RPM_OPT_FLAGS.
>+EXTRA_CFLAGS_XEN_TOOLS=
>+EXTRA_CFLAGS_QEMU_TRADITIONAL=
>+EXTRA_CFLAGS_QEMU_XEN=
>+
> This variable can be used to use DIR/include and DIR/lib during build.
> This is the same as PREPEND_LIB and PREPEND_INCLUDES. APPEND_LIB and
> APPEND_INCLUDES= will be appended to the CFLAGS/LDFLAGS variable.
>@@ -310,10 +310,10 @@ sudo make install BOOT_DIR=/ood/path/boot
>EFI_DIR=/odd/path/efi
> %build
> export WGET=$(type -P false)
> export GIT=$(type -P false)
>+export EXTRA_CFLAGS_XEN_TOOLS="$RPM_OPT_FLAGS"
>+export EXTRA_CFLAGS_QEMU_TRADITIONAL="$RPM_OPT_FLAGS"
>+export EXTRA_CFLAGS_QEMU_XEN="$RPM_OPT_FLAGS"
> %configure \
>- --with-extra-cflags-tools="$RPM_OPT_FLAGS" \
>- --with-extra-cflags-qemu-traditional="$RPM_OPT_FLAGS" \
>- --with-extra-cflags-qemu-upstream="$RPM_OPT_FLAGS" \
> --with-initddir=%{_initddir}
> unset CFLAGS CXXFLAGS FFLAGS LDFLAGS
> make
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] INSTALL: correct EXTRA_CFLAGS handling
2014-11-27 15:17 ` Konrad Rzeszutek Wilk
@ 2014-11-28 12:10 ` Ian Campbell
0 siblings, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2014-11-28 12:10 UTC (permalink / raw)
To: Konrad Rzeszutek Wilk; +Cc: Olaf Hering, Ian Jackson, xen-devel
On Thu, 2014-11-27 at 10:17 -0500, Konrad Rzeszutek Wilk wrote:
> On November 27, 2014 4:26:26 AM EST, Olaf Hering <olaf@aepfle.de> wrote:
> >The already documented configure patch was not applied.
> >Adjust documentation to describe existing behaviour.
> >
> >Signed-off-by: Olaf Hering <olaf@aepfle.de>
> >Cc: Ian Campbell <ian.campbell@citrix.com>
> >Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> >Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
>
> Reviewed-by: me.
>
> Don't need an release ack for it.
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-28 12:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-27 9:26 [PATCH v3] INSTALL: correct EXTRA_CFLAGS handling Olaf Hering
2014-11-27 15:17 ` Konrad Rzeszutek Wilk
2014-11-28 12:10 ` Ian Campbell
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.