Print

Print

command -v is a POSIX safe way to find an executable program. While which is very common, it still requires installation on many Linux operating systems, and so command -v can offer a more portable alternative with a very low amount of additional cognitive overhead on the maintainers.

Additionally, use a subshell, $(), over backticks for command substitution as $() is recommended for multiple reasons including readability.

c.f.:

You can view, comment on, or merge this pull request online at:

  https://github.com/xrootd/xrootd/pull/1658

Commit Summary

File Changes

(4 files)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: <xrootd/xrootd/pull/1658@github.com>

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1658", "url": "https://github.com/xrootd/xrootd/pull/1658", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1