search

ocaml / opam

opam is a source-based package manager. It supports multiple simultaneous compiler installations, flexible package constraints, and a Git-friendly development workflow.
https://opam.ocaml.org
Other
1.21k stars 348 forks source link

Avoid head -c GNUism in reftests #5989

Closed madroach closed 4 days ago

madroach commented 1 month ago

head -c is not posix-compliant, use cut -b instead.

kit-ty-kate commented 1 month ago

the email you authored the commit with isn't recognised as part of your github profile. It's not a big deal but if you want you can amend your commit and/or add that email address to your github profile, and force-(with-lease-)push so it appears as yours

madroach commented 1 month ago

There is a missing one in list.unix.test

This one uses head -n 1, which is POSIX conforming. It's the -c 1 parameter, which is not POSIX conforming. This job is better left to cut.

kit-ty-kate commented 4 days ago

Thanks a lot!