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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #11 from Petr Šabata <psabata@xxxxxxxxxx> ---
(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.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]