mathgl @2.4.4_4: /export.cpp:187:30: error: use of undeclared identifier 'TRUE'

[kencu (Ken)]

mathgl is not rebuilding after the recent change to libjpeg-turbo:

:info:build cd /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_math_mathgl/mathgl/work/mathgl-2.4.4/src && /opt/local/bin/clang++-mp-9.0 -DMGL_HAVE_GSL2 -DMGL_SRC -DNO_COLOR_ARRAY -Dmgl_EXPORTS -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_math_mathgl/mathgl/work/mathgl-2.4.4/include -I/opt/local/include -F//System/Library/Frameworks -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_math_mathgl/mathgl/work/mathgl-2.4.4/src/prc -pipe -Os -stdlib=libc++ -DNDEBUG -arch x86_64 -mmacosx-version-min=10.6 -fPIC -fvisibility=hidden -fvisibility-inlines-hidden -o CMakeFiles/mgl.dir/fit.cpp.o -c /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_math_mathgl/mathgl/work/mathgl-2.4.4/src/fit.cpp
:info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_math_mathgl/mathgl/work/mathgl-2.4.4/src/export.cpp:187:30: error: use of undeclared identifier 'TRUE'
:info:build         jpeg_start_compress(&cinfo, TRUE);
:info:build                                     ^

this is on 10.6.8, building with clang-5.0 or clang-9.0, which may or may not be relevant.

[kencu (Ken)]

well, it seems to go to trouble to make sure TRUE is undefined, before using it later...

#if MGL_HAVE_GIF
#include <gif_lib.h>
#undef TRUE
#undef FALSE
#endif

That seems weird.

and at other places, it doesn't like to use TRUE but seems ok to use true:

void MGL_EXPORT mgl_write_png(HMGL gr, const char *fname,const char *)
{
	long w,h;	unsigned char *f=0, **p=0;
	p =_Gr_->GetRGBLines(w,h,f,true);
	if(p)

this all smells like something rather hackish...

[Schamschula (Marius Schamschula)]

It sure does look like a hack! Let me see if this happens on a newer version of macOS.

[Schamschula (Marius Schamschula)]

I get the same error on Mojave.

[Schamschula (Marius Schamschula)]

It looks like I had a patch file to deal with jpeg, which is no longer needed with libjpeg-turbo. No need to rev bump, as mathgl shouldn't have built correctly for any OS version after the change to libjpeg-turbo.

[Schamschula (Marius Schamschula)]

In 8ec4acf0c8f0c38be971523a4f621778d0571d6a/macports-ports (master):

mathgl: remove patch obsoleted by libjpeg-turbo

Closes: #62199

[kencu (Ken)]

thanks!