merge 391617 403754
thanks
hi,
On Tue, Dec 19, 2006 at 04:17:27PM +0100, Romain Beauxis wrote:
> Please correct and update the perl module documentation.
> Actual man page is wrong and leads to false positive bugs, such as
> #391617.
>
> The two major mistakes I have found are:
> * Password seems to be needed at initialisation for submit to succeed.
hm, well, imho the manpage says it clearly:
[..]
Also, the submit method requires that the following configuration
parameters be set for this "Audio::Scrobbler" object:
* username
The username of the user's AudioScrobbler registration.
* password
The password for the AudioScrobbler registration.
[..]
> * The parameter passing on submit is submit({ 'artist' => 'blabla, ..})
> and not as stated submit('artist' => 'blabla', ...) which leads to
> the above bug.
yes, thats not clearly statet in the manpage, which only speaks about a
"hash referenced by the info parameter". One could guess it. Im going to
add the missing brackets in the SYNOPSYS section with the next upload.
> Also, please consider a complete and working example and not only 'this
> and this parameter is needed', it saves times !
the package ships /usr/bin/scrobbler-helper, you might want to have a
look at this one. Example config is shipped aswell. Im going to add a
pointer to this script in README.Debian.
bye,
- michael
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
