Re: [PATCH] Remove all Author(s): lines from source file headers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 13, 2018 at 01:27:44PM +0100, Erik Skultety wrote:
> On Thu, Dec 13, 2018 at 11:55:50AM +0000, Daniel P. Berrangé wrote:
> > In many files there are header comments that contain an Author:
> > statement, supposedly reflecting who originally wrote the code.
> > In a large collaborative project like libvirt, any non-trivial
> > file will have been modified by a large number of different
> > contributors. IOW, the Author: comments are quickly out of date,
> > omitting people who have made significant contribitions.
> >
> > In some places Author: lines have been added despite the person
> > merely being responsible for creating the file by moving existing
> > code out of another file. IOW, the Author: lines give an incorrect
> > record of authorship.
> >
> > With this all in mind, the comments are useless as a means to identify
> > who to talk to about code in a particular file. Contributors will always
> > be better off using 'git log' and 'git blame' if they need to  find the
> > author of a particular bit of code.
> >
> > This commit thus deletes all Author: comments from the source and adds
> > a rule to prevent them reappearing.
> >
> > The Copyright headers are similarly misleading and inaccurate, however,
> > we cannot delete these as they have legal meaning, despite being largely
> > inaccurate. In addition only the copyright holder is permitted to change
> > their respective copyright statement.
> >
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> 
> This fails on apibuild.py with:
> Header libvirt-common lacks a Author description (...aand a bunch of other
> headers too)
> 
> You might want to adjust the script not to require the field when serializing
> the API XML.

I'll squash this trivial change in

diff --git a/docs/apibuild.py b/docs/apibuild.py
index 5e218a9ad0..b1b5881943 100755
--- a/docs/apibuild.py
+++ b/docs/apibuild.py
@@ -2270,7 +2270,7 @@ class docBuilder:
         output.write("    <file name='%s'>\n" % (module))
         dict = self.headers[file]
         if dict.info is not None:
-            for data in ('Summary', 'Description', 'Author'):
+            for data in ('Summary', 'Description'):
                 try:
                     output.write("     <%s>%s</%s>\n" % (
                                  data.lower(),
diff --git a/docs/devhelp/devhelp.xsl b/docs/devhelp/devhelp.xsl
index add5794deb..17a980c73a 100644
--- a/docs/devhelp/devhelp.xsl
+++ b/docs/devhelp/devhelp.xsl
@@ -72,7 +72,6 @@
          <xsl:if test="deprecated">
            <p> WARNING: this module is deprecated !</p>
          </xsl:if>
-         <p>Author(s): <xsl:value-of select="author"/></p>
          <div class="refsynopsisdiv">
          <h2>Synopsis</h2>
          <pre class="synopsis">

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux