Lists: | pgsql-hackers |
---|
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | New copyright program |
Date: | 2011-08-12 20:30:30 |
Message-ID: | 20110812203030.GA18572@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Folks,
I noticed that src/tools/copyright looks like it can only be run on
Bruce's machine, so this translation to Perl is intended:
1. To make the script idempotent, which allows its safe use in
automated tools that might run it many times.
2. To get the script to run on any machine a PostgreSQL developer
would be using, as Perl is already required.
3. To make the script more efficient. As the copyright notice we
need to munge only appears once per file, it stops once it has made a
substitution.
Please find attached a patch implementing same.
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
Attachment | Content-Type | Size |
---|---|---|
copyright_01.diff | text/plain | 2.2 KB |
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 17:51:57 |
Message-ID: | 201108191751.p7JHpv624893@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
David Fetter wrote:
> Folks,
>
> I noticed that src/tools/copyright looks like it can only be run on
> Bruce's machine, so this translation to Perl is intended:
>
> 1. To make the script idempotent, which allows its safe use in
> automated tools that might run it many times.
>
> 2. To get the script to run on any machine a PostgreSQL developer
> would be using, as Perl is already required.
>
> 3. To make the script more efficient. As the copyright notice we
> need to munge only appears once per file, it stops once it has made a
> substitution.
>
> Please find attached a patch implementing same.
Thanks. Applied to HEAD. I never liked putting scripts in git that
only I could run, but I thought if something happened to me, it would be
good to record what I did. The Perl solution is perfect.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From: | Kris Jurka <books(at)ejurka(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 20:42:05 |
Message-ID: | 4E4ECA9D.2090805@ejurka.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 8/19/2011 10:51 AM, Bruce Momjian wrote:
> David Fetter wrote:
>> [Here's a new copyright program.]
>
> Thanks. Applied to HEAD. I never liked putting scripts in git that
> only I could run, but I thought if something happened to me, it would be
> good to record what I did. The Perl solution is perfect.
>
For me this fails with:
Bareword "Tie::File" not allowed while "strict subs" in use at
/home/jurka/pg/server/postgresql/src/tools/copyright.pl line 28.
Using perl -v:
This is perl 5, version 12, subversion 3 (v5.12.3) built for
x86_64-linux-gnu-thread-multi
Additionally it would be nice if this file was marked executable in git.
Kris Jurka
From: | Kris Jurka <books(at)ejurka(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 21:34:01 |
Message-ID: | alpine.BSO.2.00.1108191732050.6502@leary.csoft.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, 19 Aug 2011, Kris Jurka wrote:
> For me this fails with:
>
> Bareword "Tie::File" not allowed while "strict subs" in use at
> /home/jurka/pg/server/postgresql/src/tools/copyright.pl line 28.
>
This fixes things for me. The copyright matching wasn't working for me
either without escaping the parentheses.
Kris Jurka
Attachment | Content-Type | Size |
---|---|---|
fix-copyright.patch | text/plain | 752 bytes |
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Kris Jurka <books(at)ejurka(dot)com> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-19 21:37:09 |
Message-ID: | 20110819213709.GA10821@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, Aug 19, 2011 at 05:34:01PM -0400, Kris Jurka wrote:
> On Fri, 19 Aug 2011, Kris Jurka wrote:
>
> > For me this fails with:
> >
> > Bareword "Tie::File" not allowed while "strict subs" in use at
> > /home/jurka/pg/server/postgresql/src/tools/copyright.pl line 28.
>
> This fixes things for me. The copyright matching wasn't working for me
> either without escaping the parentheses.
Thanks for fixing this :)
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Kris Jurka <books(at)ejurka(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 21:48:42 |
Message-ID: | 201108192148.p7JLmgt14737@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Kris Jurka wrote:
>
>
> On Fri, 19 Aug 2011, Kris Jurka wrote:
>
> > For me this fails with:
> >
> > Bareword "Tie::File" not allowed while "strict subs" in use at
> > /home/jurka/pg/server/postgresql/src/tools/copyright.pl line 28.
> >
>
> This fixes things for me. The copyright matching wasn't working for me
> either without escaping the parentheses.
Was able to reproduce the error you reported with Perl 5.10. I then
tried the single-quote idea I got from Googling, but then got an error
about TIEARRAY being missing, so I recoded it as a simple file
open/close. I also incorported your regex fix. Path attached and
applied. Thanks.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/copyright.fix | text/x-diff | 2.1 KB |
From: | Kris Jurka <books(at)ejurka(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 21:57:44 |
Message-ID: | alpine.BSO.2.00.1108191755530.6502@leary.csoft.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, 19 Aug 2011, Bruce Momjian wrote:
> Was able to reproduce the error you reported with Perl 5.10. I then
> tried the single-quote idea I got from Googling, but then got an error
> about TIEARRAY being missing, so I recoded it as a simple file
> open/close. I also incorported your regex fix. Path attached and
> applied. Thanks.
>
Did you also try the "use Tie::File" addition in my fix because your
current coding doesn't work at all. The tie operation is key to actually
writing out the modified copyright notice. Your version just updates the
copyright year in memory, but never gets it back to the file.
Kris Jurka
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Kris Jurka <books(at)ejurka(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 23:31:44 |
Message-ID: | 201108192331.p7JNViu08867@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Kris Jurka wrote:
>
>
> On Fri, 19 Aug 2011, Bruce Momjian wrote:
>
> > Was able to reproduce the error you reported with Perl 5.10. I then
> > tried the single-quote idea I got from Googling, but then got an error
> > about TIEARRAY being missing, so I recoded it as a simple file
> > open/close. I also incorported your regex fix. Path attached and
> > applied. Thanks.
> >
>
> Did you also try the "use Tie::File" addition in my fix because your
> current coding doesn't work at all. The tie operation is key to actually
> writing out the modified copyright notice. Your version just updates the
> copyright year in memory, but never gets it back to the file.
Ah, that did fix it; thanks. I am attached the updated committed
copyright.pl. I also skipped symlinks.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
Attachment | Content-Type | Size |
---|---|---|
unknown_filename | text/plain | 1.3 KB |
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 23:37:29 |
Message-ID: | 4E4EF3B9.1030407@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> Kris Jurka wrote:
>> Did you also try the "use Tie::File" addition in my fix because your
>> current coding doesn't work at all. The tie operation is key to actually
>> writing out the modified copyright notice. Your version just updates the
>> copyright year in memory, but never gets it back to the file.
> Ah, that did fix it; thanks. I am attached the updated committed
> copyright.pl. I also skipped symlinks.
>
It probably doesn't matter that much in this context, but I should point
out that Tie::File is not universally available. Some years ago I had to
revert its use in the buildfarm code for that reason. In general we
should try to avoid adding extra dependencies wherever possible.
cheers
andrew
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 23:41:31 |
Message-ID: | 201108192341.p7JNfVB10290@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Andrew Dunstan wrote:
>
>
> On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> > Kris Jurka wrote:
> >> Did you also try the "use Tie::File" addition in my fix because your
> >> current coding doesn't work at all. The tie operation is key to actually
> >> writing out the modified copyright notice. Your version just updates the
> >> copyright year in memory, but never gets it back to the file.
> > Ah, that did fix it; thanks. I am attached the updated committed
> > copyright.pl. I also skipped symlinks.
> >
>
> It probably doesn't matter that much in this context, but I should point
> out that Tie::File is not universally available. Some years ago I had to
> revert its use in the buildfarm code for that reason. In general we
> should try to avoid adding extra dependencies wherever possible.
Oh, great. :-(
I can easily change this to rewrite files that contain copyright changes
--- should I?
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From: | Kris Jurka <books(at)ejurka(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)postgresql(dot)org, David Fetter <david(at)fetter(dot)org> |
Subject: | Re: New copyright program |
Date: | 2011-08-19 23:44:04 |
Message-ID: | alpine.BSO.2.00.1108191942490.4415@leary.csoft.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, 19 Aug 2011, Bruce Momjian wrote:
> Andrew Dunstan wrote:
> >
> > It probably doesn't matter that much in this context, but I should point
> > out that Tie::File is not universally available. Some years ago I had to
> > revert its use in the buildfarm code for that reason. In general we
> > should try to avoid adding extra dependencies wherever possible.
>
>
> I can easily change this to rewrite files that contain copyright changes
> --- should I?
>
No. We don't need a super portable copyright year changing script.
Kris Jurka
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 01:02:14 |
Message-ID: | 20110820010214.GB14172@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, Aug 19, 2011 at 07:37:29PM -0400, Andrew Dunstan wrote:
>
>
> On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> >Kris Jurka wrote:
> >>Did you also try the "use Tie::File" addition in my fix because your
> >>current coding doesn't work at all. The tie operation is key to actually
> >>writing out the modified copyright notice. Your version just updates the
> >>copyright year in memory, but never gets it back to the file.
> >Ah, that did fix it; thanks. I am attached the updated committed
> >copyright.pl. I also skipped symlinks.
> >
>
> It probably doesn't matter that much in this context, but I should
> point out that Tie::File is not universally available. Some years
> ago I had to revert its use in the buildfarm code for that reason.
> In general we should try to avoid adding extra dependencies wherever
> possible.
Tie::File ships as part of core Perl for all non-EOL versions, so I
really can't consider this as a problem.
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 01:22:03 |
Message-ID: | 4E4F0C3B.4030408@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 08/19/2011 09:02 PM, David Fetter wrote:
> On Fri, Aug 19, 2011 at 07:37:29PM -0400, Andrew Dunstan wrote:
>>
>> On 08/19/2011 07:31 PM, Bruce Momjian wrote:
>>> Kris Jurka wrote:
>>>> Did you also try the "use Tie::File" addition in my fix because your
>>>> current coding doesn't work at all. The tie operation is key to actually
>>>> writing out the modified copyright notice. Your version just updates the
>>>> copyright year in memory, but never gets it back to the file.
>>> Ah, that did fix it; thanks. I am attached the updated committed
>>> copyright.pl. I also skipped symlinks.
>>>
>> It probably doesn't matter that much in this context, but I should
>> point out that Tie::File is not universally available. Some years
>> ago I had to revert its use in the buildfarm code for that reason.
>> In general we should try to avoid adding extra dependencies wherever
>> possible.
> Tie::File ships as part of core Perl for all non-EOL versions, so I
> really can't consider this as a problem.
Which are those?
cheers
andrew
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 01:39:27 |
Message-ID: | 20110820013927.GA14707@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, Aug 19, 2011 at 09:22:03PM -0400, Andrew Dunstan wrote:
> On 08/19/2011 09:02 PM, David Fetter wrote:
> >On Fri, Aug 19, 2011 at 07:37:29PM -0400, Andrew Dunstan wrote:
> >>
> >>On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> >>>Kris Jurka wrote:
> >>>>Did you also try the "use Tie::File" addition in my fix because your
> >>>>current coding doesn't work at all. The tie operation is key to actually
> >>>>writing out the modified copyright notice. Your version just updates the
> >>>>copyright year in memory, but never gets it back to the file.
> >>>Ah, that did fix it; thanks. I am attached the updated committed
> >>>copyright.pl. I also skipped symlinks.
> >>>
> >>It probably doesn't matter that much in this context, but I should
> >>point out that Tie::File is not universally available. Some years
> >>ago I had to revert its use in the buildfarm code for that reason.
> >>In general we should try to avoid adding extra dependencies
> >>wherever possible.
> >Tie::File ships as part of core Perl for all non-EOL versions, so I
> >really can't consider this as a problem.
>
> Which are those?
5.12 and 5.14 are still supported. 5.10 and earlier are EOL.
http://news.perlfoundation.org/2011/05/perl-514.html
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 01:49:35 |
Message-ID: | 201108200149.p7K1nZN05749@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
David Fetter wrote:
> On Fri, Aug 19, 2011 at 09:22:03PM -0400, Andrew Dunstan wrote:
> > On 08/19/2011 09:02 PM, David Fetter wrote:
> > >On Fri, Aug 19, 2011 at 07:37:29PM -0400, Andrew Dunstan wrote:
> > >>
> > >>On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> > >>>Kris Jurka wrote:
> > >>>>Did you also try the "use Tie::File" addition in my fix because your
> > >>>>current coding doesn't work at all. The tie operation is key to actually
> > >>>>writing out the modified copyright notice. Your version just updates the
> > >>>>copyright year in memory, but never gets it back to the file.
> > >>>Ah, that did fix it; thanks. I am attached the updated committed
> > >>>copyright.pl. I also skipped symlinks.
> > >>>
> > >>It probably doesn't matter that much in this context, but I should
> > >>point out that Tie::File is not universally available. Some years
> > >>ago I had to revert its use in the buildfarm code for that reason.
> > >>In general we should try to avoid adding extra dependencies
> > >>wherever possible.
> > >Tie::File ships as part of core Perl for all non-EOL versions, so I
> > >really can't consider this as a problem.
> >
> > Which are those?
>
> 5.12 and 5.14 are still supported. 5.10 and earlier are EOL.
>
> http://news.perlfoundation.org/2011/05/perl-514.html
Odd the Tie works in my Perl 5.10; I wonder if I installed it somehow.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 02:07:40 |
Message-ID: | 4E4F16EC.1060301@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 08/19/2011 09:39 PM, David Fetter wrote:
>>>
>>> Tie::File ships as part of core Perl for all non-EOL versions, so I
>>> really can't consider this as a problem.
>> Which are those?
> 5.12 and 5.14 are still supported. 5.10 and earlier are EOL.
>
> http://news.perlfoundation.org/2011/05/perl-514.html
>
>
Well, they need to get their story straight.
<http://www.cpan.org/src/README.html> says:
Please note that branches earlier than 5.8 are no longer supported,
though fixes for urgent issues, for example severe security
problems, may still be issued.
What is more, ignoring 5.10 and older would simply foolish. Those
versions are live on many of the platforms we build Postgres on, and in
quite modern distributions too, for example in RHEL 6.
Anyway, they point seems moot in this context.
cheers
andrew
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Kris Jurka <books(at)ejurka(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: New copyright program |
Date: | 2011-08-20 02:14:32 |
Message-ID: | 20110820021432.GA15535@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Fri, Aug 19, 2011 at 09:49:35PM -0400, Bruce Momjian wrote:
> David Fetter wrote:
> > On Fri, Aug 19, 2011 at 09:22:03PM -0400, Andrew Dunstan wrote:
> > > On 08/19/2011 09:02 PM, David Fetter wrote:
> > > >On Fri, Aug 19, 2011 at 07:37:29PM -0400, Andrew Dunstan wrote:
> > > >>
> > > >>On 08/19/2011 07:31 PM, Bruce Momjian wrote:
> > > >>>Kris Jurka wrote:
> > > >>>>Did you also try the "use Tie::File" addition in my fix because your
> > > >>>>current coding doesn't work at all. The tie operation is key to actually
> > > >>>>writing out the modified copyright notice. Your version just updates the
> > > >>>>copyright year in memory, but never gets it back to the file.
> > > >>>Ah, that did fix it; thanks. I am attached the updated committed
> > > >>>copyright.pl. I also skipped symlinks.
> > > >>>
> > > >>It probably doesn't matter that much in this context, but I should
> > > >>point out that Tie::File is not universally available. Some years
> > > >>ago I had to revert its use in the buildfarm code for that reason.
> > > >>In general we should try to avoid adding extra dependencies
> > > >>wherever possible.
> > > >Tie::File ships as part of core Perl for all non-EOL versions, so I
> > > >really can't consider this as a problem.
> > >
> > > Which are those?
> >
> > 5.12 and 5.14 are still supported. 5.10 and earlier are EOL.
> >
> > http://news.perlfoundation.org/2011/05/perl-514.html
>
> Odd the Tie works in my Perl 5.10; I wonder if I installed it somehow.
Sorry for being unclear. I didn't mean to imply that earlier versions
of Perl didn't ship with File::Tie. Just that all supported versions
do.
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate