Skip to content

UPSTREAM PR #17480: Fix verbose flag output for mkdir and mv commands#42

Open
loci-dev wants to merge 1 commit intomainfrom
loci/pr-17480-fix-verbose-flag-output
Open

UPSTREAM PR #17480: Fix verbose flag output for mkdir and mv commands#42
loci-dev wants to merge 1 commit intomainfrom
loci/pr-17480-fix-verbose-flag-output

Conversation

@loci-dev
Copy link

@loci-dev loci-dev commented Feb 3, 2026

Note

Source pull request: nushell/nushell#17480

Description

Fixes #16926

The verbose flags (-v) for mkdir and mv commands produced confusing output:

  • mkdir -v showed only the directory name instead of a creation message
  • mv -v showed cryptic mv-verbose-renamed instead of meaningful output

Changes

mkdir -v

  • Single directory: created directory <path>
  • Multiple directories: created directories: <path1>, <path2>, ...

mv -v

  • Single file: moved <source> -> <target>
  • Multiple files: moved N files to <target>

Also disabled verbose output from underlying uutils libraries to avoid duplication.

Before

> mkdir -v test-dir
╭───┬──────────╮
 0  test-dir 
╰───┴──────────╯

> mv -v file.txt dest/
╭───┬───────────────────╮
 0  mv-verbose-renamed 
╰───┴───────────────────╯

After

> mkdir -v test-dir
created directory test-dir

> mv -v file.txt dest/
moved file.txt -> dest/file.txt

User-Facing Changes

Verbose output for mkdir and mv now shows clear, informative messages that follow standard Unix conventions.

Tests + Formatting

  • Verified the fix works with both single and multiple file operations
  • Ran cargo fmt and cargo clippy

Fixes issue #16926 where verbose flags had confusing output:
- mkdir -v showed only directory name instead of full creation message
- mv -v showed 'mv-verbose-renamed' instead of meaningful output

Changes:
- mkdir -v now shows 'created directory <path>' for single directories
- mkdir -v now shows 'created directories: <paths>' for multiple directories
- mv -v now shows 'moved <source> -> <target>' for single file moves
- mv -v now shows 'moved N files to <target>' for multiple file moves
- Disabled verbose output from underlying uutils libraries to avoid duplication
- Provided clear, consistent verbose messages that match user expectations

The verbose output is now informative and follows standard Unix conventions.
@loci-dev loci-dev force-pushed the main branch 7 times, most recently from 64bcc48 to d0a8c7a Compare February 17, 2026 03:26
@loci-dev loci-dev force-pushed the main branch 2 times, most recently from 6102c29 to d8ed90c Compare February 19, 2026 03:27
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

Comments