CVS

[ChakaZulu]

Hi,

ich hab' mal das CVS ausgecheckt und bin auf zwei kleinere Sachen gestossen:

1) Das ant-skript enthält Windowsverzeichnisangaben
2) Es gibt kein compile target (etwas komisch, wenn man jtjg aus einem jar starten will, dass nur .java enthält )

hier mal die diffs, vielleicht könnt ihr das ein oder andere ja übernehmen

Code: Alles auswählen

Index: build.xml
===================================================================
RCS file: /cvsroot/jackjgrabber/jtg/build.xml,v
retrieving revision 1.9
diff -u -r1.9 build.xml
--- build.xml   27 Dec 2004 02:01:01 -0000      1.9
+++ build.xml   27 Dec 2004 21:47:23 -0000
@@ -5,7 +5,7 @@

        <property name="targetJarName"          value="jackTheJGrabber.jar" />
        <property name="zipFileName"            value="jackTheJGrabber.zip" />
-       <property name="targetDir"                      value="f:\jackTheJGrabber" />
+       <property name="targetDir"                      value="." />
        <property name="mainclass"                      value="JackTheJGrabber.ControlMain" />
        <property name="classpath1"             value="./lib/dom4j.jar" />
        <property name="classpath2"             value="./lib/log4j-1.2.8.jar" />
@@ -25,8 +25,19 @@
                <delete file="${targetDir}/${zipFileName}"/>
        </target>

+       <!-- Compilieren der Sourcen -->
+       <target name="compile">
+               <javac srcdir="." debug="on">
+                       <classpath>
+                               <fileset dir="lib">
+                                       <include name="*.jar"/>
+                               </fileset>
+                       </classpath>
+               </javac>
+       </target>
+
         <!-- Erstellung des Jars aus den eigenen Sourcen -->
-       <target depends="cleanup" name="buildjar">
+       <target depends="cleanup,compile" name="buildjar">
                <jar destfile="${targetDir}/${targetJarName}">
                        <fileset dir="">
                                <include name="${sourcedir1}"/>
@@ -66,4 +77,4 @@
                                update="true"
                  />
        </target>
-</project>
\ No newline at end of file
+</project>
Index: start.sh
===================================================================
RCS file: /cvsroot/jackjgrabber/jtg/start.sh,v
retrieving revision 1.2
diff -u -r1.2 start.sh
--- start.sh    3 Dec 2004 19:34:50 -0000       1.2
+++ start.sh    27 Dec 2004 21:47:30 -0000
@@ -1,6 +1,7 @@
 #!/bin/bash
-JAVA=/opt/jdk1.5.0
+JAVA=`which java`
 JTGHOME=/home/ralph/jtg

 cd $JTGHOME
-$JAVA/bin/java -jar jackTheJGrabber.jar
\ No newline at end of file
+$JAVA  -jar jackTheJGrabber.jar
+

Im CVS hat sich noch was bzgl. der Timer getan, falls Ihr es noch
nicht gemerkt habt, hier zwei Patches (ich hoffe, das ist so ok). Damit sollte alles wie bisher funktionieren, in der GUI ist die neue Option allerdings nicht drin
Und einen Fipptehler hab ich auch gefunden

Code: Alles auswählen

Index: boxConnection/SerBoxControlNeutrino.java
===================================================================
RCS file: /cvsroot/jackjgrabber/jtg/boxConnection/SerBoxControlNeutrino.java,v
retrieving revision 1.42
diff -u -r1.42 SerBoxControlNeutrino.java
--- boxConnection/SerBoxControlNeutrino.java    19 Dec 2004 16:28:22 -0000      1.42
+++ boxConnection/SerBoxControlNeutrino.java    27 Dec 2004 21:59:47 -0000
@@ -310,7 +310,7 @@
             botimer.timerNumber=st.nextToken();
             botimer.eventTypeId=st.nextToken();
             botimer.eventRepeatId=st.nextToken();
