[lldb-dev] contributing a patch?

[Chuck Ries]

Thank you for the help. Here is the link to my first review: http://reviews.llvm.org/D8210

Please let me know if there are additional steps I need to take.

Thanks!
ChuckR

From: Zachary Turner [mailto:zturner at google.com]
Sent: Monday, March 9, 2015 5:34 PM
To: Chuck Ries; lldb-dev at cs.uiuc.edu
Subject: Re: [lldb-dev] contributing a patch?

Hi Chuck, I'll update the instructions for submitting a patch.  In the meantime, the best way to get the ball rolling is to:
a) Create a Phabricator<http://reviews.llvm.org> account
b) Subscribe to lldb-commits mailing list
c) Upload your patch to Phabricator (click Differential on the left hand pane, and then click "+ Create Diff" in the top right corner, and attach your patch there).
d) When you get to the step where you fill in reviewers, etc please make sure lldb-commits is CC'ed, and include some people as reviewers.  For this particular change, I would suggest adding Jim Ingham for the line in BreakpointLocation.cpp and abidh for the MI changes  (we should probably make a CODE_OWNERS.txt in root lldb so you'll be able to figure this out for yourself).   People will add additional reviewers as necessary.

On Mon, Mar 9, 2015 at 5:21 PM Chuck Ries <Chuck.Ries at microsoft.com<mailto:Chuck.Ries at microsoft.com>> wrote:
Hello all,
The instructions at http://lldb.llvm.org/source.html are very minimal on how to contribute to lldb, and after reading the instructions at http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch, I am unsure what applies and what does not.

At this point I have generated and attached a patch. The changes are primarily to lldb-mi: fixing bugs and filling gaps needed for our use of lldb-mi. How can I start a code review/official submission process?

Thanks,
ChuckR
_______________________________________________
lldb-dev mailing list
lldb-dev at cs.uiuc.edu<mailto:lldb-dev at cs.uiuc.edu>
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20150310/78ff196f/attachment.html>