Aaron is correct. It's the first person who is responsible so you can blame Art for the clutter and then completely ignoring the email telling him to fix it.

Cheers

Nigel

---
Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone : 510-486-5709     Email : NWMoriarty@LBL.gov
Fax   : 510-486-5909       Web  : CCI.LBL.gov

On Mon, Jun 4, 2018 at 4:43 PM, Aaron Brewster <asbrewster@lbl.gov> wrote:
Hi Graeme, I don't find these more annoying than the Jenkins build failure emails.  In fact, these buildbot emails are targeted to whoever committed the clutter, or at least the first one is anyway.  Be happy, Nigel and I get *all* the buildbot continuous-integration emails, you only get a couple!

A smarter build-bot post-commit hook would be nice, one that only alerts once per bad commit.  The relevant class is BBCCTBXCIBuilder in master.cfg in the cci buildbot svn repo.  It would need to get the last commit and only run find_clutter on those files.  Perhaps a --last-commit-only flag could be added to libtbx.find_clutter, which could then be used by BBCCTBXCIBuilder?  Of course, then untended clutter could start to build up...

-Aaron



On Mon, Jun 4, 2018 at 4:25 AM, Graeme.Winter@Diamond.ac.uk <Graeme.Winter@diamond.ac.uk> wrote:
Hi Folks,

Am I alone in finding this kind of message ** very ** annoying

I did not touch this file
I have no business touching this file
If I do fix this then it may cause conflicts for others, which will no doubt annoy them far more than the odd extra space in the code

However I am also now to blame for breaking the Phenix build, in the run up to a release

This is no a happy place

Suggestions?

Thanks & cheerio Graeme

Begin forwarded message:

From: phenix@phenix-online.org<mailto:phenix@phenix-online.org>
Subject: buildbot failure in Phenix on cctbx-ci-ci
Date: 4 June 2018 at 12:11:19 BST
To: graeme.winter@gmail.com<mailto:graeme.winter@gmail.com>
Cc: ASBrewster@LBL.Gov<mailto:ASBrewster@LBL.Gov>, NWMoriarty@LBL.Gov<mailto:NWMoriarty@LBL.Gov>

Build status: FAILURE
Buildslave for this Build:      ci
Complete logs for all build steps:      http://cci-vm-6.lbl.gov:8010/builders/cctbx-ci-ci/builds/5580
Build Reason:   The SingleBranchScheduler scheduler named 'cctbx-ci.poll' triggered this build
Build Source Stamp:     clipper: [branch master] HEAD

cbflib: [branch master] HEAD

cctbx_project: [branch master] 4de5e7f82b0302632282f78c6dda8b999cf6abf6

annlib_adaptbx: [branch master] HEAD

tntbx: [branch master] HEAD

ccp4io_adaptbx: [branch master] HEAD

boost: [branch master] HEAD

gui_resources: [branch master] HEAD


Blamelist:      graeme-winter
If you got an email and you didn't touch this file, it may because someone commited this file before you with an error and hasn't fixed it yet. You might have noticed the error if you ran libtbx.find_clutter on the entire tree.

Detailed log of last build step: http://cci-vm-6.lbl.gov:8010/builders/cctbx-ci-ci/builds/5580/steps/test ALL clutter/logs/stdio<http://cci-vm-6.lbl.gov:8010/builders/cctbx-ci-ci/builds/5580/steps/test%20ALL%20clutter/logs/stdio>
Last 80 lines of "test ALL clutter.stdio"

iota/command_line/iota_single_image.py: tabs or trailing whitespace=2, #10, #11

*** To clean up please use: libtbx.clean_clutter ***


-The Buildbot


--
This e-mail and any attachments may contain confidential, copyright and or privileged material, and are for the use of the intended addressee only. If you are not the intended addressee or an authorised recipient of the addressee please notify us of receipt by returning the e-mail and do not use, copy, retain, distribute or disclose the information in or attached to the e-mail.
Any opinions expressed within this e-mail are those of the individual and not necessarily of Diamond Light Source Ltd.
Diamond Light Source Ltd. cannot guarantee that this e-mail or any attachments are free from viruses and we cannot accept liability for any damage which you may sustain as a result of software viruses which may be transmitted in or with the message.
Diamond Light Source Limited (company no. 4375679). Registered in England and Wales with its registered office at Diamond House, Harwell Science and Innovation Campus, Didcot, Oxfordshire, OX11 0DE, United Kingdom


_______________________________________________
cctbxbb mailing list
cctbxbb@phenix-online.org
http://phenix-online.org/mailman/listinfo/cctbxbb


_______________________________________________
cctbxbb mailing list
cctbxbb@phenix-online.org
http://phenix-online.org/mailman/listinfo/cctbxbb