GNOME Bugzilla – Bug 790177
Various meson related improvements
Last modified: 2017-11-14 00:32:21 UTC
There are new meson porting guidelines[0] for naming the options, so option names should be changed. bijiben also uses `1` as the project's default warning level. However, meson already uses this level as the default warning level. [0] https://wiki.gnome.org/Initiatives/GnomeGoals/MesonPorting
Created attachment 363356 [details] [review] build: Rename build options Following the new meson porting guidelines[0], this patch renames the build options. The list of changes is as follows: - Remove the `enable` prefix from boolean options. - The character separator from multi-word options has been changed to underscore (`_`). [0] https://wiki.gnome.org/Initiatives/GnomeGoals/MesonPorting
Created attachment 363357 [details] [review] build: Remove default warning level bijiben uses `1` as the project's default warning level. However, meson already uses this level as the default warning level, so it's actually unnecessary. This patch removes the warning level from project's default options.
Created attachment 363531 [details] [review] build: Migrate to new meson porting guidelines This patch also removes the `-Werror` compiler flag, as it is already included in meson's warning levels.
Review of attachment 363357 [details] [review]: Thanks
Review of attachment 363531 [details] [review]: Thanks.