paq-nvim icon indicating copy to clipboard operation
paq-nvim copied to clipboard

log file (PaqLogOpen) interleaves logs from different jobs

Open justinmk opened this issue 3 years ago • 1 comments

Problem

from https://github.com/savq/paq-nvim/issues/7#issuecomment-780845979_

For now, the log is extremely simple. All stderr output will be redirected to the log. I wanted to add timestamps but the event loop prevents this. More often than not, the timestamps where written out-of-order, making them useless.

When I run :PaqLogOpen I see things like:

From https://github.com/nvim-treesitter/nvim-treesitter
   2f5be4585ef1..9ada5f70f98d  master     -> origin/master
vim-dadbod updating...
* presto adapter: use options, not paramsFrom https://github.com/lewis6991/gitsigns.nvim
   851cd32caaea..9110ea15a134  main       -> origin/main
nvim-treesitter updating...

The logs of different jobs are interleaved. I assume this is because paq doesn't collect the full job output before logging it.

Proposal

The logs would be more useful if Paq collected the full stdout/stderr of each job before logging it.

justinmk avatar Nov 07 '22 01:11 justinmk

#117 does this for the git log that's executed after a package is updated. But yes, collecting and then flushing sequentially should be the default for all jobs.

savq avatar Nov 07 '22 01:11 savq