tsdconseil / opencv-demonstrator

OpenCV demonstrator (GUI)
http://www.tsdconseil.fr/log/opencv/demo/index-en.html
GNU Lesser General Public License v3.0
141 stars 57 forks source link

Why do we need COPTADD in the file libcutil/data/std-makefile-defs ? #10

Closed valera-rozuvan closed 8 years ago

valera-rozuvan commented 8 years ago

The file libcutil/data/std-makefile-defs [link] has the following variable:

COPTADD  = -DLINUX -DLINUX_RUN -fPIC -isystem /usr/include/gtkmm-2.4

Why do we need it, and why does it have gtkmm-2.4?

valera-rozuvan commented 8 years ago

@juliena82 Any insight on this?

juliena82 commented 8 years ago

-isystem ... is not a necessity, it's just to accelerate the make process. And yes, this is not up to date! (should be gtkmm-3.0). I will update it someday. Anyway it does no harm. -fPIC i don't remember now (you can look in the GCC or LD options for more détails). -DLINUX is for the sources files to know we are Under linux and not Windows. -DLINUX_RUN is surely an artefact of old code (but should not be removed until verfication).

shervinemami commented 8 years ago

I believe -fPIC is needed if you want to create shared libraries (.so files) with GCC, so you should leave it there if it's already working.

valera-rozuvan commented 8 years ago

I have updated LINUX/I386 TARGET part of libcutil/data/std-makefile-defs file.