Pretty much finished hacking configure. Any requests?

On 11/06/2012 11:16 AM, David Billinghurst wrote:
> On 11/06/2012 5:24 AM, Jaime Villate wrote:
>> On 06/10/2012 06:31 PM, Jaime Villate wrote:
>>> On 06/09/2012 08:05 AM, David Billinghurst wrote:
>>>> I have pretty much finished hacking the configure scripts.  Please 
>>>> let me know if I have broken anything. 
>>> Hi David,
>>> thank you for your work on this, which is very important. I have 
>>> used the following configuration in Ubuntu:
>>>
>>> ./configure --enable-sbcl-exec
>>>
>>> and everything worked fine at the beginning and the maxima sbcl 
>>> image was built correctly, but after
>>> that, make started to do things that should only be done for the 
>>> install target in Windows. Namely, it tried
>>> to build a Starkit for xmaxima and, of course, it failed with an 
>>> error message:
>>>     /c/programs/star/tclkitsh-win32.upx.exe: not found
>> As a matter of fact, the same error appears even with one of the old 
>> targets, such as --enable-sbcl. In other
>> words, the autoconf/automake process is currently broken.
>> Also, "make distclean" fails because there is no distclean target in 
>> interfaces/xmaxima/win32
>>
>> Regards,
>> Jaime
>>
> Thanks for the report and the testing.  I will have a look at this 
> tonight.
>
> Yesterday I added options --enable-chm and --enable-xmaxima-exe for 
> the windows stuff.
> These are (supposed to be) disabled by default on all platforms, to 
> avoid the issues
> you have reported.  Must be a mistake somewhere.
>
> I didn't think to test "make distclean".  I have checked in a fix for 
> that.  Using
> automake in interfaces/xmaxima/win32 would be a better solution.  It 
> should be possible now.
>

I have committed the following.  It should work around the problem in 
the short term.

diff --git a/interfaces/xmaxima/Makefile.am b/interfaces/xmaxima/Makefile.am
index 706c0cb..d856aef 100755
--- a/interfaces/xmaxima/Makefile.am
+++ b/interfaces/xmaxima/Makefile.am
@@ -6,9 +6,12 @@
  include $(top_srcdir)/common.mk

  SUBDIRS = msgs doc
+# FIXME: The double conditional shouldn't be required. DB 2012-06-11
+if WIN32
  if XMAXIMA_EXE
  SUBDIRS += win32
  endif
+endif

  genericdir = $(xmaximadir)