search

vincentdephily / emlop

EMerge LOg Parser
GNU General Public License v3.0
36 stars 4 forks source link

A `Stop` without a matching `Start` causes `emlop l` to show an extra merge with `-1` merge time. #10

Closed vincentdephily closed 1 year ago

vincentdephily commented 6 years ago

In other words, this:

1225702647: Started emerge on: Nov 03, 2008 08:57:27
1225702647:  *** emerge --tree --deep --ask --update --verbose world
1225702692:  >>> emerge (1 of 10) sys-apps/portage-2.2_rc13 to /
1225702699:  === (1 of 10) Cleaning (sys-apps/portage-2.2_rc13::/usr/portage/sys-apps/portage/portage-2.2_rc13.ebuild)
1225702699:  === (1 of 10) Compiling/Merging (sys-apps/portage-2.2_rc13::/usr/portage/sys-apps/portage/portage-2.2_rc13.ebuild)
1225702705:  === (1 of 10) Merging (sys-apps/portage-2.2_rc13::/usr/portage/sys-apps/portage/portage-2.2_rc13.ebuild)
1225702719:  === (1 of 10) Post-Build Cleaning (sys-apps/portage-2.2_rc13::/usr/portage/sys-apps/portage/portage-2.2_rc13.ebuild)
1225702719:  ::: completed emerge (1 of 10) sys-apps/portage-2.2_rc13 to /
1225702719:  ::: completed emerge (1 of 10) sys-apps/portage-2.2_rc13 to /
1225702719:  *** RESTARTING emerge via exec() after change of portage version.
1225702719:  *** terminating.
1225702720: Started emerge on: Nov 03, 2008 08:58:40
1225702720:  *** emerge --tree --update --resume --verbose --ignore-default-opts --deep
1225702722:  *** exiting unsuccessfully with status '1'.
1225702725:  *** terminating.

causes this:

2008-11-03 08:58:39 +00:00        27 sys-apps/portage-2.2_rc13
2008-11-03 08:58:39 +00:00        -1 sys-apps/portage-2.2_rc13
vincentdephily commented 1 year ago

This has long been somewhat fixed by displaying "?" for unknown durations. There doesn't seem to be anything better to do here: a completed merge is still useful info even if we don't know when it started.

FWIW, genlop displays two entries with the same build time (definitely a bug), and qlop displays only one entry (makes some sense, but I prefer emlop's behaviour).