Skip to content

nom-cp: create remote directories before copying files#12

Open
DuncanDHall wants to merge 3 commits intointernetarchive:mainfrom
DuncanDHall:nom-cp-mkdir
Open

nom-cp: create remote directories before copying files#12
DuncanDHall wants to merge 3 commits intointernetarchive:mainfrom
DuncanDHall:nom-cp-mkdir

Conversation

@DuncanDHall
Copy link
Copy Markdown
Contributor

Ensures destination directories exist before writing files, for both the rsync (VSCode sync-rsync) and nomad alloc exec code paths.

Ensures destination directories exist before writing files, for both
the rsync (VSCode sync-rsync) and nomad alloc exec code paths.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@traceypooh
Copy link
Copy Markdown
Contributor

i like it, TY!

just one question.. did this happen to get vibe-coded or part of it? that's fine -- it's just that it's using some shell constructs i haven't seen before...

@DuncanDHall
Copy link
Copy Markdown
Contributor Author

DuncanDHall commented Feb 18, 2026

It is–and I learned some new patterns here too when asking Claude about it afterward! ${@: -1} to get last parameter, pattern matching conditional like [[ "$RSYNC_DEST" == *:* ]], and the %% and # POSIX Parameter Expansion operators.

I'm not too well versed in shell scripting, so I did lean heavily on the agent for this and when I asked it to use "more well known patterns" it spat out basically the same thing... I've added some comments to try to make it easier to parse!

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.

2 participants