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.
# or without
# to disable python2 by default, or
%bcond_without python2 # to enable python2 by default
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
Voted in: https://pagure.io/packaging-committee/issue/876#comment-565769