<div dir="ltr"><div><div>Hi,<br><br>Apologies regarding that. <br><br>Some of the lines that showed '-' were because changes were committed on the old changes which were not right. <br><br>However, I have created a new branch where the commits are much cleaner.<br><br><a href="https://github.com/spark1729/rtems-libbsd/commits/RPI_SD" target="_blank">https://github.com/spark1729/rtems-libbsd/commits/RPI_SD</a><br></div></div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 3, 2016 at 5:50 PM, Sebastian Huber <span dir="ltr"><<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/06/16 13:20, Mudit Jain wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In the latest commit I have followed the FreeBSD port guidelines given libbsd.txt using the tools to port the code. <br>
</blockquote>
<br></span>
Sorry for being pedantic, but please follow the guidelines exactly:<br>
<br>
"Only add lines.  If your patch contains lines starting with a '-', then this is<br>
wrong.  Subtract code by added `#ifndef __rtems__`.  This makes merging easier<br>
in the future.  For example:"<br>
<br>
So, if you see red lines in your patches, then this is wrong.<br>
<br>
Do not rename things in the modules unless absolutely necessary. Do not adjust header includes, instead fix the build system.<div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
</div></div></blockquote></div><br></div>