]> rtime.felk.cvut.cz Git - edu/osp-wiki.git/commitdiff
(no commit message)
authormudrama1 <mudrama1@web>
Wed, 16 May 2012 23:04:18 +0000 (01:04 +0200)
committerOSP robot <osp@rtime.felk.cvut.cz>
Wed, 16 May 2012 23:04:18 +0000 (01:04 +0200)
student/mudrama1/index.mdwn

index edb83fb7e77c3752ea463b1e744d7bb54cb67695..10d35027c0cdd28db5f1e3ae098f2faef3d874e0 100644 (file)
@@ -15,6 +15,8 @@ http://ankhsvn.open.collab.net/issues
 
 Založil jsem stability issue, který jsem našel (#708). Jedná se například o velice častou situaci, kdy je přidán nějaký soubor do verzovacího systému, ale změna ještě není commitnutá. Server tedy ví o souboru, že již existuje ale ještě nezná jeho obsah. V tuto chvíli když na tomto souboru někdo zavolá nějakou serverovou operaci AnkhSVN vyhodí vyjímku, která je ošetřena až nějakým global mechanismem a následně se OBČAS chová nějak podivně (nefungující tlačítka atd.). Tato situace nastává ve chvílích, kdy například zavoláme diff nad celou solution, nebo v případě mylného výběru tohoto souboru při výběru více položek.
 
+<img src='http://support.dce.felk.cvut.cz/osp/student/mudrama1/AnkhSVN_bug.png' width=800/>
+
 AnkhSVN používá pro komunikaci s SVN knihovnu SharpSVN, která není součástí tohoto open source projektu. Tato chyba nastává už v této používané knihovně, je však možné tuto chybu ošetřit i na straně Ankh, kdy se zkontroluje před zavoláním těchto serverových operací, zda se nejedná o soubor, který byl přidán ale necommitnut.
 
 Další řešenou issue je issue číslo #707, které se týká nemožnosti psát zprávu přes dialogové okno. Tato issue se vyskytuje velice nepravidelně. Tato chyba se vyskytla a já jsem se jí pokusil nějak opravit. Bohužel řešení této chyby neuznali, protože celý dialogový systém se aktuálně předělává kvůli VS 2011. Pro opravení této chyby stačilo vždy při konstruování tohoto okna zavolat focus do okna se zprávou. Přesný důvod této chyby se mi však nepodařilo vypátrat. Tato chyba se v issues vyskytovala velice často a #707 je posledním výskytem této chyby.
@@ -24,6 +26,7 @@ http://ankhsvn.open.collab.net/issues/show_bug.cgi?id=708
 http://ankhsvn.open.collab.net/issues/show_bug.cgi?id=707
 
 Celou opravu jsem poslal podle pravidel popsaných na adrese:
+
 http://ankhsvn.open.collab.net/servlets/ProjectProcess?documentContainer=c5__How%20to%20submit%20a%20patch
 
 
@@ -31,11 +34,64 @@ http://ankhsvn.open.collab.net/servlets/ProjectProcess?documentContainer=c5__How
 Odkazy dokládající výsledky mé práce
 ======
 
-Zde přidám odkazy podobně jako v příkladu níže a popíšu na co
-jednotlivé odkazy odkazují.
+-----
+Odkazy na mou komunikaci s vývojáři v on-line archivech veřejných diskuzí
+=======
+
+mudrama1@fel.cvut.cz
+12.4.
+komu: dev 
+[[[
+Fix issue #708/707: Dont crash when try to perform uniffiedDiff, Annotation and Lock operations on added and uncommited
+]]]
+
+
+
+Best regards Martin Mudra
+
+PatchForUncommitedFilesDiffAnotationLockOperations.patch 9 kB   Stáhnout  
+
+13.4.
+Bert Huijben bert@qqmail.nl
+
+Hi Martin,
+
+I committed slightly tweaked portions of your patch in r11156, r11157 and
+r11158.
+
+Somehow your patch was broken by the mailing list software. (There are blank
+lines between all the useful lines).
+
+Annotate:
+I used an SvnItem.HasCopyableHistory check instead of added, to allow the
+command to keep working on copied files. (The root of a copy is also
+.IsAdded)
+I applied the check in the update and execute handler to just disable the
+command where it doesn't work instead of showing an error dialog.
+(Another bug fixed was that it checked for .IsVersionable when fetching a
+targets list)
+I also fixed a TODO: Save the file if annotating against the working
+revision.
+
+Unified Diff:
+Unified diff does work on added files, but it didn't work on files that had
+the change status 'New'. I fixed this by applying the 'New' status directly
+here.
+To avoid further errors I disabled the exceptions from the diff handler and
+write the error to the patch file instead.
+
+Lock:
+Same as annotate: I applied the fix to the selection filter instead of to
+the dialog processing to avoid showing UI that doesn't work instead of
+showing errors.
+(Also: fixed a missing resource reference)
+
+Thanks for your patch!,
+       Bert
 
-* [Odkazy na mou komunikaci s vývojáři v on-line archivech veřejných diskuzí...](http://groups.google.com/group/comp.os.minix/browse_thread/thread/e3df794a2bce97da/2194d253268b0a1b?#2194d253268b0a1b)
-* [Odkaz na můj repozitář](http://ankhsvn.open.collab.net/svn/ankhsvn/trunk)
+----
+[Odkaz na repozitář](http://ankhsvn.open.collab.net/svn/ankhsvn/trunk)
+=======
 
 ---
 Odkazy na mé commity v repozitáři projektu