Re: about windows binaries

[Tatsuro M. <tma...@ya...>]

----- Original Message -----

> From: Bastian Märkisch 
> To: Tatsuro MATSUOKA ; gnuplot-beta
> Cc: 
> Date: 2014/6/10, Tue 20:34
> Subject: Re: about  windows binaries
> 
> 
> Am 10.06.2014 12:49, schrieb Tatsuro MATSUOKA:
>>  ----- Original Message -----
>>>  From: Tatsuro MATSUOKA
> (snip)
>> 
>>  I also noticed that installer for 64 bit would like to install  
> "C:\Program Files (x86)".
>> 
>>  However, this should be "C:\Program Files" for 64 bit binary.
>> 
>>  I have a look at gnuplot.iss but I cannot find where should be modified.
>> 
> 
> You can find the required information here:
> http://www.jrsoftware.org/ishelp/topic_setup_architecturesinstallin64bitmode.htm
> 
> According to the docs, you should add following to the [Setup] section 
> (untested):
> 
>   ArchitecturesAllowed=x64
>   ArchitecturesInstallIn64BitMode=x64
> 
> Note that with these modifications the script can no longer be used for 32bit 
> installs.
> 
> 
> Since 32bit binaries run perfectly fine on 64bit Windows machines, I still fail 
> to see why 64bit binaries are worth the huge extra effort right now. The only 
> exception is the LFS support, but that can be handled differently, see 
> Ethan's patch #675.


Thanks. 

Adding 
   ArchitecturesAllowed=x64
   ArchitecturesInstallIn64BitMode=x64

makes installer work properly on 64 bit windows. (tested on win 7 and 8).
I will have the change as my local patch.

> Since 32bit binaries run perfectly fine on 64bit Windows machines, I still fail 

> to see why 64bit binaries are worth the huge extra effort right now. The only 
> exception is the LFS support, but that can be handled differently, see 
> Ethan's patch #675.


You are correct. 
But now Cygwin works both 32 bit and 64 bit. 
The 64 bit gnuplot binary works on Cygwin x86_64.
I think that the existence of 64 bit binary for 64 bit platform is natural.
Once 64 bit binaries can be build, it is not so difficult to work further.

BTW, patch shown yesterday worked. (Yesterday I have mis-operated)
I have posted to the patch ticket
https://sourceforge.net/p/gnuplot/patches/693/



Please apply gnuplot.iss for qt terminal to work.
 
Tatsuro