http://bugzilla.novell.com/show_bug.cgi?id=553022

User [email protected] added comment
http://bugzilla.novell.com/show_bug.cgi?id=553022#c1


Marek Habersack <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #1 from Marek Habersack <[email protected]>  2009-11-09 
07:25:29 MST ---
Hey Sean,

  Three notes:

 - when you create a patch, please use the unified diff format. While which
file the patch applies to can be deduced from report text, it might be
ambiguous and also becomes a nuisance if one has to cd to the correct directory
instead of just applying the patch from the top of the source tree.

 - please run tests after you modify source code. Your patch broke 6 tests in
System.Web.Routing assembly

 - it is a good idea to provide a self-contained, small test case with each bug
- it makes it easier, faster and less error prone to reproduce the issue.

Fixed in r145740 (trunk), r145741 (2.6 branch), r145742 (2.4 branch)

-- 
Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to