Author: glen                         Date: Fri Feb  8 21:18:52 2008 GMT
Module: SPECS                         Tag: HEAD
---- Log message:
- rpm 4.4.9+ compatible %clean

---- Files affected:
SPECS:
   FHS.spec (1.129 -> 1.130) 

---- Diffs:

================================================================
Index: SPECS/FHS.spec
diff -u SPECS/FHS.spec:1.129 SPECS/FHS.spec:1.130
--- SPECS/FHS.spec:1.129        Thu Jan 24 23:39:02 2008
+++ SPECS/FHS.spec      Fri Feb  8 22:18:47 2008
@@ -1,5 +1,9 @@
 # $Revision$, $Date$
-# NOTE: don't use %{_*dir} macros for paths defined by FHS
+# NOTE
+# - don't use %{_*dir} macros for paths defined by FHS
+
+# avoid rpm 4.4.9 adding rm -rf buildroot, we need the dirs to check 
consistency
+%define                __spec_clean_body       %{nil}
 Summary:       Basic FHS 2.3 filesystem layout
 Summary(de.UTF-8):     Grundlegende Dateisystemstruktur
 Summary(fr.UTF-8):     Arborescence de base du système de fichiers
@@ -11,11 +15,13 @@
 License:       GPL
 Group:         Base
 URL:           http://www.pathname.com/fhs/
+BuildRequires: mktemp
 BuildRequires: rpmbuild(macros) >= 1.213
 Requires:      setup >= 2.4.6-4
 BuildRoot:     %{tmpdir}/%{name}-%{version}-root-%(id -u -n)
 
 %define                _locmandir      /usr/local/man
+
 # doesn't contain any files, but we're not noarch package
 %define        no_install_post_strip   1
 %define        no_install_post_chrpath 1
@@ -80,19 +86,23 @@
 %clean
 cd $RPM_BUILD_ROOT
 
-# %{_rpmfilename} is not expanded, so use
-# %{name}-%{version}-%{release}.%{buildarch}.rpm
-RPMFILE=%{name}-%{version}-%{release}.%{_target_cpu}.rpm
-TMPFILE=%{name}-%{version}.tmp$$
-find . | sed -e 's|^\.||g' -e 's|^$||g' | sort | grep -v $TMPFILE > $TMPFILE
-
-# find finds also '.', so use option -B for diff
-if rpm -qpl %{_rpmdir}/$RPMFILE | grep -v '^/$' | sort | diff -uB $TMPFILE - ; 
then
-       rm -rf $RPM_BUILD_ROOT
-else
-       echo -e "\nNot so good, some directories are not included in package\n"
-       exit 1;
-fi
+check_filesystem_dirs() {
+       # %{_rpmfilename} is not expanded, so use
+       # %{name}-%{version}-%{release}.%{buildarch}.rpm
+       RPMFILE=%{name}-%{version}-%{release}.%{_target_cpu}.rpm
+       TMPFILE=$(mktemp)
+       find . | sed -e 's|^\.||g' -e 's|^$||g' | LC_ALL=C sort > $TMPFILE
+
+       # find finds also '.', so use option -B for diff
+       if rpm -qpl %{_rpmdir}/$RPMFILE | grep -v '^/$' | LC_ALL=C sort | diff 
-uB $TMPFILE - ; then
+               rm -rf $RPM_BUILD_ROOT
+       else
+               echo -e "\nNot so good, some directories are not included in 
package\n"
+               exit 1
+       fi
+       rm -f $TMPFILE
+}
+check_filesystem_dirs
 
 %files
 %defattr(644,root,root,755)
@@ -207,6 +217,9 @@
 All persons listed below can be reached at <cvs_login>@pld-linux.org
 
 $Log$
+Revision 1.130  2008-02-08 21:18:47  glen
+- rpm 4.4.9+ compatible %clean
+
 Revision 1.129  2008-01-24 22:39:02  glen
 - reason why no_install_post_{strip,chrpat,compress_modules} are used, however 
these could as well not be here
 
================================================================

---- CVS-web:
    
http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/SPECS/FHS.spec?r1=1.129&r2=1.130&f=u

_______________________________________________
pld-cvs-commit mailing list
[email protected]
http://lists.pld-linux.org/mailman/listinfo/pld-cvs-commit

Reply via email to