RE: [RFC] building postgres with meson - v13 - Mailing list pgsql-hackers

From wangw.fnst@fujitsu.com
Subject RE: [RFC] building postgres with meson - v13
Date
Msg-id OS3PR01MB6275B9378C371F590978E3C49E529@OS3PR01MB6275.jpnprd01.prod.outlook.com
Whole thread Raw
In response to Re: [RFC] building postgres with meson - v13  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: [RFC] building postgres with meson - v13
List pgsql-hackers
Hi,

I tried to use meson and ninja and they are really efficient.
But when I tried to specify "c_args", it did not take effect.

Attached my steps:
[In the HEAD (7d708093b7)]
$ meson setup build --prefix /home/wangw/install/parallel_apply/ -Dcassert=true -Dtap_tests=enabled -Dicu=enabled
-Dc_args='-fno-omit-frame-pointer'

Log:
......
  Compiler Flags
    CPP FLAGS          : -D_GNU_SOURCE
    C FLAGS, functional: -fno-strict-aliasing -fwrapv -fexcess-precision=standard
    C FLAGS, warnings  : -Wmissing-prototypes -Wpointer-arith -Werror=vla -Wendif-labels -Wmissing-format-attribute
-Wimplicit-fallthrough=3-Wcast-function-type -Wformat-security -Wdeclaration-after-statement -Wno-format-truncation
-Wno-stringop-truncation
......

After I made the below modifications, the specified "c_args" took effect.
```
@@ -2439,6 +2439,10 @@ endif

 # Set up compiler / linker arguments to be used everywhere, individual targets
 # can add further args directly, or indirectly via dependencies
+
+tmp_c_args = get_option('c_args')
+cflags += tmp_c_args
+
 add_project_arguments(cflags, language: ['c'])
 add_project_arguments(cppflags, language: ['c'])
 add_project_arguments(cflags_warn, language: ['c'])
```

I might missed something. Just to confirm is there another way to add CFLAG ?

Regards,
Wang wei



pgsql-hackers by date:

Previous
From: bt22nakamorit
Date:
Subject: Differentiate MERGE queries with different structures
Next
From: Amit Kapila
Date:
Subject: Re: Fix typos in code comments