Re: [PATCH] Fix nm-version-macro includes



On Fri, 2016-01-22 at 09:34 +0100, Adrian Freihofer wrote:
nm-version-macros.h cannot be found since include directive has
been changed from " to <. This breaks for example gnome-panel
build:
/usr/include/NetworkManager/NetworkManager.h:31:31:
fatal error: nm-version-macros.h: No such file or directory.



I don't think this is right.

public headers are supposed to be included by other projects.

From the point of view of an application that uses libnm headers,
those headers are external dependencies and to be found in a system
provided include directory. Thus the <>, not "".

I think public headers should not have "" includes at all.


What did you do exactly that you experienced the failure?


Thomas

Attachment: signature.asc
Description: This is a digitally signed message part



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]