[vmchecker-dev] Status Commander

[Alexandru Moşoi]

2008/9/8 Szekeres Adriana <aaa_sz at yahoo.com>:
> Ceau,
>
> Commanderul este aproape gata. Am nevoie de un code review.

implementare:
- pentru stergere exista unlink . nu mai apela rm -rf
- in unele locuri ai exit(0). daca am inteles eu bine implementarea,
acel exit(0) nu mai sterge niste fisiere.
- replace spaces() -> nu exista ceva in stl care sa faca acelasi lucru?

chestii de coding style:
- spatii inainte si dupa operatori ( a = b nu a=b, a + b nu a+b)
- temp+"scp "+jobs_path+"/"+RUN_OUTPUT_FILE+ " "+ username+"@"+
ip+":"+"\""+ vmchecker_root+ "/"+ "checked"+"/"+ job_id+
"/"+user_id+"/" +upload_s+ "/"+  RUN_OUTPUT_FILE+"\"" -> poti sa o
faci intr-o functie. mult mai lizibil
- scurteaza liniile... la mine unele din liniile tale sunt scrise pe
doua randuri :). uite. ca guideline: nu depasi 79 caractere pe linie.
- fara using namespace std;
foloseste using std::cout; using std::ifstream; etc
- lina 152: if ( (spatiu intre if si paranteza)
- liniile 204-218: idem
- (temp+":VMPath").c_str() ?? ":VMPath" nu e mult mai lizibil?
- (char*)"Cannot get tests.zip from Upload System" -> fara (char *).



>
> M-am uitat peste check_config.py si din cate am inteles avem o variabila de mediu vmchecker_root. Eu o primesc prin intermediul instantei unei teme (inainte era BasePath, intre timp s-a modificat in VMCheckerRoot?).  Intrebarea mea este de ce VMCheckerRoot este de forma : /home/student/Curs1
> si nu doar /home/student/VMChecker caci bin-urile unde le gasim? Nu in vmchecker_root/bin ? Sau scripturile pentru deadline sau scripturile pentru fiecare vm(asta e TODO la executor).
>
> Deci nu ar trebui sa primesc doar un CursID si vmchecker_root sa o iau ca environment variable?
>
> Eu am inteles o structura de genul (spuneti-mi daca nu e bine):
>
>
> vmchecker_root/bin/*.py etc.
>              /deadline_scripts
>              /config_files
>              /jobs (imi tb un dir. intermediar unde sa pun arhivele si scripturile pentru executor)
>              /scripts (aici voi pune scripturile de build si run de forma vmname_run.sh; vmname_build.sh)
>              /Curs1
>              /Curs2
>              ....
>
> Acuma de ex pentru Curs1:
>
> /Curs1/unchecked/hw1.ini
>                /hw2.ini
>
>      /tests/tema1+vm_name.zip (sau doar temaid.zip, ma rog)
>            /tema2+vm_name.zip
>
>      /checked/temaid/nume/upload_t1/
>                          /upload_t2/
>
>      /back/
>
> Oricum, cand va uitati peste Commander va rog sa verificati caile bine.
>
> 10x,
> Adriana
>
>
>
> _______________________________________________
> 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