devel/jabberd jabberd.spec,1.1,1.2

[Ralf Corsepius]

On Wed, 2005-03-09 at 03:00 -0500, Adrian Reber wrote:
> Author: adrian
> 
> Update of /cvs/extras/devel/jabberd
> In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv10730


> @@ -45,9 +51,11 @@
>  %{__mkdir_p} $RPM_BUILD_ROOT%{_initrddir}
>  %{__install} -m 755 %{SOURCE1} $RPM_BUILD_ROOT%{_initrddir}/%{name}
>  
> -%{__perl} -pi -e 's,__BINDIR__,%{_bindir},g; \
> -                  s,__ETCDIR__,%{sysconfdir},g; \
> -                  s,__PIDDIR__,%{_localstatedir}/jabberd/pid,g;' \
> +%{__sed} -i -e "s,__BINDIR__,%{_bindir},g" \
> +          $RPM_BUILD_ROOT%{_initrddir}/%{name}
> +%{__sed} -i -e "s,__ETCDIR__,%{sysconfdir},g" \
> +          $RPM_BUILD_ROOT%{_initrddir}/%{name}
> +%{__sed} -i -e "s,__PIDDIR__,%{_localstatedir}/jabberd/pid,g" \
>            $RPM_BUILD_ROOT%{_initrddir}/%{name}
>  
>  %{__mkdir_p} $RPM_BUILD_ROOT%{_sysconfdir}/pam.d/

Any particular reason to use several sed-calls? 

One sed-call should be sufficient:

%{__sed} -i -e "s,__BINDIR__,%{_bindir},g" \
            -e "s,__ETCDIR__,%{sysconfdir},g" \
            -e "s,__PIDDIR__,%{_localstatedir}/jabberd/pid,g" \
             $RPM_BUILD_ROOT%{_initrddir}/%{name}

Ralf