public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] Update copyright years, part 2
@ 2015-01-05 12:39 Jakub Jelinek
  2015-01-13 17:16 ` Richard Sandiford
  0 siblings, 1 reply; 5+ messages in thread
From: Jakub Jelinek @ 2015-01-05 12:39 UTC (permalink / raw)
  To: gcc-patches; +Cc: Richard Sandiford, David Malcolm

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

Hi!

Patch too large to attach uncompressed, this
has been created with update-copyright.py --this-year.
Note, I had to temporarily move away gcc/jit/docs/conf.py,
the python script dies on that and leaves almost all files unchanged.

	Jakub

[-- Attachment #2: copyright-2015.bz2 --]
[-- Type: application/x-bzip2, Size: 229305 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [committed] Update copyright years, part 2
  2015-01-05 12:39 [committed] Update copyright years, part 2 Jakub Jelinek
@ 2015-01-13 17:16 ` Richard Sandiford
  2015-01-13 17:22   ` Jakub Jelinek
  0 siblings, 1 reply; 5+ messages in thread
From: Richard Sandiford @ 2015-01-13 17:16 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: gcc-patches, David Malcolm

Jakub Jelinek <jakub@redhat.com> writes:
> Patch too large to attach uncompressed, this
> has been created with update-copyright.py --this-year.
> Note, I had to temporarily move away gcc/jit/docs/conf.py,
> the python script dies on that and leaves almost all files unchanged.

Thanks for doing the update.  Is the patch below OK to fix the JIT thing?
After this change, update-copyright.py --this-year seems to update
gcc/jit correctly (including the texinfo files).

Richard


contrib/
	* update-copyright.py (Copyright.__init__): Add a regexp for
	"copyright = u'".
	(Copyright.update_copyright): Don't add a space before the year
	in that case.

Index: contrib/update-copyright.py
===================================================================
--- contrib/update-copyright.py	2014-08-05 10:29:02.695491816 +0100
+++ contrib/update-copyright.py	2015-01-13 14:13:43.500812967 +0000
@@ -183,6 +183,7 @@ class Copyright:
             '|[Cc]opyright\s+%s'
             '|[Cc]opyright\s+&copy;'
             '|[Cc]opyright\s+@copyright{}'
+            '|copyright = u\''
             '|@set\s+copyright[\w-]+)'
 
             # 2: the years.  Include the whitespace in the year, so that
@@ -363,7 +364,8 @@ class Copyright:
             return (False, orig_line, next_line)
 
         line = (line[:match.start (2)]
-                + ' ' + canon_form + self.separator
+                + ('' if intro.startswith ('copyright = ') else ' ')
+                + canon_form + self.separator
                 + line[match.end (2):])
 
         # Use the standard (C) form.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [committed] Update copyright years, part 2
  2015-01-13 17:16 ` Richard Sandiford
@ 2015-01-13 17:22   ` Jakub Jelinek
  2015-01-15 22:42     ` Richard Sandiford
  0 siblings, 1 reply; 5+ messages in thread
From: Jakub Jelinek @ 2015-01-13 17:22 UTC (permalink / raw)
  To: gcc-patches, David Malcolm, richard.sandiford

On Tue, Jan 13, 2015 at 05:06:35PM +0000, Richard Sandiford wrote:
> Jakub Jelinek <jakub@redhat.com> writes:
> > Patch too large to attach uncompressed, this
> > has been created with update-copyright.py --this-year.
> > Note, I had to temporarily move away gcc/jit/docs/conf.py,
> > the python script dies on that and leaves almost all files unchanged.
> 
> Thanks for doing the update.  Is the patch below OK to fix the JIT thing?
> After this change, update-copyright.py --this-year seems to update
> gcc/jit correctly (including the texinfo files).
> 
> Richard
> 
> 
> contrib/
> 	* update-copyright.py (Copyright.__init__): Add a regexp for
> 	"copyright = u'".
> 	(Copyright.update_copyright): Don't add a space before the year
> 	in that case.

Ok, thanks.

> --- contrib/update-copyright.py	2014-08-05 10:29:02.695491816 +0100
> +++ contrib/update-copyright.py	2015-01-13 14:13:43.500812967 +0000
> @@ -183,6 +183,7 @@ class Copyright:
>              '|[Cc]opyright\s+%s'
>              '|[Cc]opyright\s+&copy;'
>              '|[Cc]opyright\s+@copyright{}'
> +            '|copyright = u\''
>              '|@set\s+copyright[\w-]+)'
>  
>              # 2: the years.  Include the whitespace in the year, so that
> @@ -363,7 +364,8 @@ class Copyright:
>              return (False, orig_line, next_line)
>  
>          line = (line[:match.start (2)]
> -                + ' ' + canon_form + self.separator
> +                + ('' if intro.startswith ('copyright = ') else ' ')
> +                + canon_form + self.separator
>                  + line[match.end (2):])
>  
>          # Use the standard (C) form.

	Jakub

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [committed] Update copyright years, part 2
  2015-01-13 17:22   ` Jakub Jelinek
@ 2015-01-15 22:42     ` Richard Sandiford
  0 siblings, 0 replies; 5+ messages in thread
From: Richard Sandiford @ 2015-01-15 22:42 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: gcc-patches, David Malcolm, richard.sandiford

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

Jakub Jelinek <jakub@redhat.com> writes:
> On Tue, Jan 13, 2015 at 05:06:35PM +0000, Richard Sandiford wrote:
>> Jakub Jelinek <jakub@redhat.com> writes:
>> > Patch too large to attach uncompressed, this
>> > has been created with update-copyright.py --this-year.
>> > Note, I had to temporarily move away gcc/jit/docs/conf.py,
>> > the python script dies on that and leaves almost all files unchanged.
>> 
>> Thanks for doing the update.  Is the patch below OK to fix the JIT thing?
>> After this change, update-copyright.py --this-year seems to update
>> gcc/jit correctly (including the texinfo files).
>> 
>> Richard
>> 
>> 
>> contrib/
>> 	* update-copyright.py (Copyright.__init__): Add a regexp for
>> 	"copyright = u'".
>> 	(Copyright.update_copyright): Don't add a space before the year
>> 	in that case.
>
> Ok, thanks.

For the record, here's the result of running the new script on gcc/jit.
Committed as obvious.

Thanks,
Richard



[-- Attachment #2: jit-copyright.patch.bz2 --]
[-- Type: application/x-bzip2, Size: 2624 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [committed] Update copyright years, part 2
@ 2016-01-04 14:30 Jakub Jelinek
  0 siblings, 0 replies; 5+ messages in thread
From: Jakub Jelinek @ 2016-01-04 14:30 UTC (permalink / raw)
  To: gcc-patches; +Cc: Richard Sandiford, David Malcolm

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

Hi!

Patch too large to attach uncompressed, this
has been created with update-copyright.py --this-year.
The script died when not modified, I'm going to post a tweak for that.
I've also rotated a few ChangeLog files.

	Jakub

[-- Attachment #2: copyright-2016.bz2 --]
[-- Type: application/x-bzip2, Size: 274652 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-01-04 14:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-05 12:39 [committed] Update copyright years, part 2 Jakub Jelinek
2015-01-13 17:16 ` Richard Sandiford
2015-01-13 17:22   ` Jakub Jelinek
2015-01-15 22:42     ` Richard Sandiford
2016-01-04 14:30 Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).