Skip to content

RFE: check scripts with shebangs are executable #543

Closed

Description

@scop

A check that does the opposite of check-executables-have-shebangs would be useful; one that verifies scripts with shebangs are executable. Maybe call it just check-executability or something.

Do you think that would be welcome here? If yes, I think I could have a go at writing one. Let me also know if you have a name you'd prefer for this.

Activity

  1. asottile commented on Jan 8, 2021

    @asottile
    Member

    it wasn't added with check-executables-have-shebangs because there are a lot of special cases and I don't really want to encode them into a script (since it'd be very noisy). A number of things (mis)use first-line comments starting with #! to mean things that are different than shebangs unfortunately

    it's still possible to write something like this, and it may still be useful for some, but that's why I've avoided incorporating it here so far

  2. scop commented on Jan 8, 2021

    @scop
    ContributorAuthor

    Yeah, I'd keep the test simple and leave worrying about abuse of shebangs to those who do that, they can choose to use exclude the way they want, or just not use the check in the first place.

  3. asottile commented on Jan 8, 2021

    @asottile
    Member

    would you like to work on this?

  4. scop commented on Jan 8, 2021

    @scop
    ContributorAuthor

    Sure, can do, just discussing desirability here first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions