Heutiges [20120725] Dialy

[Thomas Schade]

Moin,

das heutige Dialy-Build crasht verl�sslich bei Reply und Write.
Putzigerweise immer erst beim zweiten Reply / Write. Umgehen l�sst es
sich (meistens) mittels 'Reply all'. Tritt auch im v�llig unverbastelten
Testprofil auf.

Anybody else?


Ciao
Toscha
--
It's mercy, compassion and forgiveness I lack. Not rationality.
[The Bride]

[Sascha Grage]

On 25.07.12 18:51, Thomas Schade wrote:

> das heutige Dialy-Build crasht verl�sslich bei Reply und Write.
> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen l�sst es
> sich (meistens) mittels 'Reply all'. Tritt auch im v�llig unverbastelten
> Testprofil auf.
>
> Anybody else?

Ja ich! Hab heute mein Retina Macbook bekommen und dort gerade den Daily
f�r News installiert. Hab schon gedacht, der w�re zu dem Teil
inkompatibel...

bye,
Sascha

[Manfred Hamernik]

*Thomas Schade* schrieb:

> Moin,
>
> das heutige Dialy-Build crasht verl�sslich bei Reply und Write.
> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen l�sst es
> sich (meistens) mittels 'Reply all'. Tritt auch im v�llig unverbastelten
> Testprofil auf.
>
> Anybody else?

Jep, kann ich hier best�tigen.

Gru�, Manfred

--
Mein Tagebuch: Heute ist 15.12.2011
Es regnet schon �ber eine Woche.
Meine Frau ist ganz traurig und guckt st�ndig durchs Fenster. Ich
bef�rchte wenn es so weiter regnet dass ich sie herein lassen muss...

[KaiRo - Robert Kaiser]

Thomas Schade schrieb:
> das heutige Dialy-Build crasht verlässlich bei Reply und Write.
> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen lässt es
> sich (meistens) mittels 'Reply all'. Tritt auch im völlig unverbastelten
> Testprofil auf.

https://bugzilla.mozilla.org/show_bug.cgi?id=777063

Robert Kaiser

[Thomas Schade]

On 25.07.2012 22:52, KaiRo - Robert Kaiser wrote:
> Thomas Schade schrieb:

>> das heutige Dialy-Build crasht verlï¿œsslich bei Reply und Write.
>> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen lï¿œsst es
>> sich (meistens) mittels 'Reply all'. Tritt auch im vï¿œllig unverbastelten
>> Testprofil auf.
>
> https://bugzilla.mozilla.org/show_bug.cgi?id=777063

Das klingt nicht so ganz nach 'meinem' Fehler.
- Ich schreibe grundsï¿œtzlich keine html-Mails.
- Der Crash tritt nicht erst beim Senden auf sondern bereits beim Klick
auf Reply / Write.


Ciao
Toscha
--
Many people are alive only because itï¿œs illegal to shoot them.
[Pearls of Scottish wisdom]

[Ruediger Lahl]

*Toscha* schrieb:

> Anybody else?

Add me.
--
bis denne

[Alfred Peters]

Es schrieb einmal Thomas Schade:

> On 25.07.2012 22:52, KaiRo - Robert Kaiser wrote:
>> Thomas Schade schrieb:

>>> das heutige Dialy-Build crasht verlässlich bei Reply und Write.
>>> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen lässt es

AOL - BuildID=20120724173357

Last good: BuildID=20120723172332

>> https://bugzilla.mozilla.org/show_bug.cgi?id=777063
>
> Das klingt nicht so ganz nach 'meinem' Fehler.

> - Ich schreibe grundsätzlich keine html-Mails.

> - Der Crash tritt nicht erst beim Senden auf sondern bereits beim Klick
> auf Reply / Write.

Das kann auch an Mikes - nicht ganz so genauen - Beschreibung liegen.
Zumindest passt dann Joshuas Beschreibung aus Kommentar #3. Ich vermute
mal, er hat das aus dem Stacktrace heraus gelesen.

Und der Patch aus:
<https://bugzilla.mozilla.org/show_bug.cgi?id=775676#c24>
passt in mein Zeitfenster.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120723172332 12565.8

[Alfred Peters]

Es schrieb einmal Alfred Peters:

> Es schrieb einmal Thomas Schade:

>> - Der Crash tritt nicht erst beim Senden auf sondern bereits beim Klick
>> auf Reply / Write.
>
> Das kann auch an Mikes - nicht ganz so genauen - Beschreibung liegen.

Nicht unbedingt. Hier crasht er auch direkt nach dem ersten Absenden einer
HTML-Mail oder eines normalen Replys (wie dieses hier).

Alfred
--
Thunderbird (17.0a1)
BuildID=20120724173357 12565.8

[Thomas Schade]

On 26.07.2012 06:12, Alfred Peters wrote:
> Es schrieb einmal Alfred Peters:
>> Es schrieb einmal Thomas Schade:

>>> - Der Crash tritt nicht erst beim Senden auf sondern bereits beim Klick
>>> auf Reply / Write.
>>
>> Das kann auch an Mikes - nicht ganz so genauen - Beschreibung liegen.
>
> Nicht unbedingt. Hier crasht er auch direkt nach dem ersten Absenden einer
> HTML-Mail oder eines normalen Replys (wie dieses hier).

Sag ich doch ...


Ciao
Toscha
--
So kiss me, I'm shitfaced
I'm soaked, I'm soiled and brown
in the trousers, she kissed me
And I only bought her one round [Dropkick Murphys]

[Ruediger Lahl]

*Alfred Peters* schrieb:

> Nicht unbedingt. Hier crasht er auch direkt nach dem ersten Absenden einer
> HTML-Mail oder eines normalen Replys (wie dieses hier).

...und ab und zu auch nach dem Beenden. In meinem aktuellen Build ist
noch keine Besserung festzustellen.
--
bis denne

[Alfred Peters]

Es schrieb einmal Ruediger Lahl:

Bei mir schon. :-D

Wenn Du nicht selbst baust, müsst Du wohl warten, bis der Patch auf
Comm-Central gelandet ist.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120726200004 12568.1

[Hartmut Figge]

Alfred Peters:

>Es schrieb einmal Ruediger Lahl:

>> In meinem aktuellen Build ist noch keine Besserung festzustellen.
>
>Bei mir schon. :-D

Sag lieber, der Fehler scheint bei Dir nicht aufzutreten. ;)

Hartmut, mit crashigem neuen Build
--
Usenet-ABC-Wiki http://www.usenet-abc.de/wiki/
Von Usern fuer User :-)

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

> Alfred Peters:
>> Es schrieb einmal Ruediger Lahl:
>
>>> In meinem aktuellen Build ist noch keine Besserung festzustellen.
>>
>> Bei mir schon. :-D
>
> Sag lieber, der Fehler scheint bei Dir nicht aufzutreten. ;)

Dialy! siehe Subject. -�h- Ok, auch bei meinem Daily. *d&r*

> Hartmut, mit crashigem neuen Build

<https://bugzilla.mozilla.org/show_bug.cgi?id=777063#c10>
| [...] If the fix includes code not shared between Tb & Sm, then we'll have to
| port the fix to SeaMonkey [...]

Das bedeutet dann wohl: Ja.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120726200004 12570.0

[Thomas Schade]

On 27.07.2012 20:10, Alfred Peters wrote:
> Es schrieb einmal Hartmut Figge:

>> Sag lieber, der Fehler scheint bei Dir nicht aufzutreten. ;)
>
> Dialy! siehe Subject. -�h- Ok, auch bei meinem Daily. *d&r*

Ich war zumindest konsequent, 'Dialy' schrieb ich auch im Body. :)


Ciao
Toscha
--
It's better to burn out, than to fade away.
[Neil Young]

[Karsten Düsterloh]

Thomas Schade aber hob zu reden an und schrieb:

> Ich war zumindest konsequent, 'Dialy' schrieb ich auch im Body. :)

So'nem s��en kleinen Dialy nimmt man Probleme auch gleich viel weniger
�bel. ;-)


Karsten
--
Freiheit stirbt | Fsayannes SF&F-Bibliothek:
Mit Sicherheit | http://fsayanne.tprac.de/

[Thomas Schade]

On 28.07.2012 04:09, Karsten D�sterloh wrote:
> Thomas Schade aber hob zu reden an und schrieb:

>> Ich war zumindest konsequent, 'Dialy' schrieb ich auch im Body. :)
>
> So'nem s��en kleinen Dialy nimmt man Probleme auch gleich viel weniger
> �bel. ;-)

Wobei das Kleine jetzt dann langsam mal wieder vern�nftig werden d�rfte. :)


Ciao
Toscha
--
The problem with troubleshooting is that trouble shoots back.

[Hartmut Figge]

Thomas Schade:

>Wobei das Kleine jetzt dann langsam mal wieder vernünftig werden dürfte. :)

Solange mein SM crasht... so langsam wird es lästig. Das verantwortliche
Checkin sowie die Flicken von Josuah rauswerfen? Warten? Zu einem nicht
crashenden SM zurückgehen? Nachschauen, ob sonst etwas getan werden
kann? Hm.

Hartmut

[Thomas Schade]

On 29.07.2012 07:44, Hartmut Figge wrote:
> Thomas Schade:

>>Wobei das Kleine jetzt dann langsam mal wieder vern�nftig werden d�rfte. :)
>
> Solange mein SM crasht... so langsam wird es l�stig. Das verantwortliche

> Checkin sowie die Flicken von Josuah rauswerfen? Warten? Zu einem nicht

> crashenden SM zur�ckgehen? Nachschauen, ob sonst etwas getan werden
> kann? Hm.

Mein Fallback in solchen F�llen ist dann der problemlos-langweilig
laufende TB 13. Aber f�r den ist es noch nicht arg genug. :)


Ciao
Toscha
--
Wimps and posers leave the hall!
[Manowar]

[Alfred Peters]

Es schrieb einmal Thomas Schade:

> On 29.07.2012 07:44, Hartmut Figge wrote:
>> Thomas Schade:
>

>>> Wobei das Kleine jetzt dann langsam mal wieder vernünftig werden dürfte. :)
>>
>> Solange mein SM crasht... so langsam wird es lästig. Das verantwortliche

>> Checkin sowie die Flicken von Josuah rauswerfen? Warten? Zu einem nicht

>> crashenden SM zurückgehen? Nachschauen, ob sonst etwas getan werden
>> kann? Hm.
>
> Mein Fallback in solchen Fällen ist dann der problemlos-langweilig
> laufende TB 13. Aber für den ist es noch nicht arg genug. :)

WIMRE ist der Dailiy nicht abgestürzt, wenn ich vor dem ersten Abschicken
erst mehrere Antworten in jeweils einem eigenen Compose-Fenster
geschrieben habe.
Sollte er dann aber doch abstürzen,... Schließlich benutzen nur
Warmduscher die Speicherfunktion.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120728084425 12574.7

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

> Thomas Schade:
>> Wobei das Kleine jetzt dann langsam mal wieder vernünftig werden dürfte. :)
>
> Solange mein SM crasht... so langsam wird es lästig. Das verantwortliche
> Checkin sowie die Flicken von Josuah rauswerfen? Warten? Zu einem nicht
> crashenden SM zurückgehen? Nachschauen, ob sonst etwas getan werden
> kann? Hm.

Ich wüsste da noch was:

O Einen SM-Bug anlegen
O Im Bug 777063 erwähnen, dass der Patch beim SM nicht hilft
O Dito - aber in mozilla.dev.apps.thunderbird
"Should we stop recycling the compose window?"
<WMudnYteXPmAwo3N...@mozilla.org> ff.

[Hartmut Figge]

Alfred Peters:

>O Einen SM-Bug anlegen

Tony hat das bereits in Comment 10 erwähnt, sogar unter Angabe des zu
wiederbelebenden Bugs.

>O Im Bug 777063 erwähnen, dass der Patch beim SM nicht hilft

Och, dass er nicht hilft, sollte kaum übersehbar sein. *g*

>O Dito - aber in mozilla.dev.apps.thunderbird
> "Should we stop recycling the compose window?"
> <WMudnYteXPmAwo3N...@mozilla.org> ff.

Hm. Ich weiß, ich sollte mich eigentlich ein wenig mit TB beschäftigen.
Und mit FF. Eigentlich.

Beim Absenden wird mein SM wieder abstürzen, aber da war noch eine
Verlockung in Deinem anderen Reply, der ich nicht widerstehen konnte.

| WIMRE ist der Dailiy nicht abgestürzt,

^^^^^^
Weiteren Variationen sehe ich gespannt entgegen. :)

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

