[Bug 829892] Review Request: python27 - Parallel-installable Python 2.7 for EPEL

[bugzilla at redhat.com]

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

Björn "besser82" Esser <bjoern.esser at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(mmckinst at umich.ed
                   |                            |u)



--- Comment #6 from Björn "besser82" Esser <bjoern.esser at gmail.com> ---
When manually reviewing the sources, I stumbled upon this somewhat wrong
looking macro-definition inside macros.python27:

  # This definition is identical to that one, except that in this one
  # brp-python-bytecompile is invoked with python2.6 rather than with no
  # arguments (which leads to it incorrectly using /usr/bin/python)
  %__python27_os_install_post    \
      /usr/lib/rpm/redhat/brp-compress \
      %{!?__debug_package:/usr/lib/rpm/redhat/brp-strip %{__strip}} \
      /usr/lib/rpm/redhat/brp-strip-static-archive %{__strip} \
      /usr/lib/rpm/redhat/brp-strip-comment-note %{__strip} %{__objdump} \
      /usr/lib/rpm/brp-python-bytecompile /usr/bin/python2.6 \
      /usr/lib/rpm/redhat/brp-java-repack-jars \
  %{nil}

Why this uses /usr/bin/python2.6 instead of %{__python27}?  Is this simply a
non-found left-over from an older version of this file?  To me it should use
%{__python27} for byte-compiling, since this is the only sensefull conclusion
when using python27-devel for building.

-- 
You are receiving this mail because:
You are on the CC list for the bug.