plptools.spec

[Bobby Spark]

On Wed, Sep 29, 2004 at 04:26:48PM +0200, Paweł Sakowski wrote:
> On Wed, 2004-09-29 at 01:44 +0200, Bobby Spark wrote:
> 
> > poprawki zrobiłem przy użyciu sed/awk/grep w sekcji prep bo tak mi było
> > łatwiej pracować niż patcha robić, a nawet nie wiem na ile to się nadaje
> > jeśli ok, mogę to do patcha przerzucić...
> 
> To:
> 
> > +grep -B1000 "AC_OUTPUT(" conf/configure.in.in | \
> > +       grep -v "AC_OUTPUT(" > configure.in.in
> > +cat << EOF >> configure.in.in
> > +AC_CONFIG_FILES([ plpprint/prolog.ps ])
> > +AC_CONFIG_FILES([ doc/ncpd.man ])
> > +AC_CONFIG_FILES([ doc/plpnfsd.man ])
> > +AC_CONFIG_FILES([ doc/plpftp.man ])
> > +AC_CONFIG_FILES([ doc/sisinstall.man ])
> > +AC_CONFIG_FILES([ doc/plpprintd.man ])
> > +AC_CONFIG_FILES([ doc/plpbackup.man ])
> > +EOF
> > +rm conf/configure.in.in configure{.in,} po/Makefile.in.in
> 
> i to:
> 
> > +for MFILE in `find -name Makefile.am`; do
> > +       awk ' /INCLUDES/ { sub(/$/, " -I$(top_srcdir)/include") }; {print}' \
> > +               $MFILE > ${MFILE}2
> > +       mv -f ${MFILE}2 $MFILE
> > +done
> 
> jest bardzo dobrym przykładem jak _nie_ modyfikować plików ze speca.
> Przez sed -i można robić zmiany typu s,/usr/lib,%{_libdir},g , tutaj na
> pierwszy rzut oka nie widać co się dzieje, a jest i duża szansa że przy
> następnej wersji taka sztuczka przestanie działać (szczególnie w
> przypadku configure.in.in). Przynajmniej te dwa wstawiłbym do patcha.
>
wiem że nie widać, poprostu chciałem pokazać jak to działa, jeśli się
nadaje to przerzucę to do patchy
 
> > +for LANG in en de pl; do
> > +       MFILE=kde2/doc/kioslave/$LANG/Makefile.am
> > +       echo "KDE_LANG = $LANG" > $MFILE
> > +       echo "KDE_DOCS = kioslave" >> $MFILE
> > +done
> 
> Tu byłoby czytelniej gdybyś pisał co wycinasz z M.am, a nie co
> zostawiasz. I też raczej widziałbym tu patcha.
> 
brak było KDE_DOCS = kioslave, a reszta chyba też psuła
tu poprostu Makefile.am były strasznie dziwnie zrobione, unsermak'owi te
dwie linijki wystarczą

> Przy okazji, odpowiednie kawałki speca po zoptymalizowaniu:
> 
> sed -i '/INCLUDES/s,$,-I$(top_srcdir)/include,'
> sed -i '/AC_OUTPUT/,/)/d;/_ALL_SOURCE/d'
> 
jeśli jest akceptowalne to zrobię z tego wszystkiego patcha

> I poprosiłbym o jakiś opis czym się zajmuje cvs_fixes.patch, cobym miał
> co do commit loga wstawić.
cvs_fixes to wszystkie zmiany jakie do tej pory są w plikach źrodłowych
(h, cpp, c, cc) w cvs'ie, głównie zmiany pozycji nagłówków (kosmetyka),
zmiana iostream.h (obsloete) -> iostream, strstream -> sstream, tak
samo, do tego to co do tej pory było w patchach assert i c++; i niewiele
więcej



po{zdrowi,wodz}enia ;)
-- 
 ____      Sparky{PI]                 _   ___   _   _
/____)  ___   ___   _ _  ||          |   | _ \ | |   |
\____\ | -_) '___|  ||^' ||// \\//  <    |  _/ | |   |
(____/ ||    (_-_|_ ||   ||\\  ||    |_  |_|   |_|  _|