Repository navigation
Use more descriptive long-form options for curl, tar, and checksum validation - #6496
Conversation
mthalman
left a comment
There was a problem hiding this comment.
This is limited to the three commands you described. Are there plans to expand this everywhere? At a glance, I can see ln and mkdir as candidates. I guess what's the overall plan here?
|
Here's a HN post that describes much the same thing: https://news.ycombinator.com/item?id=43440184. It's useful context for folks (including us, later) who want to understand the motivation. |
|
Yes, I acknowledge that I left out |
mthalman
left a comment
There was a problem hiding this comment.
Other commands I see that can use long form are rm -rf and useradd -l.
- tdnf install - useradd - apt-get install - rm
|
OK, here's what I did:
|
mthalman
left a comment
There was a problem hiding this comment.
This strikes a good balance of short and long form, I think.
|
(Markdown link check failure is a false positive) |
Reading through our Dockerfiles can be a little confusing if you're not familiar with specific command line option abbreviations for
curl,tar, etc.This PR attempts to make them easier to read by using the long-form options:
curl -fSLO->curl --fail --silent --location --outputtar -oxzf->tar --gzip --extract --no-same-owner --filesha512sum -c ...->sha512sum --check ...