[Bug 1195573] Review Request: dropbox-api-command - Dropbox API wrapper command

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1195573



--- Comment #11 from Petr Šabata <psabata at redhat.com> ---
(In reply to Ben Boeckel from comment #10)
> Spec URL: http://mathstuf.fedorapeople.org//dropbox-api-command.spec
> SRPM URL:
> http://mathstuf.fedorapeople.org//dropbox-api-command-1.17-4.fc23.src.rpm
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=9096834
> 
> So --prefix is required and %{perl_vendorlib} is the wrong path.

No, neither is true ;) The problem is you're using non-existant Module::Build
option; I missed that earlier.  Change `INSTALLDIRS' (used by
ExtUtils::MakeMaker) to `installdirs' or `--installdirs' (the latter also works
with Module::Build::Tiny).  Then you will be installing to the correct path,
you won't have to define prefix and you will be able to use the standard noarch
perl path macro.

Also, the `perl' buildtime dependency is still missing.

And I ack the rest of the issues were fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=jt0Zct9Mri&a=cc_unsubscribe