search

TuononenP / pkg-config-support-for-eclipse-cdt

Provides automation of configuration needed for projects using pkg-config, such as GTK+ and gtkmm. Very useful time-saving tool for all C, C++ programmers using Eclipse.
4 stars 2 forks source link

Symbol definition not added #20

Open mforkel opened 7 years ago

mforkel commented 7 years ago

Hi,

I just upgraded Pkg-config Support Feature to version 1.0.0.201710061415 on Eclipse Oxygen.1 Release (4.7.1).

Of the results of pkg-config

$ pkg-config --cflags fuse
-D_FILE_OFFSET_BITS=64 -I/usr/include/fuse
$ pkg-config --libs fuse
-lfuse -pthread

it successfully handles -I/usr/include/fuse and -lfuse, but does not handle -D_FILE_OFFSET_BITS=64 or -pthread (which in Eclipse is an option at C/C++ Build > Settings > GCC C Linker > General).

From @sheldonrobinson's comment to his pull request #19, merged in commit 073fcea, I had hoped that the symbol definition would be handled.

sheldonrobinson commented 5 years ago

I believe we're not populating the the CExternalSetting for ICSettingEntry.MACRO. The ids would probably be org.eclipse.cdt.core.cSource, org.eclipse.cdt.core.cHeader, org.eclipse.cdt.core.cxxHeader, org.eclipse.cdt.core.cxxSource. Also, ICSettingEntry.MACRO is associated with langId org.eclipse.cdt.core.g++