Skip to content

fix: Add pipe-entry-to to pipe all lines of the current log message (#1691)#1692

Closed
moaedy wants to merge 3 commits into
tstack:masterfrom
moaedy:godfix/issue-1691-run-15
Closed

fix: Add pipe-entry-to to pipe all lines of the current log message (#1691)#1692
moaedy wants to merge 3 commits into
tstack:masterfrom
moaedy:godfix/issue-1691-run-15

Conversation

@moaedy
Copy link
Copy Markdown

@moaedy moaedy commented May 12, 2026

Summary

Root cause: lnav had :pipe-to for marked lines and :pipe-line-to for the focused line, but no explicit command to pipe all lines of a multi-line log message. Change: Added a new :pipe-entry-to command in com_pipe_to (src/cmds.io.cc) that pipes the full multi-line log message via read_full_message, registered in IO_COMMANDS with its help text, listed in LNAVSECURE docs and NEWS.md. Closes #1691

@moaedy
Copy link
Copy Markdown
Author

moaedy commented May 12, 2026

Addressed feedback in eeb01f9: 1 failed check(s): main.

The failure is environmental and affects the entire repo, not just this PR. Even lnav's own master branch builds fail with the same error. The cause:

@moaedy
Copy link
Copy Markdown
Author

moaedy commented May 12, 2026

Addressed feedback in 5029b8a: 1 failed check(s): main. FOLLOW-UP SUMMARY:
Updated .cirrus.yml:2 to use freebsd-14-2 instead of the EOL freebsd-13-2 image family. As called out in the PR comment, FreeBSD 13.2 was removed from Cirrus's GCE project after reaching end-of-life, which caused the main CI task to fail across the entire repo (master and all open PRs). Switching to a currently-supported FreeBSD 14.x image family unblocks the Cirrus buil

@tstack
Copy link
Copy Markdown
Owner

tstack commented May 12, 2026

Can the human behind this bot identify themselves?

@moaedy moaedy closed this May 12, 2026
@moaedy moaedy deleted the godfix/issue-1691-run-15 branch May 12, 2026 12:36
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.

Add pipe-entry-to to pipe all lines of the current log message

2 participants