Skip Menu |

This queue is for tickets about the Mac-iPod-GNUpod CPAN distribution.

Report information
The Basics
Id: 27519
Status: resolved
Priority: 0/
Queue: Mac-iPod-GNUpod

People
Owner: Nobody in particular
Requestors: geert [...] diff.nl
Cc:
AdminCc:

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



Subject: in add_song the call _getpath($song) -> _getpath($filename)
Date: Sun, 10 Jun 2007 13:03:28 +0200
To: bug-Mac-iPod-GNUpod [...] rt.cpan.org
From: Geert-Jan <geert [...] diff.nl>
Hi, It seems to me that in add_song the line $fh->{path}, my $target) = $self->_getpath($song); should be: $fh->{path}, my $target) = $self->_getpath($filename); since earlier in add_song $filename is set with: if (ref($song) eq 'HASH') { $filename = $song->{filename}; } else { $filename = $song; } In the case when $song is a hash, _getpath($song) still returns a filename that can be used, but without the extension of the actual filename. It seems that the (in my case .mp3) extension is needed by my ipod nano to be able to play yhe song. Hope this is of some use. Thanks for the nice package, Geert-Jan van Opdorp
Hello: Believe it or not this has been in my inbox for over a year, waiting for me to get to it :). Anyway, this fix, along with some other bugfixes, has done into version 1.24, which should show up on CPAN today. On Sun Jun 10 07:10:08 2007, geert@diff.nl wrote: Show quoted text
> Hi, > > It seems to me that in add_song the line > $fh->{path}, my $target) = $self->_getpath($song); > should be: > $fh->{path}, my $target) = $self->_getpath($filename); > > since earlier in add_song $filename is set with: > > if (ref($song) eq 'HASH') { > $filename = $song->{filename}; > } > else { > $filename = $song; > } > > In the case when $song is a hash, _getpath($song) still returns > a filename that can be used, but without the extension of the actual > filename. It seems that the (in my case .mp3) extension is needed > by my ipod nano to be able to play yhe song. > > Hope this is of some use. > Thanks for the nice package, > > Geert-Jan van Opdorp > > > >