-
+            botimer.repeatCount=st.nextToken();
             valueAnno=st.nextToken();
                    valueStart=st.nextToken();
                    valueStop=st.nextToken();

Index: model/BOTimer.java
===================================================================
RCS file: /cvsroot/jackjgrabber/jtg/model/BOTimer.java,v
retrieving revision 1.19
diff -u -r1.19 BOTimer.java
--- model/BOTimer.java  19 Dec 2004 16:30:44 -0000      1.19
+++ model/BOTimer.java  27 Dec 2004 22:00:23 -0000
@@ -32,7 +32,8 @@
     public String timerNumber;
     private String modifiedId;
     public String eventTypeId;
-    public String eventRepeatId;
+    public String eventRepeatId;
+    public String repeatCount;
     public String announceTime;
     public String senderName;
     public String description;
@@ -69,13 +70,24 @@
         this.eventRepeatId = id;
     }

+    public String getRepeatCount(){
+       return this.repeatCount;
+    }
+
+    public void setRepeatCount(String count){
+       if(repeatCount!=null && !repeatCount.equals(count) ) {
+            this.setModifiedId("modify");
+        }
+        this.repeatCount = count;
+    }
+
     public String getAnnounceTime (){
         return this.announceTime;
     }

     public void setAnnounceTime(String time){
         if(announceTime!=null && !announceTime.equals(time) ) {
-            this.setModifiedId("midify");
+            this.setModifiedId("modify");
         }
         this.announceTime = time;
     }

ciao,

ChakaZulu

[z3r0]

Klasse, Danke!

Im build-script ist eine Windows-Verzeichnisangabe drin, die nach eigenen Bedürfnissen angepasst werden muss. Den Compile-Target hatte ich bisher nicht gebraucht, mit Eclipse funzt das auch so, da man hier schon fertig compilierte Sourcen vorliegen hat

Ein Jar mit Java-Files dürfte es nicht geben

<property name="sourcedir1" value="**\*.class" />

Die anderen 3 Diff´s checke ich gleich ein. Danke nochmals, das waren überhaupt die ersten Patches aus dritter Hand

edit: Ist leider nicht abwärtskompatibel
Werde ich mir morgen ansehen.

[ChakaZulu]

Ein Jar mit Java-Files dürfte es nicht geben

<property name="sourcedir1" value="**\*.class" />

ja, stimmt. da hatte ich was verwechselt (ich hatte die java files im Sourceverzeichnis in Erinnerung, als ich geschaut habe, ob es denn auch class-files gibt).

edit: Ist leider nicht abwärtskompatibel
Werde ich mir morgen ansehen.

Ja, deswegen hab ich ja auch nen Patch machen wollen

Die Rückgabe von control/timer hat sich geändert, nach der dritten Stelle ist ein neuer Wert reingekommen (Begrenzung bei sich wiederholenden Timern).

ciao,

ChakaZulu

[z3r0]

so sollte es wieder tun. Zumindest mit alten Images

Code: Alles auswählen

botimer.eventRepeatId=st.nextToken();
            //Abwärtskompatibilität gewährleisten. repeatCount erst in Images seit 12.2004 vorhanden
            String temp = st.nextToken();
            if (temp.length()>5) {
                botimer.announceTime=temp;
            } else {
                botimer.repeatCount=temp; 
            }

		    valueStart=st.nextToken();

[ChakaZulu]

hi,

mit alten funktionierts dann, aber mit neuen nicht
announceTime wird bei neuen images dann nicht gesetzt...

Code: Alles auswählen

            if (temp.length()>5) {
                botimer.announceTime=temp;
            } else {
                botimer.repeatCount=temp;
               [color=red] botimer.announceTime=st.nextToken();[/color]
            }

          valueStart=st.nextToken();

Allerdings könnte man direkt über den nhttpd
theoretisch auch timer mit 6-stellige Wiederholungen anlegen

ciao,

ChakaZulu