From kde-buildsystem Thu Oct 04 19:05:38 2012 From: "Alexander Neundorf" Date: Thu, 04 Oct 2012 19:05:38 +0000 To: kde-buildsystem Subject: Re: Review Request: Support Python 3.2's new method of handling byte compiled files in PythonMacros. Message-Id: <20121004190538.6387.24156 () vidsolbach ! de> X-MARC-Message: https://marc.info/?l=kde-buildsystem&m=134937755922830 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============3383723536770900928==" --===============3383723536770900928== Content-Type: multipart/alternative; boundary="===============6848031535835227567==" --===============6848031535835227567== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106707/#review19923 ----------------------------------------------------------- Ship it! Looks straightforward and obvious. Beside that, I don't know much about python byte code compiling. - Alexander Neundorf On Oct. 3, 2012, 10:43 a.m., Luca Beltrame wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106707/ > ----------------------------------------------------------- > = > (Updated Oct. 3, 2012, 10:43 a.m.) > = > = > Review request for Build System. > = > = > Description > ------- > = > Python 3.2 changed the compile mechanism by creating a __pycache__ direct= ory. This obviously breaks a lot of stuff that assumed that pyc files were = stored in the same directory as the py files. This change adds a check, and= adjusts files only in case Python 3.2 is found. Notice that currently I ha= ve way of testing if the suffix (*cpython) is the same on all supported OSe= s. > = > = > Diffs > ----- > = > cmake/modules/PythonMacros.cmake 774c6b2d8eb5f9bafaf7e73da1108bd347a736= 67 = > = > Diff: http://git.reviewboard.kde.org/r/106707/diff/ > = > = > Testing > ------- > = > No more errors are produced when building PyKDE4 with Python 3.2.3 > = > = > Thanks, > = > Luca Beltrame > = > --===============6848031535835227567== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/106707/

Ship it!

Looks stra=
ightforward and obvious.
Beside that, I don't know much about python byte code compiling.

- Alexander


On October 3rd, 2012, 10:43 a.m., Luca Beltrame wrote:

Review request for Build System.
By Luca Beltrame.

Updated Oct. 3, 2012, 10:43 a.m.

Descripti= on 

Python 3.2 changed the compile mechanism by creating a __pyc=
ache__ directory. This obviously breaks a lot of stuff that assumed that py=
c files were stored in the same directory as the py files. This change adds=
 a check, and adjusts files only in case Python 3.2 is found. Notice that c=
urrently I have way of testing if the suffix (*cpython) is the same on all =
supported OSes.

Testing <= /h1> 
No more errors are produced when building PyKDE4 with Python=
 3.2.3

Diffs=

  • cmake/modules/PythonMacros.cmake (774c6b2d= 8eb5f9bafaf7e73da1108bd347a73667)

View Diff
--===============6848031535835227567==-- --===============3383723536770900928== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Kde-buildsystem mailing list Kde-buildsystem@kde.org https://mail.kde.org/mailman/listinfo/kde-buildsystem --===============3383723536770900928==--