Bug 84227 - dev-tools gerritbot/send-daily-digest: in "about" only include modules from latest patchset
Summary: dev-tools gerritbot/send-daily-digest: in "about" only include modules from l...
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: ci-infra (show other bugs)
(earliest affected)
Hardware: All All
: medium normal
Assignee: Mat M
Depends on:
Reported: 2014-09-23 08:00 UTC by Stephan Bergmann
Modified: 2015-02-19 09:45 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Bergmann 2014-09-23 08:00:27 UTC
<http://lists.freedesktop.org/archives/libreoffice/2014-September/063540.html> "LibreOffice Gerrit News for core on 2014-09-23" contains

> + fdo#81356: compiler plugin for rewrite Fraction
>   in https://gerrit.libreoffice.org/11405 from Juan Picca
>   about module build, compilerplugins, include, testcompilerplugins, tools

even though <https://gerrit.libreoffice.org/#/c/11405/8> for quite a while now is only about module compilerplugins.  That confused me, as I thought the gerrit patch mentioned here is an alternative duplicate variant of the patch I am familiar with (and know only talks about compilerplugins these days), and should be abandoned, while in fact it was the very same patch.

This could probably be fixed in get_digest (gerritbot/send-daily-digest) by changing

> fpaths = (set([fileobj['file'] for pset in change['patchSets']
>                 for fileobj in pset['files']]))

to only iterate over the latest patch set, however that would be done.