<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body smarttemplateinserted="true">
<div id="smartTemplate4-template">
<p>Hi Petr,<br>
</p>
</div>
<br>
<div id="smartTemplate4-quoteHeader">-------- Original Message --------<br>
From: Petr Vorel<br>
Sent: Mon, 22 Apr 2019 00:10:30 +0200<br>
To: Zhengwang Ruan<br>
Cc: Sandeep Patil, Ltp, Steve Muckle<br>
Subject: Re: [LTP] 回复：[PATCH v1] include/mk/env_post.mk : enable
__ANDROID__ definition for Android build<br>
</div>
<blockquote type="cite" cite="mid:20190421221030.GA11307@x230">
<pre class="moz-quote-pre" wrap="">Hi Sandeep, Zhengwang,
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">I don't know why autotools fails preadv and pwritev detection with AC_CHECK_FUNCS.
I considered it as problem of my setup, but maybe it's an autotools bug.
</pre>
</blockquote>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">I also encountered this issue, but I temporarily walked around it by
applying the following patch.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">==================
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">$ git diff testcases/
diff --git a/testcases/commands/ldd/datafiles/Makefile
b/testcases/commands/ldd/datafiles/Makefile
index d8bcf5c..74fbe50 100644
--- a/testcases/commands/ldd/datafiles/Makefile
+++ b/testcases/commands/ldd/datafiles/Makefile
@@ -30,7 +30,7 @@ CLEAN_TARGETS += *.obj $(LDD_SO_FILES)
$(CC) $(CFLAGS) -shared -o $@ $^
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap=""> lddfile.out: main.o $(LDD_SO_FILES)
- $(CC) $(CFLAGS) -O -o $@ $?
+ $(CC) $(CFLAGS) $(LDFLAGS) -O -o $@ $?
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">What is this change for?</pre>
</blockquote>
<p>This is a really long story. In my build, I need to use
'-nostdlib /ndk/path/to/crtend_android.o
/ndk/path/to/crtbegin_dynamic.o' as linker arguments to
cross-compile, and I have to put these arguments into LDFLAGS
since there were other errors occurred on linking *.o to
executable if I put them into CFLAGS. <br>
</p>
<p>Right here, lddfile.out is an executable and obviously needed to
be linked, therefore LDFLAGS must be required in my case. <br>
</p>
<p><br>
</p>
<p>Regards,</p>
<p>Zhengwang<br>
</p>
<p><br>
</p>
<blockquote type="cite" cite="mid:20190421221030.GA11307@x230">
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap=""> INSTALL_DIR := testcases/data/ldd01
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">diff --git a/testcases/kernel/syscalls/Makefile
b/testcases/kernel/syscalls/Makefile
index 45a00cf..27c2e7a 100644
--- a/testcases/kernel/syscalls/Makefile
+++ b/testcases/kernel/syscalls/Makefile
@@ -43,6 +43,8 @@ FILTER_OUT_DIRS += \
sched_getaffinity sched_setaffinity sendmsg setgroups setns sighold
\
sigrelse sigsuspend sigtimedwait sigwait sigwaitinfo stime \
setdomainname sethostname symlinkat ulimit ustat vfork vhangup
vmsplice
+
+FILTER_OUT_DIRS += preadv pwritev
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">Yes, that fixes build. But manually disabling features in config.h enables these
tests to be build.
Kind regards,
Petr
</pre>
</blockquote>
</body>
</html>