Skip to content

Add support for quoted args#214

Open
vagos wants to merge 2 commits into
mainfrom
quotedargs
Open

Add support for quoted args#214
vagos wants to merge 2 commits into
mainfrom
quotedargs

Conversation

@vagos

@vagos vagos commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

This has worked in my limited experimentation.
Solution might not be perfect, but might be a good starting point for someone to finish it off.

vagos added 2 commits December 2, 2025 11:38
Signed-off-by: Evangelos Lamprou <vagos@lamprou.xyz>
Signed-off-by: Evangelos Lamprou <vagos@lamprou.xyz>
@mgree

mgree commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

This is something that is only hard because of the shell's craziness. And it's really, really hard to get right.

I don't think it's possible portably with this approach, though. If we want to try to do this, the best thing is to write each arg to its own file and generate appropriate $(cat argN) invocations... but even this will combine multiple trailing newlines.

Figured out the right approach in the car. We should thread args into both scripts using "$@". Is it okay if I push to this branch, @vagos?

@vagos

vagos commented Dec 29, 2025

Copy link
Copy Markdown
Contributor Author

@mgree Yeah, feel free to!

@angelhof

angelhof commented Jan 6, 2026

Copy link
Copy Markdown
Member

Figured out the right approach in the car.

🚗 🤔 💭 💡

@ezrizhu

ezrizhu commented Jan 6, 2026

Copy link
Copy Markdown
Member

drive safe!! blinking lights appearing from your head may be distracting to other drivers.

@ezrizhu

ezrizhu commented Jan 6, 2026

Copy link
Copy Markdown
Member

Should we re-open #155 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants