diff --git a/aqbanking.spec b/aqbanking.spec
index a5f1a27..d118575 100644
--- a/aqbanking.spec
+++ b/aqbanking.spec
@@ -4,7 +4,7 @@
 Name: aqbanking
 Summary: A library for online banking functions and financial data import/export
 Version: 2.1.0
-Release: 11
+Release: 12
 Source: http://download.sourceforge.net/aqbanking/%{name}-%{version}.tar.gz
 Group: System Environment/Libraries
 License: GPL
@@ -140,7 +140,7 @@ sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool
 make
 
 %install
-[ "$RPM_BUILD_ROOT" != "/" ] && rm -rf $RPM_BUILD_ROOT
+rm -rf $RPM_BUILD_ROOT
 
 make DESTDIR=$RPM_BUILD_ROOT install
 
@@ -157,6 +157,8 @@ mv $RPM_BUILD_ROOT/%{_datadir}/doc/{aqbanking,aqhbci} $RPM_BUILD_ROOT/%{_datadir
 
 %find_lang %{name}
 
+exit 1
+
 %clean
 [ "$RPM_BUILD_ROOT" != "/" ] && rm -rf $RPM_BUILD_ROOT
 
@@ -183,9 +185,9 @@ mv $RPM_BUILD_ROOT/%{_datadir}/doc/{aqbanking,aqhbci} $RPM_BUILD_ROOT/%{_datadir
 %{_libdir}/libcbanking.so.*
 %{_libdir}/aqbanking
 %exclude %{_libdir}/aqbanking/plugins/*/debugger
-%exclude %{_libdir}/aqbanking/plugins/*/frontends/*
+%exclude %{_libdir}/aqbanking/plugins/*/frontends/qbanking
 %exclude %{_libdir}/aqbanking/plugins/*/wizards
-%{_libdir}/gwenhywfar
+%{_libdir}/gwenhywfar/*
 %{_datadir}/aqbanking
 %exclude %{_datadir}/aqbanking/frontends/*
 %exclude %{_datadir}/aqbanking/i18n
@@ -207,25 +209,30 @@ mv $RPM_BUILD_ROOT/%{_datadir}/doc/{aqbanking,aqhbci} $RPM_BUILD_ROOT/%{_datadir
 %{_datadir}/aclocal/cbanking.m4
 
 %files -n g2banking
+%defattr(-,root,root)
 %{_libdir}/libg2banking.so.*
 %{_datadir}/aqbanking/frontends/g2banking
 
 %files -n g2banking-devel
+%defattr(-,root,root)
 %{_bindir}/g2banking-config
 %{_includedir}/g2banking
 %{_libdir}/libg2banking.so
 %{_datadir}/aclocal/g2banking.m4
 
 %files -n kbanking
+%defattr(-,root,root)
 %{_libdir}/libkbanking.so.*
 
 %files -n kbanking-devel
+%defattr(-,root,root)
 %{_bindir}/kbanking-config
 %{_includedir}/kbanking
 %{_libdir}/libkbanking.so
 %{_datadir}/aclocal/kbanking.m4
 
 %files -n qbanking
+%defattr(-,root,root)
 %{_bindir}/qb-help
 %{_libdir}/libqbanking.so.*
 %{_libdir}/aqbanking/plugins/*/debugger
@@ -235,6 +242,7 @@ mv $RPM_BUILD_ROOT/%{_datadir}/doc/{aqbanking,aqhbci} $RPM_BUILD_ROOT/%{_datadir
 %{_datadir}/aqbanking/frontends/qbanking
 
 %files -n qbanking-devel
+%defattr(-,root,root)
 %{_bindir}/qbanking-config
 %{_includedir}/qbanking
 %{_libdir}/libqbanking.so
@@ -245,6 +253,11 @@ mv $RPM_BUILD_ROOT/%{_datadir}/doc/{aqbanking,aqhbci} $RPM_BUILD_ROOT/%{_datadir
 /usr/lib/python?.?/site-packages/%{name}
 
 %changelog
+* Mon Jan 15 2007 Bill Nottingham <notting@redhat.com> - 2.1.0-12
+- fix missing %%defattrs
+- fix %%excludes
+- other cleanups from review
+
 * Sat Jan 13 2007 Bill Nottingham <notting@redhat.com> - 2.1.0-11
 - split into a variety of packages