On Sun, 27 Aug 2000, Geoffrey Lee wrote:
> > In install script of kdebase-1.93-9mdk.i586.rpm, there are lines like :
> >
> > chmod a+x /etc/X11/gdm/Sessions/KDE
> > chmod 1755 /usr/bin/konsole_grantpty
> > chmod 1755 /usr/bin/kdesu
> >
> > So, if the package is checked with rpm -V, we see immediately that :
> >
> > .M...... /usr/bin/kdesu
>
> ChrisM sucks, Chris should have used %attr for that I think.
>
Chris would have if the code was not changing daily. I need a SUID bit on one
file out of > 350. Nice.....
> > Even after a fresh install. Unpleasant, isn't it ?
>
> Yuck!!
>
> > Is there any good reason to use chmod in RPMs scripts ?
>
> No. %attr or %dedattr should be used in the filelist.
I can use chmod or I can use 350 some exact file names in a development
package. The chmod will be eliminated before the final 7.2 release, however
currently with the code still changing on a daily basis there was no better
way to do this. Yes, it should be a %attr, but you all have a choice:
1. We do no more code updates from kde cvs and I go to a static file list.
2. We leave this the way it is for now so I can continue updating and not
spending 10 hours to rebuild 2 packages and possible messing a lot of things
up.
Personally, I thing #2 is preferable judging from the large number of bug
fixes still being checked in and much more critical things that need to be
fixed.
-Chris
