[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 #8 from Petr Šabata <psabata@xxxxxxxxxx> ---
(In reply to Ben Boeckel from comment #7)
> (In reply to Petr Šabata from comment #6)
> > 1. I would use the release URL instead of just pointing to the commit (which
> > could be something entirely different and nobody will notice).  For example
> > https://github.com/s-aska/%{name}/archive/%{version}.tar.gz should work just
> > fine.  The tarball filename will be just the version plus extension but that
> > doesn't really matter.
> 
> https://fedoraproject.org/wiki/Packaging:SourceURL#Github

Indeed.  Let me quote your link:

"If the upstream does not create tarballs for releases, you can use this
mechanism to produce them. If the upstream does create tarballs you should use
them as tarballs provide an easier trail for people auditing the packages."

Your upstream provides tarballs.
https://github.com/s-aska/dropbox-api-command/releases

> > 2. Missing buildtime dependencies: perl, perl(CPAN::Meta),
> > perl(CPAN::Meta::Prereqs), perl(File::Basename), perl(File::Spec),
> > perl(strict), perl(utf8), perl(warnings)
> 
> Hmm. Is there a way to detect these automatically? I think
> perl(WebService::Dropbox) also necessary. Or is that just a runtime dep? I'm
> not all that well-versed in Perl stuff.

Yes, perl(WebService::Dropbox) is just a runtime dependency and is generated
automatically by rpmbuild (via perl-generators).  It's required by
script/dropbox-api which isn't used neither during %build nor %check phases.

You can use the `tangerine' command (from perl-Tangerine) to check perl files
for what modules they provide or require.  For example in your case, to see
buildtime deps you could run `tangerine Build.PL lib t'.

-- 
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=dzg7RAXxue&a=cc_unsubscribe
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel





[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux