Kilian / f.lux-indicator-applet

This repository is unmaintained. Visit:
https://github.com/xflux-gui/xflux-gui
Other
549 stars 105 forks source link

Added 32-bit xflux executable in setup.py and fix some bugs #67

Closed aaferrari closed 8 years ago

aaferrari commented 8 years ago

For a detailed explanation of the changes that I made in setup.py, xfluxcontroller.py and fluxapp.py see here and here respectively. I do not know if setup() can do the same as the if that I added to beginning of setup.py to install the xflux executable corresponding to the processor architecture (if so feel free to make the appropriate change). I could also have done the same using platform.architecture() instead of sys.maxsize but chose the latter method due to the next thing that I quote from here:

Note: On Mac OS X (and perhaps other platforms), executable files may be universal files containing multiple architectures. To get at the “64-bitness” of the current interpreter, it is more reliable to query the sys.maxsize attribute: is_64bits = sys.maxsize > 2**32

Kilian commented 8 years ago

Thanks!