> Thomas Schade:
>
>> Wobei das Kleine jetzt dann langsam mal wieder vern�nftig werden d�rfte. :)
>
> Solange mein SM crasht... so langsam wird es l�stig. Das verantwortliche

> Checkin sowie die Flicken von Josuah rauswerfen?

Hu? Welche Flicken meinst Du? Den Patch hast Du doch selbst lokal
eingecheckt - oder nicht?

[Hartmut Figge]

Alfred Peters:

>Es schrieb einmal Hartmut Figge:

>> Solange mein SM crasht... so langsam wird es lästig. Das verantwortliche

>> Checkin sowie die Flicken von Josuah rauswerfen?
>
>Hu?

Verantwortlich für das Desaster scheint dieses Checkin zu sein:
http://hg.mozilla.org/mozilla-central/rev/4e95ccf43b6e

Ein Checkin mit derart gravierenden Nebenwirkungen, warum es nicht
umgehend entfernt wurde, nachdem diese entdeckt wurden, nun ja.

> Welche Flicken meinst Du?

Gegenwärtig lässt es sich obiges Checkin nicht mehr so ohne weiteres
entfernen, vermutlich wegen eines oder beider Flicken von Joshua in
https://bugzilla.mozilla.org/show_bug.cgi?id=777063

>Den Patch hast Du doch selbst lokal eingecheckt - oder nicht?

Nein. Du meinst den zweiten von Joshua? Zumindest der erste scheint ja
schon eingecheckt zu sein.

Hartmut, gleich becrasht werdend

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

> Alfred Peters:
>
>> O Einen SM-Bug anlegen
>

> Tony hat das bereits in Comment 10 erw�hnt, sogar unter Angabe des zu
> wiederbelebenden Bugs.

Trotzdem k�nnte es nicht schaden, es nochmal zu tun. }:-)

>> O Dito - aber in mozilla.dev.apps.thunderbird
>> "Should we stop recycling the compose window?"
>> <WMudnYteXPmAwo3N...@mozilla.org> ff.
>

> Hm. Ich wei�, ich sollte mich eigentlich ein wenig mit TB besch�ftigen.
> Und mit FF. Eigentlich.

*Das* ist nat�rlich auch noch eine Option.

> Beim Absenden wird mein SM wieder abst�rzen, aber da war noch eine

> Verlockung in Deinem anderen Reply, der ich nicht widerstehen konnte.
>

>| WIMRE ist der Dailiy nicht abgest�rzt,

> ^^^^^^
> Weiteren Variationen sehe ich gespannt entgegen. :)

Pfft - ein Spellflame. Dabei hatte ich mir grade einen Kommentar �ber
deinen /Josuah/ verkniffen. O:-)

Alfred
--
Thunderbird (17.0a1)
BuildID=20120728084425 12574.8

[Alfred Peters]

Es schrieb einmal Hartmut Figge:
> Alfred Peters:
>> Es schrieb einmal Hartmut Figge:

>> Den Patch hast Du doch selbst lokal eingecheckt - oder nicht?
>
> Nein.

O-K! Dann d�rfte auch klar sein, warum der Patch bei Dir nicht wirkt. ;-)

> Du meinst den zweiten von Joshua?

N�, den habe ich auch nicht drin.

> Zumindest der erste scheint ja
> schon eingecheckt zu sein.

Das m�sste ich verpasst haben. In Comment 7 erw�hnt Joshua nur den checkin
im Thunderbird-Try Tree.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120728084425 12574.8

[Alfred Peters]

Es schrieb einmal Alfred Peters:

> Es schrieb einmal Hartmut Figge:

>> Zumindest der erste scheint ja
>> schon eingecheckt zu sein.
>
> Das m�sste ich verpasst haben. In Comment 7 erw�hnt Joshua nur den checkin
> im Thunderbird-Try Tree.

N�, er ist definitiv noch nicht drin:

| $ patch -u -i ../fix-compose-crash.patch -r ../../reject_patX.lst -N -p 1 --verbose
| Hmm... Looks like a unified diff to me...
| The text leading up to this was:
| --------------------------
||# HG changeset patch
||# Parent 3e83ef9a5b9bfa1306711e54f45e6ac0da39a9b6
||
|| diff --git a/mailnews/compose/src/nsMsgComposeService.cpp b/mailnews/compose/src/nsMsgComposeService.cpp
|| --- a/mailnews/compose/src/nsMsgComposeService.cpp
|| +++ b/mailnews/compose/src/nsMsgComposeService.cpp
| --------------------------
| Patching file mailnews/compose/src/nsMsgComposeService.cpp using Plan A...
| Hunk #1 succeeded at 205.
| Hunk #2 succeeded at 265.
[...]
| done

