search

mvdan / sh

A shell parser, formatter, and interpreter with bash support; includes shfmt
https://pkg.go.dev/mvdan.cc/sh/v3
BSD 3-Clause "New" or "Revised" License
7.21k stars 339 forks source link

interp: set EUID #958

Closed anderbubble closed 1 year ago

anderbubble commented 1 year ago

Set EUID variable analog to UID.

Signed-off-by: Jonathon Anderson janderson@ciq.co

anderbubble commented 1 year ago

This is meant to be a better fix for https://github.com/apptainer/apptainer/issues/837 than is possible downstream. (e.g., https://github.com/apptainer/apptainer/pull/937)

anderbubble commented 1 year ago

Absolutely. I was just too far down my downstream thought process, where we had been planning to set EUID=$UID in a script. Thanks for pointing this out.

PR updated.

anderbubble commented 1 year ago

@mvdan how about these?

anderbubble commented 1 year ago

Thanks for the merge!

Do you have an estimate for when this might be included in a release?

mvdan commented 1 year ago

I'll probably do a bugfix release in the coming weeks. I just did a minor (feature) release, so that likely won't happen again for a few months. If you need this sooner, you should be able to use master.

DrDaveD commented 1 year ago

So, would you include this in a bugfix release?

mvdan commented 1 year ago

Sure, why not. It doesn't feel too risky. Though, again, you should be able to use master easily.