Code review comment for lp:~mdavidsaver/epics-base/devlib-cleanup

Revision history for this message
Andrew Johnson (anj) wrote :

Hi Michael,

Please ensure all files have an appropriate Copyright header, I at least one file is missing this completely and the other files don't currently have Brookhaven listed as a Copyright holder.
Delete the old RCS expanded $Id$ keywords; you must have taken your originals from an official EPICS Base release since they are all missing the $ signs. Bazaar only supports keywords with a plugin, but $Id$ is not one of those anyway, so we'll probably drop all the $Id$ keywords currently in Base.

I haven't actually looked at the code yet...

« Back to merge proposal