[Bug 1195573] Review Request: dropbox-api-command - Dropbox API wrapper command
by Red Hat Bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195573
--- Comment #8 from Petr Šabata <psabata(a)redhat.com> ---
(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
9 years, 4 months
[Bug 1195573] Review Request: dropbox-api-command - Dropbox API wrapper command
by Red Hat Bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195573
--- Comment #7 from Ben Boeckel <mathstuf(a)gmail.com> ---
(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
> 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.
> 3. Use perl macros for the perl lib paths in %files; changing
> %{_datadir}/perl5/App/dropboxapi.pm to %{perl_vendorlib}/* would be fine.
Ah, I used rpmdev-newspec -t perl for this. Will do.
> 4. A more useful %description would be nice. This isn't necessary.
I'll put something together.
> 5. You shouldn't need to define prefix, I suppose?
rpmdev-newspec's doing; I'll remove it if that's more modern.
> 6. Perhaps the project Github page would work better for the URL.
Probably. Will do.
--
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=Pfv1gkInjQ&a=cc_unsubscribe
9 years, 4 months