[Bug 1693013] Review Request: perl-Test-RunValgrind - Tests that an external program is valgrind-clean

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

 



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

Robert-André Mauchin <zebob.m@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |POST
                 CC|                            |zebob.m@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |zebob.m@xxxxxxxxx
              Flags|                            |fedora-review+



--- Comment #1 from Robert-André Mauchin <zebob.m@xxxxxxxxx> ---
 - Don't mix %{buildroot} and  $RPM_BUILD_ROOT

 - Don't package json and yaml files

 - %{__perl} → perl



URL and Source addresses are Ok.
Source archive (SHA-256:
70947565ad0be3e5d0cd9aca9e1fd0cb07c873e574310e92e8eca629ec6cd631f4e62631) is
original. Ok.
Summary verified from lib/Test/RunValgrind.pm. Ok.
Description verified from llib/Test/RunValgrind.pm. Ok.
License verified from lib/Test/RunValgrind.pm and README.md. Ok.
No XS code, BuildArch is noarch.

$ rpmlint perl-Test-RunValgrind.spec
review-perl-Test-RunValgrind/results/perl-Test-RunValgrind-0.2.0-1.fc31.noarch.rpm
review-perl-Test-RunValgrind/results/perl-Test-RunValgrind-0.2.0-1.fc31.src.rpm 
perl-Test-RunValgrind.noarch: W: spelling-error Summary(en_US) valgrind -> val
grind, val-grind, Valerian
perl-Test-RunValgrind.noarch: W: spelling-error %description -l en_US valgrind
-> val grind, val-grind, Valerian
perl-Test-RunValgrind.noarch: W: spelling-error %description -l en_US manpage
-> manage, man page, man-page
perl-Test-RunValgrind.src: W: spelling-error Summary(en_US) valgrind -> val
grind, val-grind, Valerian
perl-Test-RunValgrind.src: W: spelling-error %description -l en_US valgrind ->
val grind, val-grind, Valerian
perl-Test-RunValgrind.src: W: spelling-error %description -l en_US manpage ->
manage, man page, man-page
2 packages and 1 specfiles checked; 0 errors, 6 warnings.

$ rpm -q -lv -p perl-Test-RunValgrind-0.2.0-1.fc31.noarch.rpm           
17:18:32
drwxr-xr-x    2 root    root                        0 Mar 29 17:10
/usr/share/doc/perl-Test-RunValgrind
-rw-r--r--    1 root    root                      631 Jan 26  2018
/usr/share/doc/perl-Test-RunValgrind/Changes
-rw-r--r--    1 root    root                    16603 Jan 26  2018
/usr/share/doc/perl-Test-RunValgrind/META.json
-rw-r--r--    1 root    root                    10385 Jan 26  2018
/usr/share/doc/perl-Test-RunValgrind/META.yml
-rw-r--r--    1 root    root                    10385 Mar 29 17:10
/usr/share/doc/perl-Test-RunValgrind/MYMETA.yml
-rw-r--r--    1 root    root                      318 Jan 26  2018
/usr/share/doc/perl-Test-RunValgrind/README
drwxr-xr-x    2 root    root                        0 Mar 29 17:10
/usr/share/licenses/perl-Test-RunValgrind
-rw-r--r--    1 root    root                     1158 Jan 26  2018
/usr/share/licenses/perl-Test-RunValgrind/LICENSE
-r--r--r--    1 root    root                     4156 Mar 29 17:10
/usr/share/man/man3/Test::RunValgrind.3pm.gz
drwxr-xr-x    2 root    root                        0 Mar 29 17:10
/usr/share/perl5/vendor_perl/Test
-r--r--r--    1 root    root                     8103 Mar 29 17:10
/usr/share/perl5/vendor_perl/Test/RunValgrind.pm
File permissions and layout are Ok.

$ rpm -q --requires -p perl-Test-RunValgrind-0.2.0-1.fc31.noarch.rpm | sort -f
| uniq -c
      1 perl(:MODULE_COMPAT_5.28.1)
      1 perl(:VERSION) >= 5.14.0
      1 perl(Carp)
      1 perl(Path::Tiny)
      1 perl(strict)
      1 perl(Test::More)
      1 perl(Test::Trap)
      1 perl(warnings)
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

rpm -q --provides -p perl-Test-RunValgrind-0.2.0-1.fc31.noarch.rpm 
perl(Test::RunValgrind) = 0.2.0
perl-Test-RunValgrind = 0.2.0-1.fc31
Binary provides are Ok.

Package builds in Koji
(https://koji.fedoraproject.org/koji/taskinfo?taskID=33816549).
Ok


Resolution: Package approved.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux