On Monday 04 March 2002 11:20 am, Jean-Marc Lasgouttes wrote: > >>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: > > Angus> That would be good, Jean-Marc. Can I leave this in your capable > Angus> hands? > > Angus> I dunno what to call the variables. Something like: > > Angus> #define USE_XFORMS_IMAGE_LOADER > Angus> #define USE_XFORMS_PS_IMAGE_LOADER > > Autoconf rather wants to work by checking for functions. > > Angus> The test for USE_XFORMS_IMAGE_LOADER: > > Would HAVE_FLIMAGE_DUP be enough or do you also need to test for > flimage_to_pixmap()?
Ok, understood. Let's be safe and check for both. There's no need to perform these checks at all if we're using the qt2 or gnome frontends. They'll get the crappy loader by default till they write there own! Is that too cruel? We'd use these two tests to decide whether to add xformsGImage.[Ch] to frontends/xforms/Makefile.am or not. If we do so then we don't need GraphicsImageXPM.[Ch] in graphics/Makefile.am We'd also need something like the following in frontends/xforms/GUIRunTime.C #ifdef HAVE_FLIMAGE_DUP && HAVE_FLIMAGE_TO_PIXMAP ... #else ... #endif > Angus> If that passes, then use this one for > Angus> USE_XFORMS_PS_IMAGE_LOADER > > Angus> #include FORMS_H_LOCATION int main () { flimage_enable_ps(); > Angus> return 0; } > > So HAVE_FLIMAGE_ENABLE_PS would be fine? Yes. There's no need to check for this if the above two tests don't succeed. It'd be used within xformGImage.C #ifdef HAVE_FLIMAGE_ENABLE_PS flimage_enable_ps(); #endif > Supporting these is just one more line in configure.in > > JMarc If you tell me what the line is I'll do it. You'll also need to show me how to add things selectively to Makefile.am Yuch. Sounds complicated to me. Perhaps it's easier for you ;-) Angus