Skip to content

clipdel: add -q to suppress output#260

Closed
temporecras wants to merge 1 commit intocdown:developfrom
temporecras:quiet-delete
Closed

clipdel: add -q to suppress output#260
temporecras wants to merge 1 commit intocdown:developfrom
temporecras:quiet-delete

Conversation

@temporecras
Copy link
Copy Markdown

While working on an script involving clipmenu I ran into it printing all the pastes to be deleted to stdout. This was a major inconvenience as I was trying to delete sensitive information following a list of regexs.

This PR adds a -q (quiet) flag to allow the output to be suppressed natively without the need for workarounds (as redirecting output to /dev/null).

Changes:
added -q flag to clipdel.c
updated clipdel.1 man page
Added integration tests in tests/x_integration_tests to verify the quiet behavior

@N-R-K
Copy link
Copy Markdown
Contributor

N-R-K commented Feb 12, 2026

as redirecting output to /dev/null

This seems like a valid and universal solution to me. What's the problem with this?

@temporecras
Copy link
Copy Markdown
Author

Redirection works, and that's a fair point, but native support for quietness is common in similar utilities for better ergonomics. It keeps scripts cleaner and I believe it's a low-maintenance addition that makes the tool more intuitive.

@temporecras
Copy link
Copy Markdown
Author

Hi! Just checking on this PR as it's been a while. Happy to make any change if needed.

@cdown
Copy link
Copy Markdown
Owner

cdown commented Apr 16, 2026

Apologies for not getting back sooner.

My opinion is basically the same as @N-R-K I guess. I don't feel strongly about it either way, but I also think there's the question to ask of:

  1. If this is added to clipdelmenu, are there other tools it may need adding to now or in the future for consistency?
  2. If so, how invasive will that be?

This is a pretty weak argument since I doubt clipmenu as a suite of tools will grow that much, or the functionality either, but using >/dev/null avoids having to worry about it in the first place.

So I guess I weakly prefer to leave it as is.

@temporecras
Copy link
Copy Markdown
Author

I get your point and understand it makes sense to leave it as is. Thanks for your feedback!

@temporecras temporecras deleted the quiet-delete branch April 19, 2026 10:28
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.

3 participants