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

Reply via email to