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

Consider changing "run" to use temporary script files #1

Open
solvingj opened this issue Mar 8, 2019 · 0 comments
Open

Consider changing "run" to use temporary script files #1

solvingj opened this issue Mar 8, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@solvingj
Copy link
Owner

solvingj commented Mar 8, 2019

I noticed this in azure pipelines log:

##[command]"C:\windows\system32\cmd.exe" /D /E:ON /V:OFF /S /C "CALL "D:\a\_temp\e89baa3f-7c45-4af5-90e8-16c62e8dbc25.cmd""

and this on linux

[command]/bin/bash --noprofile --norc /home/vsts/work/_temp/74d504a3-043d-4534-a3f4-cb1c9041ed03.sh

I've seen the pattern in jenkins logs and other logs as well. I'm sure that we'll hit situations where people want to run command-lines requiring quotes, and then escaping various characters will start to become a real nuisance. I think this technique is probably the most reliable approach.

@solvingj solvingj added the enhancement New feature or request label Mar 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant