Skip to content

Conversation

@tstack
Copy link
Contributor

@tstack tstack commented Sep 10, 2025

Mostly need to make some more fields public so
lnav can read them.

Also, restore checking if the file name mentioned
in the log is in the discovered source path. The
check is pretty loose since I think I've seen
file names truncated before.

Mostly need to make some more fields public so
lnav can read them.

Also, restore checking if the file name mentioned
in the log is in the discovered source path.  The
check is pretty loose since I think I've seen
file names truncated before.
@tstack
Copy link
Contributor Author

tstack commented Sep 10, 2025

I've done enough wiring to where I can add the source path and then do a SELECT on the all_logs virtual SQLite table:

lnav -c ':add-source-path ../lnav/src' -c ';SELECT log_line,log_msg_format,log_msg_values,log_msg_src FROM all_logs LIMIT 10'

The result looks about right:

image

Seeing the variable names in there is pretty neat! Nice work!

(The green bar is because of the line number, need to see about turning that off...)

@ttiimm
Copy link
Owner

ttiimm commented Sep 10, 2025

Nice to see it working end to end!

@ttiimm ttiimm merged commit cd423a2 into ttiimm:main Sep 10, 2025
7 checks passed
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