HTH

[Hartmut Figge]

Alfred Peters:
>Es schrieb einmal Hartmut Figge:
>> Alfred Peters:
>>> Es schrieb einmal Hartmut Figge:

>>> Den Patch hast Du doch selbst lokal eingecheckt - oder nicht?
>>
>> Nein.
>

>O-K! Dann dürfte auch klar sein, warum der Patch bei Dir nicht wirkt. ;-)

Grund für meine Vermutung, er sei schon eingecheckt, war, dass sich das
böswillige Checkin nicht mehr entfernen ließ. Es gab also schon wieder
Änderungen in dem Bereich.

>> Du meinst den zweiten von Joshua?
>

>Nö, den habe ich auch nicht drin.

Warum nicht? Zu anstrengend? ;)

>> Zumindest der erste scheint ja
>> schon eingecheckt zu sein.
>

>Das müsste ich verpasst haben. In Comment 7 erwähnt Joshua nur den checkin
>im Thunderbird-Try Tree.

Bei TB kenne ich mich überhaupt nicht aus. Na gut, dann werde ich mal
patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber
das ging nicht. Einzeln sind beide brav aber der zweite will nicht, wenn
der erste angewandt wird.

Hartmut

[Hartmut Figge]

Hartmut Figge:

>Na gut, dann werde ich mal patchen.

Oh. Kein crash de de.test. Angenehm. :)

>Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber das
>ging nicht. Einzeln sind beide brav aber der zweite will nicht, wenn
>der erste angewandt wird.

Statt des ersten Patches den zweiten nehmen? Bin ich neugierig genug? Ja.

[Hartmut Figge]

Hartmut Figge:

>Statt des ersten Patches den zweiten nehmen?

Auch dies ergibt einen crashfreien SM. Nochmal zur Wirkung von ccache:

hafi@i5_64 ~ $ time build-affe
[...]
real 7m32.374s
user 5m7.055s
sys 0m55.587s

[Alfred Peters]

Es schrieb einmal Hartmut Figge:
> Alfred Peters:
>> Es schrieb einmal Hartmut Figge:
>>> Alfred Peters:
>>>> Es schrieb einmal Hartmut Figge:

>>> Du meinst den zweiten von Joshua?
>>
>> Nö, den habe ich auch nicht drin.
>
> Warum nicht? Zu anstrengend? ;)

| The extra change to mozmill tests is necessary to get them to go green,

Klingt uninteressant.

>> Das müsste ich verpasst haben. In Comment 7 erwähnt Joshua nur den checkin
>> im Thunderbird-Try Tree.
>
> Bei TB kenne ich mich überhaupt nicht aus. Na gut, dann werde ich mal
> patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber
> das ging nicht. Einzeln sind beide brav aber der zweite will nicht, wenn
> der erste angewandt wird.

Ja, der zweite scheint komplett zu sein. Sollte der erste dann nicht für
obsolet erklärt werden?

Alfred
--
Thunderbird (17.0a1)
BuildID=20120728084425 12574.9

[Hartmut Figge]

Alfred Peters:
>Es schrieb einmal Hartmut Figge:

]Zweiter Patch]

>| The extra change to mozmill tests is necessary to get them to go green,
>
>Klingt uninteressant.

Nun ja, da gibt es ja mehr au�er 'extra change'. ;)

>> Bei TB kenne ich mich �berhaupt nicht aus. Na gut, dann werde ich mal

>> patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber

>> das ging nicht. Einzeln sind beide brav ...

Das bedeutete kein Build, sondern nur einen Test mittels '--dry-run'.

>Ja, der zweite scheint komplett zu sein.

Im Augenblick habe ich nur den zweiten. Mein SM crasht nicht und ich
bemerke keine b�sen Nebenwirkungen.

>Sollte der erste dann nicht f�r obsolet erkl�rt werden?

Ein Kommentar k�nnte hilfreich sein. Wegen 'Product: Thunderbird' sollte
dieser von jemandem kommen, der den TB benutzt. Zum Beispiel von Dir. O:-)

[Alfred Peters]

Es schrieb einmal Hartmut Figge:
> Alfred Peters:
>> Es schrieb einmal Hartmut Figge:

>>> Bei TB kenne ich mich �berhaupt nicht aus. Na gut, dann werde ich mal
>>> patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber
>>> das ging nicht. Einzeln sind beide brav ...
>
> Das bedeutete kein Build, sondern nur einen Test mittels '--dry-run'.

Das glaube ich nicht - Tim.

<https://tbpl.mozilla.org/?tree=Thunderbird-Try>
<https://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/Pidg...@gmail.com-eba790efa638/>

>> Ja, der zweite scheint komplett zu sein.
>
> Im Augenblick habe ich nur den zweiten. Mein SM crasht nicht und ich
> bemerke keine b�sen Nebenwirkungen.

Na gut, dann werde ich den auch mal probieren.

>> Sollte der erste dann nicht f�r obsolet erkl�rt werden?
>
> Ein Kommentar k�nnte hilfreich sein. Wegen 'Product: Thunderbird' sollte
> dieser von jemandem kommen, der den TB benutzt.

Ausrede. Au�erdem k�nntest Du damit gleich Comment 10 widersprechen.

> Zum Beispiel von Dir. O:-)

Faultier! ;-)

Alfred
--
Thunderbird (17.0a1)
BuildID=20120729093026 12575.0

[Hartmut Figge]

Alfred Peters:
>Es schrieb einmal Hartmut Figge:
>> Alfred Peters:
>>> Es schrieb einmal Hartmut Figge:

>>>> Bei TB kenne ich mich �berhaupt nicht aus. Na gut, dann werde ich mal
>>>> patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber
>>>> das ging nicht. Einzeln sind beide brav ...
>>
>> Das bedeutete kein Build, sondern nur einen Test mittels '--dry-run'.
>
>Das glaube ich nicht - Tim.

�h? Ich hatte beide Patche getestet mittels 'patch --dry-run -p1 < tst'
angewandt auf den Source vom Trunk-SM. Beide erfolgreich. Einzeln. Was
davon kannst Du nicht glauben?

><https://tbpl.mozilla.org/?tree=Thunderbird-Try>

Sieht kompliziert aus. Unverst�ndlich ohne weitere Nachforschung.

><https://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/Pidg...@gmail.com-eba790efa638/>

Dies try-builds sind also spezielle TB-Builds f�r experimentelle Zwecke.
Sieht jedenfalls danach aus.

[Hartmut Figge]

Hartmut Figge:

>Ich hatte beide Patche getestet mittels 'patch --dry-run -p1 < tst'
>angewandt auf den Source vom Trunk-SM. Beide erfolgreich. Einzeln.

Da das entsprechende xterm noch offen ist, kann ich ja mal kopieren, was
genau ich gemacht hatte:

hafi@i5_64 ~/hg-moz/src $ patch --dry-run -p1 <
~/ztmp/attachment.cgi\?id\=645745
patching file mailnews/compose/src/nsMsgComposeService.cpp
patching file mailnews/compose/src/nsMsgComposeService.h
hafi@i5_64 ~/hg-moz/src $ patch --dry-run -p1 <
~/ztmp/attachment.cgi\?id\=646224
patching file mail/test/mozmill/composition/test-forward-headers.js
patching file mailnews/compose/src/nsMsgComposeService.cpp
patching file mailnews/compose/src/nsMsgComposeService.h
patching file mailnews/mailnews.js
hafi@i5_64 ~/hg-moz/src $

[Jens Hatlak]

Alfred Peters schrieb:

> Es schrieb einmal Hartmut Figge:

>> Bei TB kenne ich mich überhaupt nicht aus. Na gut, dann werde ich mal
>> patchen. Im ersten Anlauf wollte ich gleich beide Joshuas nehmen, aber
>> das ging nicht. Einzeln sind beide brav aber der zweite will nicht, wenn
>> der erste angewandt wird.
>
> Ja, der zweite scheint komplett zu sein. Sollte der erste dann nicht für
> obsolet erklärt werden?

Soweit ich das sehen kann, handelt es sich bei den Patches um zwei
alternative Herangehensweisen (a: fix caching, b: stop caching).
Insofern wundert es nicht, dass nur einer von beiden zugleich angewandt
werden kann und noch keiner als obsolet markiert wurde: Bis sich
Patchautor und Reviewer geeinigt haben, sind beide gültige Vorschläge.
Die endgültige Fassung kann dann nochmal komplett anders aussehen, aber
so oder so ist es Joshuas Entscheidung, ob oder wann er was als obsolet
markiert.

HTH

Jens

--
Jens Hatlak <http://jens.hatlak.de/>
SeaMonkey Trunk Tracker <http://smtt.blogspot.com/>

[Jens Hatlak]

Hartmut Figge schrieb:

> Dies try-builds sind also spezielle TB-Builds f�r experimentelle Zwecke.
> Sieht jedenfalls danach aus.

Try-Builds sind automatisch erstellte Builds, f�r die ebenso automatisch
Tests durchlaufen werden - quasi so wie Nightly Builds, nur dass
derjenige, der so ein Try Build anst��t, selbst bestimmen kann, welche
Patches in das Build mit einflie�en sollen und f�r welche Plattformen
Builds erstellt werden sollen.

SM hat (bislang) keine eigene Try-Infrastruktur, aber da wir im selben
Repository (comm-central) leben wie TB, kann jeder mit Checkin-Rechten
auch SM-Try-Builds �ber die TB-Try-Infrastruktur ansto�en. Das haben wir
z.B. damals in Vorbereitung f�r SM Sync gemacht.

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

>> Es schrieb einmal Hartmut Figge:

>>>> Es schrieb einmal Hartmut Figge:

>>>>> Alfred Peters:

>>>>>> [Thunderbird-Try Tree]

>>>>> Bei TB kenne ich mich �berhaupt nicht aus. [...]

>>> Das bedeutete kein Build, sondern nur einen Test mittels '--dry-run'.

> �h? Ich hatte beide Patche getestet mittels 'patch --dry-run -p1 < tst'

Sorry - Miss-Verst�ndnis meinerseits.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120729161356 12575.8

[Alfred Peters]

Es schrieb einmal Jens Hatlak:
> Alfred Peters schrieb:

>> Ja, der zweite scheint komplett zu sein. Sollte der erste dann nicht f�r
>> obsolet erkl�rt werden?

>
> Soweit ich das sehen kann, handelt es sich bei den Patches um zwei
> alternative Herangehensweisen (a: fix caching, b: stop caching). Insofern

Das �ndert nat�rlich die Sachlage. Wie schon geschrieben, habe ich nur den
Satz �ber die 'mozmill tests' gelesen und dem zweiten Patch deshalb gar
keine weitere Beachtung geschenkt.

Es funktionieren beide Varianten. Ich kann allerdings subjektiv keinen
Geschwindigkeitsunterschied feststellen. Ich habe extra noch mal ein
Clobber-Build erzeugt, um sicher zu gehen, dass auch die andere
Patch-Version benutzt wurde.

[Sascha Grage]

On 25.07.12 18:51, Thomas Schade wrote:
> das heutige Dialy-Build crasht verl�sslich bei Reply und Write.
> Putzigerweise immer erst beim zweiten Reply / Write. Umgehen l�sst es
> sich (meistens) mittels 'Reply all'. Tritt auch im v�llig unverbastelten
> Testprofil auf.

Scheint heute nicht mehr zu crashen... ;-)

Bye,
Sasch

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/17.0
Thunderbird/17.0a1 ID:20120731031044

[Hartmut Figge]

Sascha Grage:

>Scheint heute nicht mehr zu crashen... ;-)

Dann sollte eigentlich mein n�chstes Build fehlschlagen, weil der Patch
nicht mehr angewendet werden kann. Hat Zeit bis Morgen.

[Alfred Peters]

Es schrieb einmal Hartmut Figge:

> Sascha Grage:
>> Scheint heute nicht mehr zu crashen... ;-)
>
> Dann sollte eigentlich mein n�chstes Build fehlschlagen, weil der Patch
> nicht mehr angewendet werden kann.

Vermutlich. Hier ist auch grade das Diff-Tool aufgepopt.

Alfred
--
Thunderbird (17.0a1)
BuildID=20120729161356 12581.2

[Ruediger Lahl]

*Sascha Grage* schrieb:

> Scheint heute nicht mehr zu crashen... ;-)

Ja. Das ging mir schon m�chtig auf den Zeiger.
--
bis denne