Skip to content

fixes #411 [Lidarr] Fix BeetsTagger broken pipe: find | read → find | grep -q .#412

Open
barrettclark wants to merge 1 commit intoRandomNinjaAtk:mainfrom
barrettclark:fix/beetstaggler-find-broken-pipe
Open

fixes #411 [Lidarr] Fix BeetsTagger broken pipe: find | read → find | grep -q .#412
barrettclark wants to merge 1 commit intoRandomNinjaAtk:mainfrom
barrettclark:fix/beetstaggler-find-broken-pipe

Conversation

@barrettclark
Copy link
Copy Markdown

Summary

  • Fixes find "$1" -type f -iname "*.flac" | read in BeetsTagger.bash line 49
  • read closes the pipe after the first line, sending SIGPIPE to find and producing log errors on every BeetsTagger run
  • Replaced with grep -q . which exits 0 if any output exists without closing the pipe early

Test plan

  • BeetsTagger runs without find: 'standard output': Broken pipe / find: write error log errors
  • Albums with FLAC files are still processed normally
  • Albums without FLAC files still log the "Only supports flac files" error and return

🤖 Generated with Claude Code

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.

1 participant