Re: LibreOffice Gerrit News for core on 2013-12-09

Le Mon, 09 Dec 2013 15:23:06 +0100, Stephan Bergmann <sbergman@redhat.com> 
a écrit:

> On 12/09/2013 02:32 PM, bjoern wrote:
> > Of course, patches are welcome! ;)
>
> No, they are apparently not, as long as they linger in git and don't get 
> put into production.  ;)
Well, at least now, they got pushed in an email to the list :-P

BTW, I sent an off-list mail to Bjoern on the 04th, but well...
So I republish to a wider audience, who knows  :)

I saw the issue but could not reproduce either on Linux Mint nor on cygwin 
under win.

I join what I got from my linux mint call to gerrit at 9:45pm GMT+1 on the 
4th
David O & Matus are neatly identified as elders.

Could someone log into the machine running the script, then go to the git  
repo folder, launch an interactive python, then try the following lines

import subprocess
all_users = subprocess.check_output(['git','shortlog','-s'])
'David Ostrovsky' in all_users
=> should return true

If still return true and still not exclude elders, we may need to test
(also changing the msg_to target )
by adding this patch

diff --git a/gerritbot/send-daily-digest b/gerritbot/send-daily-digest
index 6786db3..9be7c1f 100755
--- a/gerritbot/send-daily-digest
+++ b/gerritbot/send-daily-digest
@@ -36,6 +36,8 @@ def get_digest(query):

     if do_fresh:
         all_users = init_freshers()
+        with open ('/var/tmp/ausers.log', 'a') as f:
+            f.write (all_users)
         do_module = ['--files', '--patch-sets']

     for line in subprocess.check_output(['ssh', args['gerrit'], 'gerrit',
'query'] + do_module +

so we can scan the all_users list. I attach mine if it helps
if you don't have time for this, may I require a ssh access to the server  
launching the command so I can test almost live ?

Best,

Mat

---
Ce courrier électronique ne contient aucun virus ou logiciel malveillant parce que la protection 
avast! Antivirus est active.
http://www.avast.com