diff --git a/python-urlgrabber.spec b/python-urlgrabber.spec
index fb8dcd5..aa72db0 100644
--- a/python-urlgrabber.spec
+++ b/python-urlgrabber.spec
@@ -3,13 +3,12 @@
 Summary: A high-level cross-protocol url-grabber
 Name: python-urlgrabber
 Version: 3.0.0
-Release: 1%{?dist}
+Release: 2%{?dist}
 Source0: urlgrabber-%{version}.tar.gz
 Patch0: urlgrabber-keepalive.patch
-License: LGPL
+License: LGPLv2+
 Group: Development/Libraries
-BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-buildroot
-Prefix: %{_prefix}
+BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root
 BuildArch: noarch
 BuildRequires: python-devel
 Url: http://linux.duke.edu/projects/urlgrabber/
@@ -21,13 +20,14 @@ and file locations.  Features include keepalive, byte ranges, throttling,
 authentication, proxies and more.
 
 %prep
-%setup -n urlgrabber-%{version}
+%setup -q -n urlgrabber-%{version}
 %patch0 -p0
 
 %build
 python setup.py build
 
 %install
+rm -rf $RPM_BUILD_ROOT
 python setup.py install -O1 --root=$RPM_BUILD_ROOT
 rm -rf $RPM_BUILD_ROOT/%{_docdir}/urlgrabber-%{version}
 
@@ -35,13 +35,15 @@ rm -rf $RPM_BUILD_ROOT/%{_docdir}/urlgrabber-%{version}
 rm -rf $RPM_BUILD_ROOT
 
 %files
-%defattr(-,root,root)
+%defattr(-,root,root,-)
 %doc ChangeLog LICENSE README TODO
-%dir %{python_sitelib}/urlgrabber
-%{python_sitelib}/urlgrabber/*
+%{python_sitelib}/urlgrabber
 %{_bindir}/urlgrabber
 
 %changelog
+* Mon Aug 27 2007 Jeremy Katz <katzj@redhat.com> - 3.0.0-2
+- fixes for package review (#226347)
+
 * Thu May 31 2007 Jeremy Katz <katzj@redhat.com> - 3.0.0-1
 - update to 3.0.0