On Saturday, 09 June 2007, at 20:58:48 (+0200),
Olivier Thauvin wrote:

> hum, code has since rpm 4.4.8 so.

Check rpm-4_5 branch. :)

> I know what is the problem, I added the chown myself.

The chmod is one possible solution, but there are others.  IMHO, the
chmod could easily be moved into %install.

> (yes install removing read perm for the user sucks, but that's the
> life).

Only for bad packages.  And packages can always be patched. :)

> I'll check in cvs code, but it will failed in the case I am talking
> about w/o proper chown.
> 
> This software is munin, and there are other application doing that.

So in munin's spec file:

%define __spec_clean_body \
%{?buildroot:%{__chown} -R u+rX %{buildroot} ; %{__rm} -rf %{buildroot}}\
%{nil}

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <mej@kainx.org>
n + 1, Inc., http://www.nplus1.net/       Author, Eterm (www.eterm.org)
-----------------------------------------------------------------------
 "You did not tell the truth, and so you will have to pay the
  consequences."                -- Bob Barker, "Truth or Consequences"