Ainsi parlait David Walser :
> Guillaume Rousse wrote:
> > [Contrib-RPM]
> > Name        : imp3                         Relocations: (not
> > relocateable) Version     : 3.2.2                             Vendor:
> > MandrakeSoft Release     : 1mdk                          Build Date: Mon
> > 08 Sep 2003 10:46:41 PM CEST -=-=-=-
> > Guillaume Rousse <[EMAIL PROTECTED]> 3.2.2-1mdk
> >
> > - 3.2.2
> > - speac cleanup
>
> You call that a cleanup?  Geez, I'd hate to see your room.
I admit it is a question of preference, but i really think having 
standardisation between all horde specs, all macros defined in one place, 
pretty printed tags, some some comments for anyone reading the spec, and 
dropping ugly apache 1 post scripts can be called a cleanup

> > - remove useless files from webroot (.dist, doc files, .po)
> > - properly tag localisation files
> > - drop apache1 integration
> > - drop ugly configuration scripts, perl rulez
>
> Well that's fine, but part of the point before was, the script was included
> with the package, so if the user changed their hostname later on, they
> could run the script again and have their config be right again.  Do the
> same with your perl script.
All the script was doing was to replace "localhost.localdomain" by the correct 
values in a config file, using two scripts, a shell one and a awk one. It 
tooks me almost 10 minutes to understand it, whereas a single one-liner in 
perl could do in one single place. Moreover, i don't see the point of adding 
aditional code, meaning potential security and maintainance issues,
for such a trivial task, especially for what is supposed to be a server. 
Either you write a complete configuration wizard, and install it elsewhere in 
the filesystem, either you just takes cares of it in post-install 
configuration step.
-- 
Any system which depends upon human reliability is unreliable
                -- SNAFU Equations (JB's Scholastic Laws) n�2


Reply via email to