cvs.spec

[Tomasz Wittner]

Dnia Friday 02 of January 2004 01:07, Andrzej Krzysztofowicz napisał:
> Tomasz Wittner wrote:
> > Dnia Thursday 01 of January 2004 20:07, Paweł Sikora napisał:
> > > paczka sie nie buduje jesli w systemie nie ma jednego
> > > z edytorow: vim, vi, nano, pico
> >
> > Zajrzyj do configure.in . Najpierw jest sprawdzana zawartość zm. śr.
> > $EDITOR , a dopiero potem (jeżeli $EDITOR == '') obecność vim vi emacs
> > nano pico edit. Jeżeli nie jest "pusta", to cvs buduje się z wywołaniem
> > tego, co jest w $EDITOR na potrzeby edycji commitmessages. Przykładowo,
> > za %build wstawiłem: export EDITOR=imaginary_editor i mam
> > (Oczywiście, sensowniej podstawić vim'a [i nie prowadzić świętych wojen])
> >
> > grep imaginary_editor ../LOGS/cvs.spec.log
> > + export EDITOR=imaginary_editor
> > checking for vim... imaginary_editor
> >
> > strings `which cvs` | grep imaginary_editor
> > imaginary_editor
> >
> > Jeżeli żadna ze zm. śr. EDITOR, CVSEDITOR, VISUAL nie jest właściwie
>
> Ale piszesz tu o sprawdzaniu prz _uzywaniu_, czy przy _budowaniu_ ?
> Jesli to pierwsze, to OT.
Przy budowaniu. Jeszcze raz: najpierw jest sprawdzana podczas budowania 
$EDITOR (stąd pomysł, żeby podstawić export EDITOR=/path/to/vim i nie bawić 
się w BR: vim, czy inne), potem jest szukanie przez configure vim vi emacs 
nano pico edit. I wynik tego "szukania ląduje" w binarce. Jeżeli nie masz 
ustawionych [CVS]EDITOR czy VISUAL, to cvs próbuje uruchomić, to co "wpadło' 
podczas budowania (co nijak ma się do dystrybucji)
> Jesli to drugie, to IMO nalezaloby zdefiniowac w specu jedna z tych
> zmiennych (np. na /bin/vi), jesli zadna nie jest zdefiniowana.
Konkretnie EDITOR, bo VISUAL i CVSEDITOR nie są sprawdzane podczas budowania.
Myślę o czymś takim:
cvs diff -u cvs.spec
Index: cvs.spec
 %build
+export EDITOR=/bin/vi
 %{__aclocal}

Tyle, że zaraz zacznie się dyskusja, że ten ma, a tamten nie ma /bin/vi (ja 
nie mam ;-) )
> > ustawiona , to:
> > cvs ci tags_test
> > cvs commit: cannot exec imaginary_editor: No such file or directory
> > cvs commit: warning: editor session failed
> >
> > Log message unchanged or not specified
> > a)bort, c)ontinue, e)dit, !)reuse this message unchanged for remaining
> > dirs Action: (continue)
> >
> > > uprasza sie o dodanie stosownego BR
> >
> > Absolutnie nie widzę takiej potrzeby.
>
> W takim razie configure nie powinien sprawdzac istnienia zadnego z
> _niewymaganych_ edytorow przy _zadnych_ ustawieniach srodowiska.
> Zwlaszcza, jezeli nie jest on _uzywany_ w procesie budowania pakietu.
Ale jest podczas budownia wykorzystywana _domyślnie_ ścieżka z $EDITOR, albo, 
gdy jest $EDITOR == '', to ścieżka do znalezionej binarki vi, vim ... na 
dysku.
Autor sam ma "dramat":
dnl FIXME - Using --without-editor will probably break a compile at
dnl the moment, but maybe it is reasonable for someone to want to
dnl compile a CVS executable that refuses to run if no $EDITOR,
dnl $CVS_EDITOR, or -e option is specified?  Making a preliminary
dnl design decision in this direction, subject to discussion.

> BTW: IMVHO ta sama polityka powinna dotyczyc wymagania/sprawdzania
> obecnosci pakietu less podczas budowania man.spec .
OT. W FreeBSD jest "na dzieńdobry" PAGER=more ;-)
-- 
Tomasz Wittner

2. Always build before committing

--------------------------------
You may be sure you've just corrected some small error and it will
not influence the build process... but try it.