Review Request: Respect PYTHONDONTWRITEBYTECODE env variable

[Luca Beltrame]

> On Nov. 6, 2012, 10:49 p.m., Luca Beltrame wrote:
> > Good catch, I missed that. Push to 4.9: as I'm not sure it would merge cleanly as I changed a bit of things in 4.10, CC me in the commit and I'll see about trying to adapt it.

Bear in mind that if you use this to avoid writing bytecode for Python 3.2+ (due to the change in the compile mechanism), 4.10 will offer support for installing __pycache__ directories in the build system.


- Luca


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107228/#review21511
-----------------------------------------------------------


On Nov. 6, 2012, 10:50 a.m., Andrea Scarpino wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107228/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2012, 10:50 a.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> cmake/modules/PythonMacros.cmake defines PYTHON_INSTALL macro. This macro should not create/install byte-compiled Python modules (*.pyc) when PYTHONDONTWRITEBYTECODE environmental variable is set.
> 
> See also: http://docs.python.org/using/cmdline.html#envvar-PYTHONDONTWRITEBYTECODE
> 
> patch by Arfrever Frehtes Taifersar Arahesis, I just updated it to be applied to master/KDE4.9
> 
> 
> This addresses bug 276151.
>     http://bugs.kde.org/show_bug.cgi?id=276151
> 
> 
> Diffs
> -----
> 
>   cmake/modules/PythonMacros.cmake 661e32d 
> 
> Diff: http://git.reviewboard.kde.org/r/107228/diff/
> 
> 
> Testing
> -------
> 
> Works. kdelibs Arch Linux package uses it since 4.8.4
> 
> 
> Thanks,
> 
> Andrea Scarpino
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121106/580448c8/attachment.htm>