Rename --no-error to --werror and flip default
authorNirbheek Chauhan <nirbheek@centricular.com>
Wed, 5 Sep 2018 12:00:27 +0000 (17:30 +0530)
committerNirbheek Chauhan <nirbheek@centricular.com>
Wed, 5 Sep 2018 12:04:11 +0000 (17:34 +0530)
We should not default to -Werror because that's not what we default to
anywhere in gstreamer, and it's bad for releases anyway. The CI will
be fixed to pass --werror manually.

setup.py

index 5f286b9..fccf416 100755 (executable)
--- a/setup.py
+++ b/setup.py
@@ -23,9 +23,9 @@ class GstBuildConfigurer:
         self.args = args
 
     def get_configs(self):
-        if self.options.no_error:
-            return []
-        return ['--werror']
+        if self.options.werror:
+            return ['--werror']
+        return []
 
     def configure_meson(self):
         if not self.options.reconfigure:
@@ -74,7 +74,7 @@ if __name__ == "__main__":
                         ' You can also use `ninja reconfigure` to just'
                         ' make sure meson is rerun but the build folder'
                         ' is kept.')
-    parser.add_argument("--no-error", action='store_true',
+    parser.add_argument("--werror", action='store_true',
                         default=False, help="Do not error out on warnings")
 
     options, args = parser.parse_known_args()