[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=567348

Toshio Ernie Kuratomi <a.badger@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |a.badger@xxxxxxxxx

--- Comment #21 from Toshio Ernie Kuratomi <a.badger@xxxxxxxxx> 2010-09-16 00:03:52 EDT ---
I'm sorry for not being clear.  Your solution is more complex than it needs to
be.  The following three lines will work:

# get python3 version
%global py3_ver 0
%{?__python3: %global py3_ver %(%{__python3} -c "import sys;
sys.stdout.write(sys.version[:3])")}

When creating the srpm that's fed to the buildsystem, the spec file needs to be
parsable but it doesn't need to have the correct values.  So we need py3_ver to
have a number (in this case 0) so that when it's used in the Requires line
later the spec file will parse correctly.

That srpm is fed to the buildsystem which then creates the binary rpms and a
new srpm.  These files are created in a buildroot that has python3-devel
installed and thus py3_ver is set to the proper python3 version when those are
built.

(the other part of your patch to byte compile the things in %{python3_sitelib}
is still necessary.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]