Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

Jitka Plesnikova <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |
              Flags|                            |fedora-review+

--- Comment #7 from Jitka Plesnikova <[email protected]> ---
Spec file changes:

 BuildRequires:  perl(Module::Build) >= 0.36
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Capture::Tiny)
+BuildRequires:  perl(ExtUtils::Manifest)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec::Functions)
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(Fatal)
+BuildRequires:  perl(Exporter)

> FIX: Add missing BR for tests (./t/lib/TestUtils.pm)
> perl(ExtUtils::Manifest)
> perl(File::Path)
> perl(File::Spec::Functions)
> perl(Cwd)
> perl(Fatal)
> perl(Exporter)
Ok

 %install
 ./Build install destdir=%{buildroot} create_packlist=0
 find %{buildroot} -type f -name '*.bs' -size 0 -exec rm -f {} \;
-find %{buildroot} -depth -type d -exec rmdir {} 2>/dev/null \;

> TODO: Remove the deleting empty directories in %install section. This is 
> default behavior.
Ok.


Package APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to