Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document CMD-object customization #28466

Closed
wants to merge 4 commits into from

Conversation

ChristianKurz
Copy link
Contributor

fix #19286

@kshyatt kshyatt added docs This change adds or pertains to documentation backport pending 1.0 labels Aug 13, 2018
@ChristianKurz
Copy link
Contributor Author

CI Failures are unrelated:

  • Circle i686 & x86_64: did not find test files for the following tests: OldPkg/pkg

@KristofferC
Copy link
Member

You need to rebase on master for Circle CI.

@KristofferC KristofferC mentioned this pull request Aug 19, 2018
@@ -365,3 +365,14 @@ stages have different latency so they use a different number of parallel workers
saturated throughput.

We strongly encourage you to try all these examples to see how they work.

## `CMD`-Objects
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, why all caps?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no reason, just a habbit. Will switch to lowercase and remove the hyphen while i'm at it.

Cmd(`pwd`, dir="..")
```

Also see [`setenv`](@ref), [`withenv`](@ref) for further information.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to be a bit more specific than "information"?

Also, how is withenv relevant here?

@StefanKarpinski StefanKarpinski added triage This should be discussed on a triage call backport 1.0 and removed triage This should be discussed on a triage call labels Jan 31, 2019
@KristofferC KristofferC removed backport 1.0 triage This should be discussed on a triage call labels Jan 31, 2019
@vtjnash
Copy link
Member

vtjnash commented Apr 20, 2021

Replaced by #40539. Sorry this sat for so long!

@vtjnash vtjnash closed this Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Manual: "Running External Programs" does not explain how to change working directory
5 participants