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

checkbashism: 2.0.0.2 -> 2.18.1 #38102

Merged
merged 1 commit into from Apr 1, 2018

Conversation

ryantm
Copy link
Member

@ryantm ryantm commented Mar 30, 2018

Semi-automatic update generated by https://github.com/ryantm/nix-update tools.

This update was made based on information from https://repology.org/metapackage/checkbashisms/versions.

These checks were done:

Semi-automatic update generated by https://github.com/ryantm/nix-update tools.

This update was made based on information from https://repology.org/metapackage/checkbashisms/versions.

These checks were done:

- built on NixOS
- Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
- directory tree listing: https://gist.github.com/6e3ce06d60c67525f1f11df0822e6b36
@matthewbauer matthewbauer merged commit 651f6bf into NixOS:master Apr 1, 2018
@ryantm ryantm deleted the auto-update/checkbashisms branch April 1, 2018 03:56
@dotlambda
Copy link
Member

I'm getting

bash: ./result/bin/checkbashisms: cannot execute binary file: Exec format error

This is because the url now points to a tarball which needs to be unpacked.

Should we revert this change?

@matthewbauer
Copy link
Member

Yep that is probably best.

@dotlambda
Copy link
Member

Reverted in 18e4125.

dotlambda pushed a commit that referenced this pull request Apr 26, 2018
Synthetica9 pushed a commit to Synthetica9/nixpkgs that referenced this pull request May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants