diff --git a/perl-Inline.spec b/perl-Inline.spec
index e6bd78b..4f2dc50 100644
--- a/perl-Inline.spec
+++ b/perl-Inline.spec
@@ -1,19 +1,20 @@
 Name:           perl-Inline
 Version:        0.44
-Release: 	16
+Release:        17%{?dist}
 Summary:        Inline Perl module
 
 Group:          Development/Libraries
 License:        GPL+ or Artistic
 Url:            http://search.cpan.org/dist/Inline/
-Source0:      http://www.cpan.org/authors/id/I/IN/INGY/Inline-%{version}.tar.gz
+Source0:        http://www.cpan.org/authors/id/I/IN/INGY/Inline-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 BuildArch:      noarch
-BuildRequires:  perl >= 1:5.6.1
 BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(Parse::RecDescent)
-Requires:  perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
+# Uncomment the next line when bug #351711 is fixed, and perl-Inline-Files is included in Fedora
+#BuildRequires:  perl(Inline::Files)
+Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
 
 %description
 The Inline module allows you to put source code from other programming
@@ -38,10 +39,12 @@ never notice the compilation time.
 %prep
 %setup -q -n Inline-%{version} 
 
+
 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor < /dev/null
 make %{?_smp_mflags}
 
+
 %install
 rm -rf $RPM_BUILD_ROOT
 make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
@@ -49,9 +52,11 @@ find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} ';'
 find $RPM_BUILD_ROOT -type d -depth -exec rmdir {} 2>/dev/null ';'
 chmod -R u+w $RPM_BUILD_ROOT/*
 
-%check || :
+
+%check
 make test
 
+
 %clean 
 rm -rf $RPM_BUILD_ROOT
 
@@ -64,6 +69,9 @@ rm -rf $RPM_BUILD_ROOT
 
 
 %changelog
+* Wed Oct 24 2007 Robin Norwood <rnorwood@redhat.com> - 0.44-17
+- Various fixes from package review
+
 * Tue Oct 16 2007 Tom "spot" Callaway <tcallawa@redhat.com> - 0.44-16
 - correct license tag
 - add BR: perl(ExtUtils::MakeMaker)