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

No way to escape $ in os.ExpandEnv #43482

Closed
scudette opened this issue Jan 4, 2021 · 1 comment
Closed

No way to escape $ in os.ExpandEnv #43482

scudette opened this issue Jan 4, 2021 · 1 comment

Comments

@scudette
Copy link

scudette commented Jan 4, 2021

There does not seem to be a way to insert a literal $ into the expanded string. With other types of exapantions it is usual to be able to escape the $ with \ or maybe even $$ but none of these work

What version of Go are you using (go version)?

$ go version
go version go1.14.2 linux/amd64

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/mic/.cache/go-build"
GOENV="/home/mic/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/mic/go/"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/mic/build/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/mic/build/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/mic/projects/velociraptor/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build224101121=/tmp/go-build -gno-record-gcc-switches"

What did you do?

https://play.golang.org/p/Z2Z86w7ukwe

What did you expect to see?

I expected to see "hello $world" (i.e. the $ should be kept literally)

What did you see instead?

Hello \

@ianlancetaylor
Copy link
Contributor

What you say is correct, but the function acts as documented, and I don't think we can change it now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants