Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove shebang from nonexecutable script #192

Commits on Apr 9, 2020

  1. Remove shebang from nonexecutable script

    When packaging chardet and pip (which bundles it) in Fedora, we have realized
    that there is a nonexecuatble file with a shebang line.
    
    It seems that the primary purpose of this file is to be imported from Python
    code or to be executed via python chardet/cli/chardetect.py or
    python -m chardet.cli.chardetect and hence the shebang appears to be unnecessary.
    
    Shebangs are hard to handle when doing downstream packaging, because it makes
    sense for upstream to use #!/usr/bin/env python while in the RPM package, we
    need to avoid that and use a more specific interpreter. Since the shebang was
    unused, I propose to remove it to avoid the problems.
    hrnciar committed Apr 9, 2020
    Copy the full SHA
    db619e2 View commit details
    Browse the repository at this point in the history