On Mon, Jun 30, 2014 at 03:40:45PM -0700, Thiago Macieira wrote: > On Monday 30 June 2014 14:34:58 Dirk Hohndel wrote: > > Minor nitpick > > > > The archdetect code just checks which architecture I build for, not which > > architecture it runs on. > > > > Since I build Windows binaries for i386 to reach more users, it always > > reports i386, even when run on a 64bit system. > > > > I think it would be more useful to detec the system architecture at run > > time... > > That was intentional and it was written for the Subsurface's use-case. If > you're upgrading a 32-bit Subsurface, you want to install a 32-bit update, > even if the system is 64-bit capable. > > I can add a currentCpuArchitecture() and rename this one to > buildCpuArchitecture().
I want both data points when collecting data for the survey. But what you are saying makes a lot of sense for the upgrade usecase. > As soon as I can find out how to determine that on Windows... I pushed code that does that. /D _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
