dwcaress / MB-System

MB-System is an open source software package for the processing and display of bathymetry and backscatter imagery data derived from multibeam, interferometry, and sidescan sonars.
https://www.mbari.org/products/research-software/mb-system/
Other
127 stars 45 forks source link

glib.h not found #1484

Open anbj opened 1 month ago

anbj commented 1 month ago

Tested on git head and 5.8.1.

I'm getting an error which is new to me:

[ 73%] Building C object src/mbaux/CMakeFiles/mbaux.dir/mb_readwritegrd.c.o
[ 73%] Built target mb_read_init_test
[ 77%] Built target mbview
In file included from /usr/local/include/gmt/gmt_dev.h:157,
                 from /home/anbj/MB-System/src/mbaux/mb_readwritegrd.c:39:
/usr/local/include/gmt/gmt_glib.h:36:10: fatal error: glib.h: No such file or directory
   36 | #include <glib.h>
      |          ^~~~~~~~
compilation terminated.

It seems to come from a gmt header file. I've installed the dependencies.

This 'error' may not belong to MB-System, but creating this report in hope for some tips. @joa-quim, do you know of any changes lately that may lead to this?

joa-quim commented 1 month ago

It seems that somehow this is related to what triggered https://github.com/GenericMappingTools/gmt/pull/8577 But gmt_glib.h:36 has

#ifdef HAVE_GLIB_GTHREAD
#include <glib.h>

So, if HAVE_GLIB_GTHREAD is set that means the location of glib.h should be know too. Don't understand the failure.

dwcaress commented 1 month ago

Joaquim,

I don’t have time to work on it now, but I suspect the problem is that GLIB has been found but that the CMakeList.txt file in mbsystem/src/mbaux does not add GLIB to the target include directories and link libraries lists.

Cheers, Dave

On Oct 22, 2024, at 3:07 PM, Joaquim @.***> wrote:

It seems that somehow this is related to what triggered GenericMappingTools/gmt#8577 https://github.com/GenericMappingTools/gmt/pull/8577 But gmt_glib.h:36 has

ifdef HAVE_GLIB_GTHREAD

include

So, if HAVE_GLIB_GTHREAD is set that means the location of glib.h should be know too. Don't understand the failure.

— Reply to this email directly, view it on GitHub https://github.com/dwcaress/MB-System/issues/1484#issuecomment-2430403874, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTSXXCC5QQAJPLWOOYIS4LZ43EDDAVCNFSM6AAAAABQNIQQ3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZQGQYDGOBXGQ. You are receiving this because you are subscribed to this thread.


David W. Caress Principal Engineer Seafloor Mapping Lab

Monterey Bay Aquarium Research Institute 7700 Sandholdt Road Moss Landing, CA 95039

@.*** https://www.mbari.org/person/dave-caress/ https://www.mbari.org/team/seafloor-mapping/

Phone: 831-775-1775

anbj commented 1 month ago

It's a recent thing at least. I'm pretty sure I successfully compiled MB-system just a few weeks/months ago (with bleeding edge GMT). I know for sure it worked in May, so the reason you refer to @joa-quim seems probable.