fehler im patch-parser

[AA_]

im zusammenhang mit meiner svn-erweiterung für die projektverwaltung habe ich schwächen im patch-parser entdeckt.

der parser berücksichtigt warnungen des diff-programmes nicht und erkennt patches mit warnmeldungen nicht als patchdateien. die entwickler haben hier die 3 normalen indikatoren am zeilenanfang berücksichtigt:

case ' ': $line_type = 'context'; break;
case '+': $line_type = 'added'; break;
case '-': $line_type = 'removed'; break; 

es gibt noch einen 4. indikator und zwar '\ '. damit werden warnmeldungen der diff-engine eingeleitet. mir bekannt ist nur die meldung, dass eine datei nicht mit einem EOL endet, also kein zeilenumbruch vorhanden ist. ein solcher patch ist trotz dieser warnungen ein gültiger patch, weil die patch-programme dies berücksichtigen.

dies sollte auch der patch-parser und der patch-viewer beachten!

vorschlag dazu: man könnte einen zusätzlichen container ($line['type']) 'warning' einbauen, dazu dann ein neues template für diesen typ (pt_patchbit_line_warning) mit einer besonderen hervorhebung der zeile(n) und somit die darstellung des patches ermöglichen.

[StGaensler]

Hallo,

solch eine Zeile ist im unified diff-Format nicht vorgesehen: http://www.gnu.org/software/diffutil...iled%20Unified
Das was die Projektverwaltung kann, ist eben das unified-Format. Ich weiß nicht, ob das vielleicht später erweitert wird, würde dir aber raten, diese Warnungen eventuell zu entfernen, oder das Verhalten von vBulletin über dein Add-on entsprechend zu verändern.

Viele Grüße

Stefan

[AA_]

Hallo,

solch eine Zeile ist im unified diff-Format nicht vorgesehen: http://www.gnu.org/software/diffutil...iled%20Unified
Das was die Projektverwaltung kann, ist eben das unified-Format. Ich weiß nicht, ob das vielleicht später erweitert wird, würde dir aber raten, diese Warnungen eventuell zu entfernen, oder das Verhalten von vBulletin über dein Add-on entsprechend zu verändern.

Viele Grüße

Stefan

die dokumentation ist sehr spärlich und natürlich erzeugen einige programme im unified diff format eben jene oben genannte zeile.

"(...) As a special case, unified diff expects to work with files that end in a newline. If either file does not, unified diff will emit the special line
\ No newline at end of file
after the modifications. The patch program should be aware of this. (...)"

mehr dazu hier: http://en.wikipedia.org/wiki/Unified...Unified_format

zu deinem tipp, die warnung zu entfernen: das geht nicht, denn dann stimmt die ausgabe des patch-parsers nicht mehr, wenn die einzige änderung darin bestand, den zeilenumbruch hinzuzufügen, der in der vorherigen version fehlte.

[StGaensler]

Hallo,

ich hab das mal als Bug gemeldet. Kannst du mir mal ein Beispiel von solch einem Patch anhängen?

Viele Grüße

Stefan

[AA_]

klar doch: http://www.vbulletin-germany.com/for...43#attachments

[AA_]

eventl ergänzt du den eintrag noch um die bemerkung, dass die warnmeldung nicht unbedingt in englisch ist. sicher ist - das weiss ich aus einigen diskussionen in diversen newsgroups - dass eine warnmeldung immer mit '\ ' beginnt, der rest ist variabel.

[StGaensler]

Vielen Dank, ich hab die beiden Dateien dort auch noch angehängt.

In welcher Sprache die Meldung ist, ist ja egal, genauso wie es ja auch egal ist, welcher Code in der Datei steht Ich glaube "used for warnings of the diff engine" ist allgemein genug

Viele Grüße

Stefan

[AA_]

ach und weil ich gerade dabei bin... die können gleichmal diesen bug mit beheben (im patch_parser):

        $old_file_bits = explode("\t", $old_file);
        if (sizeof($old_file_bits) == 1)
        {
            $old_file_bits = preg_split('#\s+#', $old_file);
        }
        $new_file_bits = explode("\t", $new_file);
        if (sizeof($new_file_bits) == 1)
        {
            $new_file_bits = preg_split('#\s+#', $new_file);
        }

        // create holder for this file's chunks
        $this->files[] = array(
            'old' => array(
                'filename' => $old_file_bits[0],
                'date' => $old_file_bits[1],
                'revision' => $old_file_bits[2]
            ),
            'new' => array(
                'filename' => $new_file_bits[0],
                'date' => $new_file_bits[1],
                'revision' => $new_file_bits[2]
            ),
            'chunks' => array()
        ); 

$old_file_bits[2] und $new_file_bits[2] werden bei svn-diffs immer warnungen erzeugen und die revision leer sein, denn die zeile lautet so:

--- trunk/test/diff_3.txtTAB2008-10-03 21:39:28 UTCSPACE(rev 4048)

das explode liefert hier also nur 2 array-teile. im 2. ist dann die uhrzeit und die revision enthalten.

lt. den erhältlichen dokumentationen zu diff, ist nur sicher, dass der 2. teil der timestamp ist. alles danach ist optional (zb. eine revisionsangabe). möglicherweise haben die mit cvs-diffs gearbeitet, wo die revisionsnummer einem TAB folgt.

wie auch immer, es funktioniert nicht wie es soll