[vmchecker-dev] [code review] main Makefile

[Alexandru Moşoi]

si code review-ul

-Makefile:
cred ca poti sa faci regula aia fie un for. for dir in a b c; do ... ;
done. e mai clar si mai putin error prone.

-commander/Makefile
commander: commander.o
commander.o: commander.cpp
sunt doua reguli implicite. cred ca poti fi sterse.
dupa dist: all sunt doua taburi, unul e suficient

-vmexecutor/Makefile
acelasi lucru cu regulile implicite..


si inca un lucru te rog. code reviews inseamna doar review, nu mai
modifica sursa adrianei, caci poti introduce accidental un bug si
trebuie ca adriana sa invete ca liniile lungi sunt a bad habbit.


On Wed, Sep 17, 2008 at 2:38 PM, Lucian Adrian Grijincu
<lucian.grijincu at gmail.com> wrote:
> Salut,
>
> Am tras în trunk/ un Makefile simpluț cu două reguli: `dist` și
> `clean`. Ăsta se duce în subdirectoare și cheamă aceeași regulă pe
> makefile-ul de acolo. Am modificat și Makefile-urile din Commander/ și
> VMexecutor/ pentru a băga reguli de `dist` -- regula asta construiește
> binarul din acel director (dacă s-au modificat sursele) și-l copiază
> în ../bin/.
>
> Mai e nevoie de ceva la `make dist`?
>
> --
> Lucian
> _______________________________________________
> vmchecker-dev mailing list
> vmchecker-dev at lists.rosedu.org
> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev
>



-- 
Alexandru Moşoi
http://alexandru.mosoi.googlepages.com

Please avoid sending me Word or PowerPoint attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html