<div dir="ltr">Also the newlib community really appreciates it if we review the patches on the RTEMS development list first. When the patch is OK, then it can go to the newlib list.<div><br></div><div>--joel</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 27, 2019 at 12:12 PM Vaibhav Gupta <<a href="mailto:vaibhavgupta40@gmail.com">vaibhavgupta40@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Hello, okay I will keep that in mind.<div dir="auto">.</div><div dir="auto">.</div><div dir="auto">.</div><div dir="auto">.<br><div dir="auto">But there is one problem with this patch then. </div><div dir="auto">.</div><div dir="auto">.</div><div dir="auto">If i just copy-paste ndbm.h, db.h(needed by ndbm.h) and ndbm.c and make the entry of ndbm.c  in Makefile.am. </div><div dir="auto">Then run 'autoreconf' to add the entry for ndbm.c in Makefile.in.  And then run `make` command. There will be compilation error. </div><div dir="auto">.</div><div dir="auto">Header files needed by libc/posic/*.c files are present in libc/include directory. It is default path.</div><div dir="auto">.</div><div dir="auto">But ndbm.c also requires "hash.h" which is present inside "libc/search".</div><div dir="auto">.</div><div dir="auto">To over come it, i had to modify the entry of ndbm.c in Makefile.in and add "-I (topr_srcdir)/search" to specify the compiler for alternative source path.</div><div dir="auto">.</div><div dir="auto">.</div><div dir="auto">The problem is, to modify Makefile.in there should be entry of ndbm.c . And for that i need to run autoreconf command. Because of that multiple files get modified and hence the patch size got bigger..</div><div dir="auto">.</div><div dir="auto">.</div><div dir="auto"><br></div><div dir="auto">One solution can be that I should not use autoreconf and make manual entries for ndbm.c in Makefile.in . Should I try that?</div><div dir="auto">Because there is need to mention alternative source path for header files, to compile ndbm.c.</div><div dir="auto"><br></div><div dir="auto">.</div><div dir="auto">Thankyou</div><div dir="auto">Vaibhav Gupta</div><div dir="auto"> </div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 27 May, 2019, 7:38 PM Aditya Upadhyay, <<a href="mailto:aadit0402@gmail.com" target="_blank">aadit0402@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Vaibhav,<br>
<br>
You have sent a completely wrong patch. You do not need to include all<br>
these files what are existed there. You should generate a patch for<br>
your work like addition or modification or if you have added any other<br>
file.<br>
<br>
This patch should go to newlib mailing list. For your entire GSoC<br>
period, You will send patch to Newlib-Cygwin and For testing<br>
contributed methods, you will be adding testcases in rtems/testsuite.<br>
<br>
For testing those methods, you will have to apply patches to<br>
rtems-source-builder, see instructions here:<br>
<a href="https://docs.rtems.org/releases/rtemsdocs-4.10.2/share/rtems/html/started/started00036.html" rel="noreferrer noreferrer" target="_blank">https://docs.rtems.org/releases/rtemsdocs-4.10.2/share/rtems/html/started/started00036.html</a><br>
<br>
If you have not introduced yourself on newlib mailing list, then<br>
Introduce yourself there. Point your plan what you are going to do<br>
this GSoC Period. Though these are parts of Community bonding period.<br>
<br>
I appreciate your effort. This is just an initial phase of GSoC. You<br>
should take care of these things.<br>
<br>
Thanks,<br>
</blockquote></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>