[libvirt] [PATCH] Update how to compile with -Werror
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests: -- 1.7.8.6
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests:
ACK Jirka
On 09/25/2012 09:38 AM, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests:
ACK
Jirka
Don't forget to change it in the HACKING as well ;) Martin
On Tue, Sep 25, 2012 at 11:16:32 +0200, Martin Kletzander wrote:
On 09/25/2012 09:38 AM, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests:
ACK
Jirka
Don't forget to change it in the HACKING as well ;)
That file is autogenerated from hacking.html.in ;) Jirka
On 09/25/2012 11:23 AM, Jiri Denemark wrote:
On Tue, Sep 25, 2012 at 11:16:32 +0200, Martin Kletzander wrote:
On 09/25/2012 09:38 AM, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests:
ACK
Jirka
Don't forget to change it in the HACKING as well ;)
That file is autogenerated from hacking.html.in ;)
NO WAY! Every day there's something to learn =) Sorry for the confusion, Doug. Martin
On 09/25/2012 03:30 AM, Martin Kletzander wrote:
On 09/25/2012 11:23 AM, Jiri Denemark wrote:
On Tue, Sep 25, 2012 at 11:16:32 +0200, Martin Kletzander wrote:
On 09/25/2012 09:38 AM, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
ACK
Jirka
Don't forget to change it in the HACKING as well ;)
That file is autogenerated from hacking.html.in ;)
Yes, but only after 'make', and since it is one of the few version-controlled generated files, it DOES need to be touched as part of this commit, so that fresh checkouts aren't dirty. Given Dan's reminder, I'm squashing this in, regenerating HACKING, and pushing the patch. diff --git i/docs/hacking.html.in w/docs/hacking.html.in index c3f859c..9de5eb1 100644 --- i/docs/hacking.html.in +++ w/docs/hacking.html.in @@ -74,7 +74,9 @@ <li>Make sure your patches apply against libvirt GIT. Developers only follow GIT and don't care much about released versions.</li> <li><p>Run the automated tests on your code before submitting any changes. - In particular, configure with compile warnings set to -Werror:</p> + In particular, configure with compile warnings set to + -Werror. This is done automatically for a git checkout; from a + tarball, use:</p> <pre> ./configure --enable-werror </pre> -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
On Tue, Sep 25, 2012 at 10:11 AM, Eric Blake <eblake@redhat.com> wrote:
On 09/25/2012 03:30 AM, Martin Kletzander wrote:
On 09/25/2012 11:23 AM, Jiri Denemark wrote:
On Tue, Sep 25, 2012 at 11:16:32 +0200, Martin Kletzander wrote:
On 09/25/2012 09:38 AM, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
ACK
Jirka
Don't forget to change it in the HACKING as well ;)
That file is autogenerated from hacking.html.in ;)
Yes, but only after 'make', and since it is one of the few version-controlled generated files, it DOES need to be touched as part of this commit, so that fresh checkouts aren't dirty.
Given Dan's reminder, I'm squashing this in, regenerating HACKING, and pushing the patch.
diff --git i/docs/hacking.html.in w/docs/hacking.html.in index c3f859c..9de5eb1 100644 --- i/docs/hacking.html.in +++ w/docs/hacking.html.in @@ -74,7 +74,9 @@ <li>Make sure your patches apply against libvirt GIT. Developers only follow GIT and don't care much about released versions.</li> <li><p>Run the automated tests on your code before submitting any changes. - In particular, configure with compile warnings set to -Werror:</p> + In particular, configure with compile warnings set to + -Werror. This is done automatically for a git checkout; from a + tarball, use:</p> <pre> ./configure --enable-werror </pre>
--
Ah. And I actually pulled it out of the commit before sending it along because I saw it was autogenerated. But your rationale makes sense. Thanks guys. -- Doug Goldstein
On Tue, Sep 25, 2012 at 09:38:42AM +0200, Jiri Denemark wrote:
On Mon, Sep 24, 2012 at 18:44:34 -0500, Doug Goldstein wrote:
--enable-compile-warnings=error has been renamed to --enable-werror so update the HACKING and the hacking.html to reflect that. --- docs/hacking.html.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/docs/hacking.html.in b/docs/hacking.html.in index a97dc22..c3f859c 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -76,7 +76,7 @@ <li><p>Run the automated tests on your code before submitting any changes. In particular, configure with compile warnings set to -Werror:</p> <pre> - ./configure --enable-compile-warnings=error + ./configure --enable-werror </pre> <p> and run the tests:
ACK
BTW, if you are running ./configure from a GIT checkout, then the --enable-werror flag is automaticaly enabled without asking. You only need to manually specify it if building fromm a tar.gz dist Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
participants (5)
-
Daniel P. Berrange -
Doug Goldstein -
Eric Blake -
Jiri Denemark -
Martin Kletzander