Re: [riminilug-general] AFTA

[KiyuKo <eof@xxxxxxxxxx>]

> qualcosa cosi' a caso dando un'occhiata veloce:

se e' bastata solo un'occhiata veloce per trovare tutti questi errori... mi 
immagino cosa salterebbe fuori da un'analisi approfondita!

> = Makefile
> * prefix (meglio se PREFIX) ma poi non lo usi! (e cio' e' male)

lo uso per copiare lo scirpt (cp afta $(prefix);\)

> * docdir e mandir dovrebbero essere in funzione di PREFIX

come? spiegati meglio.

> = afta
> * si aspetta di trovare i file che gli servono in /etc/afta (anziche'
>   PREFIX/etc/afta vedi sopra)

forse sono io che ho capito male 'sta storia del prefix... se ce la fai a 
spiegarmela per esteso... tra l'altro ho notato poco fa che cerca il file 
afta.conf nella stessa directory dal quale e' eseguito (mi serviva per i 
test), e non in /etc/afta/afta.conf come dovrebbe, e ho corretto.

> * crea i suoi file in ~ (o li crei nella directory corrente, o
>   specifichi un'opzione per salvarli altrove o crei una directory
>   ~/.afta )

sono solo files temporanei, che dovrebbero (dico dovrebbero perche', in 
effetti, non succede :) essere eliminati subito dopo aver dato
# afta track some_package
per liberare spazio, ma in effetti stavo valutando la possibilita' di tenere 
afta.track e rinominarlo in afta.prepare, di modo da poter evitare di fare
# afta prepare
la volta successiva (a meno di modifiche nel sistema, ovviamente), e credo che 
la tua idea di creare una cartella .afta nella home sia molto buona.

> * AFTA_PATH e' un po' fuorviante potrebbe essere il PATH (inteso nel
>   senso classico del termine) che usa afta, io cambierei tipo in
>   AFTA_DIRS

in effetti e' il corrispettivo del classico PATH, solo che viene usato da 
AFTA, percio' il nome piu' adatto mi sembrava proprio AFTA_PATH...

> * proteggi piu' che puoi tra "" (tipo:
>    66     if echo -n $line | grep / > /dev/null
>   cambialo in if echo -n "$line" eccecc e anche i vari rm)

per quale motivo? me lo puoi spiegare? in effetti devo ammettere che di bash 
scripting non me ne intendo molto (e si vede :)...

> * aggiungerei anche un'opzione --dry-run tipo diff per vedere cosa
>   succederebbe se...

magari in futuro: per ora voglio completare almeno quelle previste dall'help, 
e ci voraa' gia' tempo per quelle...

> altro mi verra' in mente, spero di esserti stato d'aiuto per ora :)

altroche'! e' esattamente il genere di aiuto di cui avevo bisogno... grazie! e 
giusto per dimostrare che sono un tipo onesto, ti voglio offrire un'altra 
possibilita' per aiutarmi: perche' se scrivo

if . afta.conf 2> /dev/null
  echo " Done."
else
  echo " Error."
  exit 1
fi

(questo non c'e' nella versione 0.0.3, ci sto lavorando sopra adesso) non fa 
quello che dovrebbe, cioe' stampare " Done." se il sourcing di afta.conf ha 
avuto successo oppure stampare " Error." e uscire con codice 1 in caso 
contrario?

-----
kiyuko DOT org