C_WARNINGS ignored with CModule shared makefile?

I’m working on a module that uses the TCP/IP libraries, resulting in a lot of “non-ANSI include” warnings. The flag to disable these is -Wp.

I’ve added C_WARNINGS = -Wp to my makefile, below the include CModule line. However, this doesn’t seem to have any effect. If I change the include to CApp and run AMU then it compiles without the ANSI warnings (obviously failing further down the line since it’s supposed to be a module). If I remove -Wp and recompile then the warnings pop back up.

So, everything’s working fine with the CApp makefile, but it appears that the C_WARNINGS line is being ignored by CModule. I’ve had a cursory look at the CModule makefile itself and it doesn’t contain C_WARNINGS at all, seemingly passing everything off to StdTools (same with CApp). I’m a bit confused!

Does anyone know what’s going on here?

I’m using DDE 27.

Does anyone know what’s going on here?

Note that C_WARNINGS is set in StdTools with :=, which I believe means it’ll be read only once and can’t be redefined. If you look at other makefiles in the OS sources they tend to use CFLAGS += -Wp or similar.

I wonder why it works when using CApp, which also inherits StdTools (the examples included with the DDE also use C_WARNINGS). But nevertheless, CFLAGS does indeed do the trick. Thanks!

I wonder why it works when using CApp

I think it’s because CModule fixes the value of C_WARNINGS when CFLAGS (which includes C_WARNINGS) is used on the right hand side in CModule. C_WARNINGS doesn’t get used in CApp.

You can override these values in CModule by overriding them before calling CModule like this:

override C_WARNINGS  :=
override C_NO_FNAMES :=

include CModule

You can set the values to anything you want. I use

override C_NO_FNAMES :=

if I want to include function names in my module – the default C_NO_FRAMES removes function names.

No I think it’s simpler than that. Its because CModule fixes the value of CFLAGS so it no longer references C_WARNINGS. If you put C_WARNINGS after CModule it is changed but has no effect.

Aha… I was only searching for “warnings” so I didn’t catch that. Good spotting!