#882 Python 2 SHOULD be conditionalized via %if %{with python2} if applicable
Merged by decathorpe. Opened by churchyard.
churchyard/packaging-committee bcond_python2  into  master

Download 882.patch

Voted in: https://pagure.io/packaging-committee/issue/876#comment-565769

I'd update this comment from # or without to # to disable python2 by default, or and add a second line with %bcond_without python2 # to enable python2 by default.
I can't remember which switch does what if my life depended on it, and I think other people might appreciate the clarification as well.

I'd put this switch at the top of the .spec file. Most packages I've had to deal with put those switches at the top, where they are easily found.

Otherwise, looks good to me!

rebased onto 4cda13469dfcc131943227d5ab9d3292062f2dd8

amended.

Looks good, thank you!

Pull-Request has been merged by decathorpe

Metadata