https://bugzilla.redhat.com/show_bug.cgi?id=2126943
Damian Wrobel changed:
What|Removed |Added
Resolution|--- |RAWHIDE
Status|ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #16 from Gwyn Ciesla ---
(fedscm-admin): The Pagure repository was created at
https://src.fedoraproject.org/rpms/perl-Template-Plugin-CGI
--
You are receiving this mail because:
You are on the CC list for the bug.
You are
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #15 from Petr Pisar ---
It would make sense and I asked for it RPM developers some time ago. (I cannot
a link now.) But the idea was rejected with an explanation that there are
actually two kinds of run-time dependencies.
One is
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #14 from Damian Wrobel ---
(In reply to Petr Pisar from comment #13)
> Thanks for polishing the package.
Thank you for the review.
> TODO: You can remove the underspecified, and now redundant, dependency on
> unversioned
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
Petr Pisar changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #13 from
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #12 from Damian Wrobel ---
(In reply to Petr Pisar from comment #11)
> Now reviewing the second release of the spec file.
>
> TODO: You can use a plain 'perl' command instead of macro '%{__perl}'
>
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #11 from Petr Pisar ---
Now reviewing the second release of the spec file.
TODO: You can use a plain 'perl' command instead of macro '%{__perl}'
(perl-Template-Plugin-CGI.spec:32, perl-Template-Plugin-CGI.spec:47).
FIX:
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #10 from Petr Pisar ---
(In reply to Damian Wrobel from comment #6)
> (In reply to Petr Pisar from comment #1)
>
> I'm in a process of fixing the issue and have the following doubts:
>
> > FIX: Build-require 'perl(Template) >=
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #9 from Petr Pisar ---
(In reply to Damian Wrobel from comment #5)
> For the reverse direction I'm getting:
>
> [dw@dell Plugin]$ scandeps.pl -V -B -R CGI.pm
> # Template/Plugin.pm [module]
> # Template/Plugin/CGI.pm [module]
>
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #8 from Petr Pisar ---
(In reply to Damian Wrobel from comment #4)
> Here is what I see:
>
> [dw@dell Plugin]$ scandeps.pl CGI.pm
> 'CGI::Cookie' => '4.54',
> 'CGI::File::Temp' => '4.54',
> 'CGI::Util' =>
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #7 from Damian Wrobel ---
(In reply to Petr Pisar from comment #1)
> URL and Source0 addresses are usable. Ok.
> Source0 archive (SHA512:
> 2ca0a9f0a72a2c26e73a5e7202ae3773349c032acc0c15b016a84e629fe326ac1fd53f45f0ffa
>
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #6 from Damian Wrobel ---
(In reply to Petr Pisar from comment #1)
I'm in a process of fixing the issue and have the following doubts:
> FIX: Build-require 'perl(Template) >= 3.100' (t/cgi.t:21).
This doesn't exist. dnf
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #5 from Damian Wrobel ---
For the reverse direction I'm getting:
[dw@dell Plugin]$ scandeps.pl -V -B -R CGI.pm
# Template/Plugin.pm [module]
# Template/Plugin/CGI.pm [module]
# base.pm [module]
# strict.pm [module]
# warnings.pm
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #4 from Damian Wrobel ---
Here is what I see:
[dw@dell Plugin]$ scandeps.pl CGI.pm
'CGI::Cookie' => '4.54',
'CGI::File::Temp' => '4.54',
'CGI::Util' => '4.54',
'Carp'=> '1.52',
'Cwd'
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #3 from Petr Pisar ---
Your are completely right: If a Perl module requires CGI::Cookie, then an RPM
package which delivers the module should require perl(CGI::Cookie). Not
perl(CGI).
However, in Template-Plugin-CGI-3.101 I
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #2 from Damian Wrobel ---
(In reply to Petr Pisar from comment #1)
Thank you for taking the review and for your valuable comments. I'll fix the
issues.
>
> FIX: Remove unused dependencies: perl(Carp), perl(CGI::Cookie),
>
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
--- Comment #1 from Petr Pisar ---
URL and Source0 addresses are usable. Ok.
Source0 archive (SHA512:
2ca0a9f0a72a2c26e73a5e7202ae3773349c032acc0c15b016a84e629fe326ac1fd53f45f0ffa6d23311bc1fa7f7ac1645df187373c59f5c6b1046985655d1cd)
is
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
Petr Pisar changed:
What|Removed |Added
Doc Type|--- |If docs needed, set a value
https://bugzilla.redhat.com/show_bug.cgi?id=2126943
Damian Wrobel changed:
What|Removed |Added
Blocks||2124543
Referenced Bugs:
19 matches
Mail list logo