cpan-testers / CPAN-Reporter

(Perl) Adds CPAN Testers reporting to CPAN.pm
http://search.cpan.org/dist/CPAN-Reporter/
20 stars 18 forks source link

path with spaces in "transport" [rt.cpan.org #59286] #55

Open xdg opened 8 years ago

xdg commented 8 years ago

https://rt.cpan.org/Ticket/Display.html?id=59286

CPAN::Reporter does not like path with spaces for "id_file" key

Example:
transport=Metabase uri https://metabase.cpantesters.org/beta/ id_file
"C:\Documents and Settings\Sasha\Мои
документы\.cpanreporter\metabase_id.json"

I guess problem is in line
my @transport_args = split " ", $transport;

First idea is to use Text::CSV, but this will mean one more prereq.

-- 
Alexandr Ciornii, http://chorny.net
garu commented 8 years ago

I added a fix for this in CPAN::Testers::Common::Client, if it becomes solid enough I'll merge it into CPAN::Reporter. If I get enough tuits I might even finish porting CTCC back into CPAN::Reporter (which would also fix the issue).

More information on this as it develops!