Bug 71911 - gerrit daily digest could show the top level directories of a new change
Summary: gerrit daily digest could show the top level directories of a new change
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: WWW (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other All
: medium normal
Assignee: Mat M
URL:
Whiteboard:
Keywords: difficultyBeginner, easyHack, skillScript
Depends on:
Blocks:
 
Reported: 2013-11-22 09:56 UTC by Björn Michaelsen
Modified: 2016-02-18 16:37 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Björn Michaelsen 2013-11-22 09:56:10 UTC
Since fdo#70682 (Thanks MatM!) was fixed, we now have a full copy of the repo available for investigation on digest creation:

We could checkout the change and do a "git diff --stat" or somesuch to extract the top level directories a change touches and report that for new changes.

This would make it easier for reviewers to quickly scan for changes that are relevant for them.
Comment 1 Mat M 2013-11-22 23:55:53 UTC
FYI, I'll move ssh to subprocess in the same patch

And we do not have a full copy, we have a bare repository !
Comment 2 Mat M 2013-11-23 13:30:19 UTC
Current layout is:

+ more debug logs, extra debug layer, file is not used in p3k
   in https://gerrit.libreoffice.org/5267 from James Michael Dupont


Proposal is to add a 3rd line if we want to stay below the 73 chars limit
+ more debug logs, extra debug layer, file is not used in p3k
   in https://gerrit.libreoffice.org/5267 from James Michael Dupont
   about module sw

or 
+ fdo#54514 - PRINT PREVIEW: Old icon for page left - fixed
   in https://gerrit.libreoffice.org/6583 from Rolf Koetter
   about module icon-themes, solenv

Basically, we could just add the folders like
   in https://gerrit.libreoffice.org/5267 (sw) from James Michael Dupont
or
   in https://gerrit.libreoffice.org/5267 from James Michael Dupont (sw) 

but that only works for mono-modules changes. Or we accept to be longer.
Comment 3 Björn Michaelsen 2013-11-23 13:48:05 UTC
(In reply to comment #2)
> + fdo#54514 - PRINT PREVIEW: Old icon for page left - fixed
>    in https://gerrit.libreoffice.org/6583 from Rolf Koetter
>    about module icon-themes, solenv

This looks good to me.
Comment 4 Robinson Tryon (qubit) 2015-12-15 16:24:56 UTC
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyBeginner SkillScript )
[NinjaEdit]
Comment 5 Robinson Tryon (qubit) 2016-02-18 16:37:21 UTC
Remove LibreOffice Dev List from CC on EasyHacks
(curtailing excessive email to list)
[NinjaEdit]