Conversation
The idiom `sed -n 's/.../.../p'` already correctly filters the lines. The same pattern can also be used in the sed expression for curl.
* refactor(function): use tail instead of sed "sed -e 1,2d" is interchangeable with "tail -n +3". This usage of "tail" is already used in many places in the codebase. The current sed expression was introduced as "1d" in commit ea10afb and modified to "1,2d" in commit 4351ae5. * refactor(_comp_compgen_p{ids,gids,names}): use tail instead of sed * refactor(2to3,mcrypt,msynctool,pdftotext): use tail instead of sed "sed -e 1d" used in other places can also be replaced with "tail -n +2".
scop
left a comment
There was a problem hiding this comment.
LGTM, thanks!
I have a nagging feeling that some POSIX features of basic tools cannot be taken for granted e.g. on Solaris, but I've no easy access to such a box any more, and at least the ones I tried work on FreeBSD, so let's do this.
|
Thank you.
|
|
I checked the behavior in Solaris 11.3. It turned out that Solaris I haven't checked the behavior of Solaris 11.4 because I don't have a virtual machine currently, but I found an article (written in Japanese 1) saying that Solaris 11.4 By the way, do we care about the support for Solaris 11.4? Solaris isn't mentioned in #1429 nor in Minimum bash versions - Wiki. Solaris 11.4 support seems to continue until 2037. If we care about it, we may need to continue to support Bash 4.4 until 2037. We probably want to intentionally ignore Solaris 11.4. Footnotes |
No description provided.