Bug #97672 for Map-Tube-London: london-map.xml Not Installed with cpanm

Fri Aug 01 11:48:52 2014 Smylers [...] stripey.com - Ticket created

Subject:	london-map.xml Not Installed with cpanm
Date:	Fri, 1 Aug 2014 16:48:40 +0100
To:	bug-Map-Tube-London [...] rt.cpan.org
From:	Smylers <Smylers [...] stripey.com>

I successfully installed this module with: $ cpanm Map::Tube::London However, this hasn't installed london-map.xml anywhere, so attempting to use the module for anything fails with: $ perl -wMMap::Tube::London -E 'Map::Tube::London->new->get_shortest_route(@ARGV)' 'Baker Street' 'Farringdon' File does not exist: london-map.xml at /usr/local/share/perl/5.18.2/Map/Tube.pm line 102. And manually copying london-map.xml into to the directory containing London.pm, /usr/local/share/perl/5.18.2/Map/Tube/, still gives that same error message. If I copy london-map.xml to my current directory then it works. If that's the intended mode of operation, then please document it as such. Otherwise, it'd be good if london-map.xml were installed when the module does, and to a location where the module can find it. Cheers Smylers -- Girls don't wear their shoes out, and boys don't care about style? Clarks think so: http://j.mp/clarksgirlboyposters Disagree? Don't want shops promoting gender stereotypes to children? Please sign the petition: http://j.mp/clarksgirlboypetition

Fri Aug 01 14:29:17 2014 ether [...] cpan.org - Correspondence added

On 2014-08-01 08:48:52, Smylers@stripey.com wrote: Show quoted text

> Otherwise, it'd be good if london-map.xml were installed when the > module > does, and to a location where the module can find it.

The normal mechanism for using data files with modules is to use File::ShareDir within the module, and File::ShareDir::Install from Makefile.PL.

Fri Aug 01 14:29:17 2014 The RT System itself - Status changed from 'new' to 'open'

Fri Aug 01 23:06:26 2014 MANWAR [...] cpan.org - Severity Important added

Fri Aug 01 23:06:26 2014 MANWAR [...] cpan.org - Broken in 0.04 added

Fri Aug 01 23:07:04 2014 MANWAR [...] cpan.org - Taken

Fri Aug 01 23:10:59 2014 MANWAR [...] cpan.org - Correspondence added

Hi, Thanks for reporting the issue. I will try to get it sorted asap. I accept it is a bug in my code. Best Regards, Mohammad S Anwar

Fri Aug 01 23:13:28 2014 MANWAR [...] cpan.org - Correspondence added

Hi Karen, Many thanks for your valuable suggestions. I will try to get that sorted asap. Best Regards, Mohammad S Anwar

Sun Aug 03 00:50:57 2014 MANWAR [...] cpan.org - Correspondence added 240 min

Hi, Just uploaded the Map::Tube::London v0.05 with patch for issue your raised earlier. Please give it a try and let me know if this resolves your issue. FYI, the patch is also available on GitHub public repository. https://github.com/Manwar/Map-Tube-London/commits?author=Manwar Best Regards, Mohammad S Anwar

Sun Aug 03 00:50:58 2014 MANWAR [...] cpan.org - Status changed from 'open' to 'patched'

Fri Nov 07 20:51:33 2014 MANWAR [...] cpan.org - Correspondence added

Closing the ticket as issue is resolved.

Fri Nov 07 20:51:34 2014 MANWAR [...] cpan.org - Status changed from 'patched' to 'resolved'

Fri Nov 07 20:51:35 2014 MANWAR [...] cpan.org - Fixed in 0.05 added

Wed Nov 12 08:32:34 2014 Smylers [...] stripey.com - Correspondence added

Subject:	Re: [rt.cpan.org #97672] Resolved: london-map.xml Not Installed with cpanm
Date:	Wed, 12 Nov 2014 13:32:23 +0000
To:	Mohammad Sajid Anwar via RT <bug-Map-Tube-London [...] rt.cpan.org>
From:	Smylers <Smylers [...] stripey.com>

Belatedly confirming that the fix does work for me. Thanks for fixing it, and apologies for not responding sooner. Smylers -- http://twitter.com/Smylers2