<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 28, 2019 at 11:47 AM Aditya Upadhyay <<a href="mailto:aadit0402@gmail.com">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">On Mon, May 27, 2019 at 10:42 PM Vaibhav Gupta <<a href="mailto:vaibhavgupta40@gmail.com" target="_blank">vaibhavgupta40@gmail.com</a>> wrote:<br>
><br>
> Hello, okay I will keep that in mind.<br>
> .<br>
> .<br>
> .<br>
> .<br>
> But there is one problem with this patch then.<br>
> .<br>
> .<br>
> 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.<br>
> Then run 'autoreconf' to add the entry for ndbm.c in Makefile.in.  And then run `make` command. There will be compilation error.<br>
> .<br>
<br>
See Instructions here about regenerating the configuration files in newlib.<br>
<a href="https://sourceware.org/newlib/README" rel="noreferrer" target="_blank">https://sourceware.org/newlib/README</a><br>
Newlib uses a specific version of autoconf. You can use `autoreconf<br>
-fvi` command to regenerate that<br>
Makefile.in file.<br></blockquote><div>Okay <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Header files needed by libc/posic/*.c files are present in libc/include directory. It is default path.<br>
> .<br>
> But ndbm.c also requires "hash.h" which is present inside "libc/search".<br>
> .<br>
> 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.<br>
<br>
If you specify any header file inclusion in Makefile.am, then you will<br>
need to add the things manually<br>
and when you will run autoreconf command, this .in file will get updated.<br>
<br></blockquote><div>I guess I will need to modify <a href="http://configure.in">configure.in</a> file. If I am not wrong that is responsible for what autoreconf is doing. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> .<br>
> .<br>
> 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..<br>
> .<br>
> .<br>
<br>
I think, you have the problem with your git cloning of newlib-repo.<br>
Otherwise It would not have generated this much<br>
bigger patch.<br></blockquote><div>No, Actually inclusion of any .c file affects almost 300 + files. They get modified when autoreconf is run. 'git diff' gave a rough idea, this is why patch got bigger.  <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
><br>
> One solution can be that I should not use autoreconf and make manual entries for ndbm.c in Makefile.in . Should I try that?<br>
> Because there is need to mention alternative source path for header files, to compile ndbm.c.<br>
><br>
You can do this manually, but it would be a wise approach to use autoreconf.<br>
<br>
> .<br>
> Thankyou<br>
> Vaibhav Gupta<br>
><br>
><br>
> On Mon, 27 May, 2019, 7:38 PM Aditya Upadhyay, <<a href="mailto:aadit0402@gmail.com" target="_blank">aadit0402@gmail.com</a>> wrote:<br>
>><br>
>> 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" 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></div>