Skip to content
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.

Avoid undocumented usage of the head utility. #163

Closed
wants to merge 1 commit into from

Conversation

rojkov
Copy link

@rojkov rojkov commented May 2, 2016

Some implementations of head (e.g. the one in Busybox)
can't handle options like "-1". Replace them with "-n1".

Signed-off-by: Dmitry Rozhkov dmitry.rozhkov@linux.intel.com

Some implementations of head (e.g. the one in Busybox)
can't handle options like "-1". Replace them with "-n1".

Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
@ztombol
Copy link

ztombol commented May 7, 2016

This patch fixes all uses of head. Haven't tested it, but the new usage conforms to POSIX and works with busybox according to its documentation.

@rojkov
Copy link
Author

rojkov commented Mar 4, 2020

closing as it got merged in bats-core/bats-core#17

@rojkov rojkov closed this Mar 4, 2020
@rojkov rojkov deleted the fix-head-util branch March 4, 2020 12:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants