Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the File-MimeInfo CPAN distribution.

Report information
The Basics
Id: 85383
Status: resolved
Priority: 0/
Queue: File-MimeInfo

People
Owner: MICHIELB [...] cpan.org
Requestors: CHORNY [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.16
Fixed in: (no value)



Subject: fails on perl 5.18.0
See for ex. http://www.cpantesters.org/cpan/report/4e33b530-bdc9-11e2-b3a5-4f6010d45c6b perl 5.18.0 should be released in a week. -- Alexandr Ciornii, http://chorny.net
Subject: Re: [rt.cpan.org #85383] fails on perl 5.18.0
Date: Thu, 23 May 2013 15:43:03 +0200
To: bug-File-MimeInfo [...] rt.cpan.org
From: Jaap Karssenberg <jaap.karssenberg [...] gmail.com>
These tests tend to break every other perl release due to perl itself changing. Since I'm not actively workign on this project anymore, this is low prio on my todo list. On Fri, May 17, 2013 at 6:33 PM, Alexandr Ciornii via RT < bug-File-MimeInfo@rt.cpan.org> wrote: Show quoted text
> Fri May 17 12:33:20 2013: Request 85383 was acted upon. > Transaction: Ticket created by CHORNY > Queue: File-MimeInfo > Subject: fails on perl 5.18.0 > Broken in: 0.16 > Severity: (no value) > Owner: Nobody > Requestors: CHORNY@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=85383 > > > > See for ex. > http://www.cpantesters.org/cpan/report/4e33b530-bdc9-11e2-b3a5-4f6010d45c6b > perl 5.18.0 should be released in a week. > > -- > Alexandr Ciornii, http://chorny.net > >
Here's a patch to fix this issue. Cheers, gregor, Debian Perl Group
Subject: regexp-test.patch
Description: add more regexen to test (test failures in perl 5.18) Origin: vendor Bug: https://rt.cpan.org/Ticket/Display.html?id=85383 Forwarded: https://rt.cpan.org/Ticket/Display.html?id=85383 Bug-Debian: http://bugs.debian.org/709795 Author: gregor herrmann <gregoa@debian.org> Last-Update: 2013-05-26 --- a/t/01_normal.t +++ b/t/01_normal.t @@ -19,10 +19,10 @@ # test _glob_to_regexp my $i = 0; for my $glob ( - [ '*.pl', [ '(?-xism:^.*\.pl$)', '(?^u:^.*\.pl$)' ] ], # 4 - [ '*.h++', [ '(?-xism:^.*\.h\+\+$)', '(?^u:^.*\.h\+\+$)' ] ], # 5 - [ '*.[tar].*', [ '(?-xism:^.*\.[tar]\..*$)', '(?^u:^.*\.[tar]\..*$)' ] ], # 6 - [ '*.?', [ '(?-xism:^.*\..?$)', '(?^u:^.*\..?$)' ] ], # 7 + [ '*.pl', [ '(?-xism:^.*\.pl$)', '(?^u:^.*\.pl$)', '(?^:^.*\.pl$)' ] ], # 4 + [ '*.h++', [ '(?-xism:^.*\.h\+\+$)', '(?^u:^.*\.h\+\+$)', '(?^:^.*\.h\+\+$)' ] ], # 5 + [ '*.[tar].*', [ '(?-xism:^.*\.[tar]\..*$)', '(?^u:^.*\.[tar]\..*$)', '(?^:^.*\.[tar]\..*$)' ] ], # 6 + [ '*.?', [ '(?-xism:^.*\..?$)', '(?^u:^.*\..?$)', '(?^:^.*\..?$)' ] ], # 7 ) { my $converted = File::MimeInfo::_glob_to_regexp( $glob->[0] );
RT-Send-CC: jaap.karssenberg [...] gmail.com
Hi Jaap, On Thu May 23 09:43:33 2013, jaap.karssenberg@gmail.com wrote: Show quoted text
> These tests tend to break every other perl release due to perl itself > changing. Since I'm not actively workign on this project anymore, this > is low prio on my todo list.
I understand this. I have an interest in your module; if it would help I would like to offer support in this - it would involve you giving co-maint or transfering ownership of the module to me via PAUSE, I could then make sure maintenance happens on your module. This is the URL where you can change ownership or add co-maintainers: https://pause.perl.org/pause/authenquery?ACTION=share_perms Of course you still would stay the primary author and retain copyright on your work, and could still contribute if you would feel the need! Please let me know if you would be interested in my offer. -- Michiel
Subject: Re: [rt.cpan.org #85383] fails on perl 5.18.0
Date: Mon, 3 Jun 2013 11:47:40 +0200
To: bug-File-MimeInfo [...] rt.cpan.org
From: Jaap Karssenberg <jaap.karssenberg [...] gmail.com>
Hi Michiel, Thank you for the offer and sorry for the long delay in answering you - things have been a bit too hectic here this week :( I just added you as co-maintainer, feel free to commit these fixes. Regards, Jaap On Mon, May 27, 2013 at 12:26 PM, Michiel Beijen via RT < bug-File-MimeInfo@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=85383 > > > Hi Jaap, > > On Thu May 23 09:43:33 2013, jaap.karssenberg@gmail.com wrote:
> > These tests tend to break every other perl release due to perl itself > > changing. Since I'm not actively workign on this project anymore, this > > is low prio on my todo list.
> > I understand this. I have an interest in your module; if it would help I > would like to offer support in this - it would involve you giving co-maint > or transfering ownership of the module to me via PAUSE, I could then make > sure maintenance happens on your module. > > This is the URL where you can change ownership or add co-maintainers: > https://pause.perl.org/pause/authenquery?ACTION=share_perms > > Of course you still would stay the primary author and retain copyright on > your work, and could still contribute if you would feel the need! > > Please let me know if you would be interested in my offer. > -- > Michiel >
RT-Send-CC: jaap.karssenberg [...] gmail.com
Jaap, On Mon Jun 03 05:47:59 2013, jaap.karssenberg@gmail.com wrote: Show quoted text
> Thank you for the offer and sorry for the long delay in answering you - > things have been a bit too hectic here this week :( > > I just added you as co-maintainer, feel free to commit these fixes.
Thanks a lot. Did you use source code control for the module or should i just download the dist off cpan? -- Michiel
RT-Send-CC: jaap.karssenberg [...] gmail.com
On Sun May 26 13:52:51 2013, GREGOA wrote: Show quoted text
> Here's a patch to fix this issue.
Thanks for the fix, and CHORNY thanks for reporting. It's fixed in version 0.17 which is now on CPAN. -- Michiel
Subject: Re: [rt.cpan.org #85383] fails on perl 5.18.0
Date: Tue, 4 Jun 2013 09:25:21 +0200
To: bug-File-MimeInfo [...] rt.cpan.org
From: Jaap Karssenberg <jaap.karssenberg [...] gmail.com>
I have a old branch, but I think the revision control I used for it is not even supported anymore as it merged with the bazaar project. Better to keep your own branch. I I need to make commits I will import again from CPAN to make sure I have latest version anyway. Regards, Jaap On Mon, Jun 3, 2013 at 4:29 PM, Michiel Beijen via RT < bug-File-MimeInfo@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=85383 > > > Jaap, > > On Mon Jun 03 05:47:59 2013, jaap.karssenberg@gmail.com wrote: >
> > Thank you for the offer and sorry for the long delay in answering you - > > things have been a bit too hectic here this week :( > > > > I just added you as co-maintainer, feel free to commit these fixes.
> > Thanks a lot. Did you use source code control for the module or should i > just download the dist off cpan? > -- > Michiel >