On Mon, Oct 31, 2016 at 12:05 PM, Pierre-Yves Chibon <pingou@pingoured.fr> wrote:
On Mon, Oct 31, 2016 at 11:59:47AM +0100, Michal Novotny wrote:
>    On Mon, Oct 31, 2016 at 8:40 AM, Pierre-Yves Chibon <pingou@pingoured.fr>
>    wrote:
>
>      We used to send more info but had to trim it down due to some massive
>      commits
>      that were basically breaking datagrepper (OutOfMemory), but if there is
>      an
>      use-case I'm fine with expending the amount of information sent.
>      Worst case, do the computation client-side, listen to fedmsg, get the
>      start and
>      stop commits and see which files were touched in between.
>
>    In the simplest case, we would need directory/file names of level 1 where
>    there was
>    a change. That's perhaps a little too specific though. I would still very
>    much welcome
>    this or overall list of modified paths. Not saying, I can't put together
>    some really messy
>    code to get that information from the patches :).

If you're not against shelling out commands, you could try:

Get the list of commits:
  git rev-list old_commit..new_commit

Get the list of file changed in a specific commit:
  git diff-tree --no-commit-id --name-only -r <hash of the commit>

The problem is that we don't have the repository cloned at the point the fedmsg is received.
And to always clone the repo to run the commands would be a bit troublesome.
 

Pierre
_______________________________________________
copr-devel mailing list -- copr-devel@lists.fedorahosted.org
To unsubscribe send an email to copr-devel-leave@lists.fedorahosted.org