这是indexloc提供的服务,不要输入任何密码
Skip to content

Bash shebang #34

@gafusss

Description

@gafusss

Hello!

I'm not sure whether this issue goes here or to termux-packages.

Some of the scripts here (e.g. termux-brightness) have a bash shebang.
Bash package is not marked as essential in termux-packages, neither it is located in the bootstrap. This package does not depend on the bash package either.

Running such scripts from dash (which is, IIRC, the default shell, since it is "essential") produces an error.

I see two possible solutions to this problem:

  • replace bash shebang with sh, modify affected scripts if needed (i.e. if they use "bash-only" features)

  • in termux-packages add bash as a dependency for this package (though that seems excessive) DONE

UPD: I was wrong and bash is actually included in the bootstrap, and is actually used as a login shell.
However, it is still possible to remove the bash package (which can be removed since termux-api does not depend on it), which makes some scripts (e.g. termux-dialog) unusable.

UPD2: termux-api now depends on bash, which is now also an essential package. Feel free to close this and the corresponding PR, if you feel that it is not necessary to remove the bash dependency

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