search

danielvarga / hunglish-webapp

Automatically exported from code.google.com/p/hunglish-webapp
0 stars 0 forks source link

clean up absolute-relative paths in .properties files #44

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
A leheto legkevesebb helyen legyen a propertiesben abszolut path. Az osszes 
tobbi path relativ pathkent legyen megadva a fent emlitett abszolut path-okhoz 
kepest.

Mostanra stabilizalodott az alkalmazas konyvtar-layout-ja, ugyhogy haromnal 
tobb abszolut path szinte biztosan felesleges, annak a jele, hogy valami me'g 
nincs vegiggondolva. A harom, amirol tudok:
1. hol vannak a nyelvi eroforrasok.
2. hol van az ugynevezett deployment konyvtar, ami alatt minden adatkonyvtar 
van. A java webapp nezopontjabol ezek kozul egyebkent csak harom relevans van: 
fileUpload,hunglishIndex,hunglishIndexTmp.
3. hol van a harness_cronjob.sh.

A kodban termeszetesen egyaltalan semmilyen path ne legyen, se relativ, se 
abszolut.

Original issue reported on code.google.com by Varga.Da...@gmail.com on 1 Mar 2011 at 9:03

GoogleCodeExporter commented 9 years ago

Original comment by attila.b...@gmail.com on 2 Mar 2011 at 1:47

GoogleCodeExporter commented 9 years ago 
Nem Java issue, hanem python, de az is ide tartozik, hogy ki kell talalni, hogy 
hol legyen a control_harness.py. (Jelenleg az eles alkalmazas dilettans modon a 
munkakonyvtaramra referal.)

A tiszta megoldas talan az, ha a tomcat deploy lerakja valahova 
/srv/tomcat6/webapps/hunglish ala a src/main/python konyvtarat, es a java tudja 
hogy ott kell keresse a harness_cronjob.sh-t, a harness_cronjob.sh pedig tudja, 
hogy ott kell keresse a control_harness-t. Az me'g ezutan is homalyos, hogy a 
control_harness hol keresse a harness.py-t, de ez kisebb gond, es ezt is 
megoldana', ha atkerulne google code ala a harness a cvs alol.

Original comment by Varga.Da...@gmail.com on 11 Mar 2011 at 2:10

GoogleCodeExporter commented 9 years ago

Original comment by Varga.Da...@gmail.com on 7 Apr 2011 at 10:53

GoogleCodeExporter commented 9 years ago 
Nagyjából kipucoltam a python kódból a beégetett pathokat. Most pontosan a 
következő a logika:

1. A Java a src/main/resources/META-INF/spring/application.properties konfigbol 
tudja, hogy hova van deploy-olva a webapp, es hol van az adatokat tartalmazo 
deployment konyvtar.

2. Nem jottem ra pontosan hogyan, de ez alapjan kifilozza, hogy hol van a 
harness_cronjob.sh, es MockJob.java meghivja ket argumentummal: a deployment 
konyvtarnak, es a control_harness.py konyvtaranak helyevel.

3. A harness_cronjob.sh kilogikazza a datum es a deployment helye alapjan, hogy 
hova kell loggolni. Meghivja a control_harness.py-t. A loggolas ket szinten is 
megjelenik: a harness_cronjob.sh atiranyitja a c_h.py kimeneteit, plusz odaadja 
neki ergumentumkent, hogy hova kell loggolnia a harness futasokat. (logprefix)

4. A c_h.py eloszor is egy eleg gonosz dolgot csinal: megallapitja, hogy melyik 
konyvtarban fut o" maga. Ez alapjan beegetve kiszamol ket konyvtarat ez alatt:
- tcg/harness a hely, ahol meghivja a harness.py-t. 
- tcg/scripts -et odaadja a harness-nek a --startup_values=scripts_dir=? 
szintaxissal, amit defaultnak hasznal a harness a commandfile feldolgozasa 
soran.

A commandfile-ban azert me'g maradt egy TODO: a 
pipeline_dir=/big3/Work/HunglishMondattar/ még mindig be van égetve a 
hunglishcommands-ba, ezt hivatkozva eri el a binárisokat harness.binaries 
alatt (pl. hunalign) és a resource-okat resources alatt (pl. textcat.models).

Original comment by Varga.Da...@gmail.com on 18 May 2011 at 12:42