Bug #55227 for Lingua-Conlang-Numbers: These modules should be support older Perl

Thu Mar 04 09:41:53 2010 MORIYA [...] cpan.org - Ticket created

Subject:

These modules should be support older Perl

Hello, Mr. Patch. I intended to make Lingua::EO::Numbers, but it was scooped by you :) I respect your great work. In my opinion, these modules should be support older Perl through lack of use // (defined-or operator) and given/when syntax. The latest version of Perl, which treats Unicode correctly, is 5.8.1. I solved it in my "topic/older-perl-support" branch of my repository on GitHub ( http://github.com/gardejo/lingua-conlang-numbers/tree/topic/older-perl-support ), and I'll request you to merge it. I appreciate your consideration. Regards, MORIYA Masaki, alias Gardejo

Thu Mar 04 22:03:26 2010 MORIYA [...] cpan.org - Correspondence added

Subject:	I found bug in "12e2a93" commit
RT-Send-CC:	n [...] atemoya.net

Hellow, Mr. Patch. I'm sorry, I found bug in "12e2a93" commit. I cannot remove "named capture buffer" and "smart matching operator" completely . Because I am in the field now, I cannot fix it. I'll fix it at this evening of JST (at about 1:00 PM of UTC). I owe you a sincere apology for the careless mistakes. Regards, MORIYA Masaki, alias Gardejo

Thu Mar 04 22:03:27 2010 The RT System itself - Status changed from 'new' to 'open'

Fri Mar 05 01:28:55 2010 PATCH [...] cpan.org - Correspondence added

Show quoted text

> In my opinion, these modules should be support older Perl through lack > of use // (defined-or operator) and given/when syntax. > The latest version of Perl, which treats Unicode correctly, is 5.8.1.

I was hoping that I could get away with requiring 5.10 since I thought this was a more esoteric module and I really like the new language features, but I'm happy to provide support for older perls since your request shows there's obviously demand for it. Show quoted text

> I solved it in my "topic/older-perl-support" branch of my repository > on > GitHub ( > http://github.com/gardejo/lingua-conlang-numbers/tree/topic/older- > perl-support > ), and I'll request you to merge it.

I pulled and merged your changes, as well as removing the remaining named captures and smart-match operator. However, I kept 5.10 in the synopsis examples since I like to promote modern Perl and it doesn't prohibit people on older systems from installing and using the modules. Let me know if you see a reason why I shouldn't do that. These changes will be included in a release this weekend. Show quoted text

> I appreciate your consideration.

And I appreciate your help! I'm glad to see folks using my first CPAN module. :) Best, Nick

Fri Mar 05 09:53:07 2010 MORIYA [...] cpan.org - Correspondence added

Hello, Mr. Patch. Thank you for your quick attention. Then, I'm sorry for sending you request in as imperfect condition as Toyota cars. Show quoted text

> However, I kept 5.10 in the > synopsis examples since I like to promote modern Perl and it doesn't > prohibit people on older systems from installing and using the modules.

Because it already passed more than two years since 5.10.0 was released, I also use new features in my application (not in my reusable module). It is very comfortable when I use them. I favor them :) Therefore, I approve your opinion. Because you already pull, merge and correct my code, I close this matter. Have a nice weekend. Regards, MORIYA Masaki, alias Gardejo

Sun Mar 07 19:49:56 2010 PATCH [...] cpan.org - Correspondence added

Tested with perl 5.8.8 and 5.10.1 and just released to CPAN in 0.02.

Sun Mar 07 19:50:00 2010 PATCH [...] cpan.org - Status changed from 'open' to 'resolved'