Skip to content

Commit

Permalink
Exclude python scripts from RPM shebang check
Browse files Browse the repository at this point in the history
The newest Fedora packaging rules print warnings for scripts using the
/usr/bin/python shebang:

    *** WARNING: mangling shebang in /usr/bin/arc_summary.py from
    #!/usr/bin/python to #!/usr/bin/python2. This will become an ERROR,
    fix it manually!

Fedora wants all cross compatible scripts to pick python3.  Since we
don't want our users to have to pick a specific version of python, we
exclude our scripts from the RPM build check.

Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Giuseppe Di Natale <[email protected]>
Signed-off-by: Tony Hutter <[email protected]>
Closes #7360 
Closes #7399
  • Loading branch information
tonyhutter authored and behlendorf committed Apr 6, 2018
1 parent 55d80e6 commit 812323b
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions rpm/generic/zfs.spec.in
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,16 @@
%bcond_with asan
%bcond_with systemd

# Python permits the !/usr/bin/python shebang for scripts that are cross
# compatible between python2 and python3, but Fedora 28 does not. Fedora
# wants us to choose python3 for cross-compatible scripts. Since we want
# to support python2 and python3 users, exclude our scripts from Fedora 28's
# RPM build check, so that we don't get a bunch of build warnings.
#
# Details: https://github.com/zfsonlinux/zfs/issues/7360
#
%global __brp_mangle_shebangs_exclude_from arc_summary.py|arcstat.py|dbufstat.py|test-runner.py

# Generic enable switch for systemd
%if %{with systemd}
%define _systemd 1
Expand Down

0 comments on commit 812323b

Please sign in to comment.