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
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
https://bugzilla.redhat.com/show_bug.cgi?id=1195862
Denis Fateyev <denis(a)fateyev.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags| |fedora-cvs?
--- Comment #4 from Denis Fateyev <denis(a)fateyev.com> ---
New Package SCM Request
=======================
Package Name: perl-Class-Virtual
Short Description: Base class for virtual base classes in Perl
Owners: dfateyev
Branches: f20 f21 f22 el6 epel7
InitialCC:
--
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=E4Ig5dHvIG&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=1195573
--- Comment #6 from Petr Šabata <psabata(a)redhat.com> ---
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.
2. Missing buildtime dependencies: perl, perl(CPAN::Meta),
perl(CPAN::Meta::Prereqs), perl(File::Basename), perl(File::Spec),
perl(strict), perl(utf8), perl(warnings)
3. Use perl macros for the perl lib paths in %files; changing
%{_datadir}/perl5/App/dropboxapi.pm to %{perl_vendorlib}/* would be fine.
4. A more useful %description would be nice. This isn't necessary.
5. You shouldn't need to define prefix, I suppose?
6. Perhaps the project Github page would work better for the URL.
--
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=RMbyYtMaKD&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=1195573
Petr Šabata <psabata(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
CC| |psabata(a)redhat.com
Assignee|nobody(a)fedoraproject.org |psabata(a)redhat.com
Flags| |fedora-review?
--- Comment #5 from Petr Šabata <psabata(a)redhat.com> ---
I'll do the rest of the review :)
--
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=qLX2Ymvpry&a=cc_unsubscribe