Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFA/doco] Update the "Start of New Year Procedure"
Date: Fri, 01 Jan 2010 11:41:00 -0000	[thread overview]
Message-ID: <20100101114034.GV2788@adacore.com> (raw)
In-Reply-To: <83eimaf6k7.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 1263 bytes --]

> > +Run the @code{copyright.sh} script to add the new year in the copyright
> > +notices of most source files.  This script requires Emacs 22 or later to
> > +be installed.
> 
> It's better to put copyright.sh in @file, not in @code.

Fixed.

>   @item
>   @file {*.f}
>   ...
> 
> Did you have a good reason for not using the canonical form?  If not,
> I'd prefer to use it, although I won't object to what you did as well.

No problem, I did not have a specific reason for writing it this way,
other than it seems easier to read it.  I also fixed another list
just a few lines above.

> Also, perhaps it is worthwhile to spell out the language for each file
> type.  Again, that's not a blocking comment.

I don't think this is worth the effort, here. This is just a checklist
for someone like me who needs an easy reminder of which files will need
to be updated by hand. I don't need to know in which language each file
is written to know how to update it. It's usually obvious once opened.

Here is the patch I ended up checking in.

2010-01-01  Joel Brobecker  <brobecker@adacore.com>

        Update the "Start of New Year Procedure".
        * gdbint.texinfo: Add the list of files that need to be updated
        manually.  Minor reformatting.

-- 
Joel

[-- Attachment #2: start-of-new-year.diff --]
[-- Type: text/x-diff, Size: 1694 bytes --]

Index: gdbint.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdbint.texinfo,v
retrieving revision 1.315
diff -u -p -r1.315 gdbint.texinfo
--- gdbint.texinfo	1 Jan 2010 07:54:37 -0000	1.315
+++ gdbint.texinfo	1 Jan 2010 11:37:17 -0000
@@ -6752,15 +6752,44 @@ Update the copyright year in the startup
 
 Update the copyright year in:
 @itemize @bullet
-@item file @file{top.c}, function @code{print_gdb_version}
-@item file @file{gdbserver/server.c}, function @code{gdbserver_version}
-@item file @file{gdbserver/gdbreplay.c}, function @code{gdbreplay_version}
+  @item
+  file @file{top.c}, function @code{print_gdb_version}
+  @item
+  file @file{gdbserver/server.c}, function @code{gdbserver_version}
+  @item
+  file @file{gdbserver/gdbreplay.c}, function @code{gdbreplay_version}
 @end itemize
 
 @item
-Add the new year in the copyright notices of all source and documentation
-files.  This can be done semi-automatically by running the @code{copyright.sh}
-script.  This script requires Emacs 22 or later to be installed.
+Run the @file{copyright.sh} script to add the new year in the copyright
+notices of most source files.  This script requires Emacs 22 or later to
+be installed.
+
+@item
+The new year also needs to be added manually in all other files that
+are not already taken care of by the @file{copyright.sh} script:
+@itemize @bullet
+  @item
+  @file{*.s}
+  @item
+  @file{*.f}
+  @item
+  @file{*.f90}
+  @item
+  @file{*.igen}
+  @item
+  @file{*.ac}
+  @item
+  @file{*.texi}
+  @item
+  @file{*.texinfo}
+  @item
+  @file{*.tex}
+  @item
+  @file{*.defs}
+  @item
+  @file{*.1}
+@end itemize
 
 @end itemize
 

  reply	other threads:[~2010-01-01 11:41 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-01  9:10 Joel Brobecker
2010-01-01 10:42 ` Eli Zaretskii
2010-01-01 11:41   ` Joel Brobecker [this message]
  -- strict thread matches above, loose matches on Subject: below --
2009-01-06  5:03 [RFA/doco] Update the ``Start of New Year Procedure'' Joel Brobecker
2009-01-06  6:36 ` Daniel Jacobowitz
2009-01-07  4:43   ` Joel Brobecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100101114034.GV2788@adacore.com \
    --to=brobecker@adacore.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox