student/mudrama1/index.mdwn

   1 [[!meta title="Mudra Martin"]]
   2
   3 **Název projektu:** AnkhSVN
   4
   5 **Domovská stránka projektu:** <http://ankhsvn.open.collab.net/>
   6
   7 Zadání
   8 ======
   9
  10 AnkhSVN je integrace SVN do visual studia. Jeji jedina konkurence je komercni a placeny visualSVN plugin.
  11 Bohuzel tortoise neni prilis stabilni a proto bych chtel celkove zvysit stabilitu tohoto doplnku tak, aby nebyli vyvojari nuceni platit za visualSVN. Ankh svn ma mnoho otevrenych issues tykajicich se stability a také má mnoho issues, které nejsou nahlášeny.
  12
  13 Odkaz na issue Tracker:
  14 http://ankhsvn.open.collab.net/issues
  15
  16 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.
  17
  18 <img src='http://support.dce.felk.cvut.cz/osp/student/mudrama1/AnkhSVN_bug.png' width=800/>
  19
  20 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.
  21
  22 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.
  23
  24 Issues jsou založeny na adrese:
  25 http://ankhsvn.open.collab.net/issues/show_bug.cgi?id=708
  26 http://ankhsvn.open.collab.net/issues/show_bug.cgi?id=707
  27
  28 Celou opravu jsem poslal podle pravidel popsaných na adrese:
  29
  30 http://ankhsvn.open.collab.net/servlets/ProjectProcess?documentContainer=c5__How%20to%20submit%20a%20patch
  31
  32
  33
  34 Odkazy dokládající výsledky mé práce
  35 ======
  36
  37 -----
  38 Odkazy na mou komunikaci s vývojáři v on-line archivech veřejných diskuzí
  39 =======
  40
  41 **mudrama1@fel.cvut.cz**
  42
  43 12.4.
  44 komu: dev
  45 [[[
  46 Fix issue #708/707: Dont crash when try to perform uniffiedDiff, Annotation and Lock operations on added and uncommited
  47 ]]]
  48
  49 ---
  50
  51 Best regards Martin Mudra
  52
  53 PatchForUncommitedFilesDiffAnotationLockOperations.patch 9 kB   Stáhnout
  54
  55
  56 **Bert Huijben bert@qqmail.nl**
  57
  58 13.4.
  59 Hi Martin,
  60
  61 I committed slightly tweaked portions of your patch in r11156, r11157 and
  62 r11158.
  63
  64 Somehow your patch was broken by the mailing list software. (There are blank
  65 lines between all the useful lines).
  66
  67 Annotate:
  68 I used an SvnItem.HasCopyableHistory check instead of added, to allow the
  69 command to keep working on copied files. (The root of a copy is also
  70 .IsAdded)
  71 I applied the check in the update and execute handler to just disable the
  72 command where it doesn't work instead of showing an error dialog.
  73 (Another bug fixed was that it checked for .IsVersionable when fetching a
  74 targets list)
  75 I also fixed a TODO: Save the file if annotating against the working
  76 revision.
  77
  78 Unified Diff:
  79 Unified diff does work on added files, but it didn't work on files that had
  80 the change status 'New'. I fixed this by applying the 'New' status directly
  81 here.
  82 To avoid further errors I disabled the exceptions from the diff handler and
  83 write the error to the patch file instead.
  84
  85 Lock:
  86 Same as annotate: I applied the fix to the selection filter instead of to
  87 the dialog processing to avoid showing UI that doesn't work instead of
  88 showing errors.
  89 (Also: fixed a missing resource reference)
  90
  91 Thanks for your patch!,
  92        Bert
  93
  94
  95 -----
  96
  97 **scrumtraining@collab.net**
  98
  99 Certified ScrumMaster Training coming to Zurich, Switzerland
 100 For a Limited Time: One Third Off Enrollment!
 101 Hi Martin,
 102
 103 Do you want to be an Agile genius? Learn from the experts - CollabNet. CollabNet has more Certified Scrum trainers on staff than anyone, so you know you’ll be on the path to success.
 104
 105 Our Certified ScrumMaster training is perfect for all your team members – ScrumMasters, Product Owners, project managers, developers, testers, and architects will all benefit from the practical, hands-on exercises and small group discussions. You will leave with a thorough understanding of Agile and a clear plan to put it to work to optimize your team’s performance!
 106
 107 To encourage Agile adoption across your organization, CollabNet is offering 33% off enrollment. Forward this invitation to anyone who wants to deliver quality projects and products faster.
 108
 109
 110
 111 Our next Certified ScrumMaster Training will be held in Zurich 24-25 May. This is the only CollabNet Certified ScrumMaster training in Zurich until winter of this year, so don’t miss out! Click here to registerr.
 112
 113 In addition to meeting the Scrum Alliance certification requirements, this CSM course provides 14 PMI PDUs and a 2 year membership to the Scrum Alliance.
 114
 115 If you have any questions about this class, or about private consulting or coaching, please contact us at scrumtraining@collab.net.
 116
 117
 118
 119 Best Regards,
 120
 121 The CollabNet Team
 122 scrumtraining@collab.net
 123
 124 ----
 125 [Odkaz na repozitář](http://ankhsvn.open.collab.net/svn/ankhsvn/trunk)
 126 =======
 127
 128 ---
 129 Odkazy na mé commity v repozitáři projektu
 130 ========
 131 použitá subversion nepodporuje přímé odkazy na revize.
 132 Commitnuté revize jsou:
 133
 134 **r11158:**
 135
 136 Disable the lock command on new, replaced and copied files.
 137
 138 **r11157:**
 139
 140 Add status 'New' files from the unified diff handler to allow showing them in the diff result. Instead of failing on errors add a comment line to the diff that shows what error occured.
 141
 142 **r11156:**
 143
 144 Don't try to annotate new files and save documents before annotating to the working revision.
 145
 146 * [Další relevantní odkazy dokládající mou práci...](http://lwn.net/Articles/385586/)
 147
 148 Dokumentace
 149 ========
 150 Jedná se o stability issues programu, dokumentace oprav je přítomná v issuetrackeru a samotné opravy v kódu.
 151 Ukázky:
 152
 153 1
 154
 155     ...added)//For this command server must know object, when it is not commited it doesnt make sense to use this...
 156
 157 2
 158
 159     ...
 160     /// <summary>
 161     ///   Looks up a localized string similar to Item &apos;{0}&apos; is not commited. Perform commit operation on this file first..
 162     /// </summary>
 163     internal static string TargetNotAdded {
 164     get {return ResourceManager.GetString("TargetNotAdded", resourceCulture);}}
 165     ...
 166
 167
 168
 169
 170
 171 Prezentace
 172 ==========
 173
 174 * [[Prezentace záměru práce|AnkhSVN.pdf]]
 175 * [[Prezentace řešení|AnkhSVNSolution.pdf]]
 176
 177 Ohloh
 178 =====
 179 <a href='http://www.ohloh.net/accounts/167396?ref=Detailed'>
 180 <img alt='Ohloh profile for LightCZ' height='35' src='http://www.ohloh.net/accounts/167396/widgets/account_detailed.gif' width='191' />
 181 </a>