<p>Michael Bacarella has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30293">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Documentation/lessons/lesson2.md: clarify running make gitconfig<br><br>It's easy to misinterpret or miss altogether the instruction to<br>run 'make gitconfig', which will cause strange problems a few<br>commands later.  Revise the documentation to make it clearer.<br>Also adds a blurb further down with a link to find Gerrit<br>workflow docs.<br><br>Change-Id: I49734c724c4d6da716a358cd849938ef14dab3b1<br>Signed-off-by: Michael Bacarella <michael.bacarella@gmail.com><br><br>detached from FETCH_HEAD<br><br>Change-Id: I7ce93f1554a9e289d65e96f0b6d4d61292ded1f4<br>---<br>M Documentation/lessons/lesson2.md<br>1 file changed, 13 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/30293/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/Documentation/lessons/lesson2.md b/Documentation/lessons/lesson2.md</span><br><span>index a95dd80..c354bfd 100644</span><br><span>--- a/Documentation/lessons/lesson2.md</span><br><span>+++ b/Documentation/lessons/lesson2.md</span><br><span>@@ -71,14 +71,18 @@</span><br><span> If you are using HTTP, instead, select **http** from the tabs under "Project coreboot"</span><br><span> and run the command that appears</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-After it finishes cloning, "cd coreboot" will take you into the local</span><br><span style="color: hsl(0, 100%, 40%);">-git repository. Run "make gitconfig" to set up the hooks and configurations.</span><br><span style="color: hsl(0, 100%, 40%);">-For example, you will be asked to run the following commands to set your</span><br><span style="color: hsl(0, 100%, 40%);">-username and email.</span><br><span style="color: hsl(120, 100%, 40%);">+Now is a good time to configure your global git identity, if you haven't</span><br><span style="color: hsl(120, 100%, 40%);">+already.</span><br><span> </span><br><span>      git config --global user.name "Your Name"</span><br><span>  git config --global user.email "Your Email"</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+Finally, enter the local git repository and set up repository specific hooks</span><br><span style="color: hsl(120, 100%, 40%);">+and other configurations.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+  cd coreboot</span><br><span style="color: hsl(120, 100%, 40%);">+   make gitconfig</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> ## Part 4: Submit a commit</span><br><span> </span><br><span> An easy first commit to make is fixing existing checkpatch errors and warnings</span><br><span>@@ -149,6 +153,11 @@</span><br><span> your commit will be on coreboot.org, but is only visible to those you add</span><br><span> as reviewers.</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+This has been a quick primer on how to submit a change to Gerrit for review</span><br><span style="color: hsl(120, 100%, 40%);">+using git.  You may wish to review the [Gerrit code review workflow</span><br><span style="color: hsl(120, 100%, 40%);">+documentation](https://gerrit-review.googlesource.com/Documentation/intro-user.html#code-review),</span><br><span style="color: hsl(120, 100%, 40%);">+especially if you plan to work on multiple changes at the same time.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> ## Part 4b: Using git cola to stage and submit a commit</span><br><span> </span><br><span> If git cola is not installed on your machine, see</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30293">change 30293</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30293"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I7ce93f1554a9e289d65e96f0b6d4d61292ded1f4 </div>
<div style="display:none"> Gerrit-Change-Number: 30293 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Michael Bacarella <michael.bacarella@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>