<div dir="ltr">Hi Hans,<div><br></div><div>Thanks for the review! Actually I copied debian/copyright from the original libunwind package ;)</div><div><br></div><div>The debian/<a href="http://detect-arch.mk">detect-arch.mk</a> queries DEB_HOST_ARCH_CPU variable which I think returns one of the values in the "Debian name" column of /usr/share/dpkg/cputable file, and I think this is the same variable for the "Architecture" field in debian/control. Once I was putting armel and armhf to the "Architecture" field and Lintian said that they are not valid CPU types, then I headed to #debian-qa for the issue, someone told me that the value of "Architecture" field can be found in <span style="font-size:13.1999998092651px;line-height:19.7999992370605px">/usr/share/dpkg/cputable file. So I think "arm" covers both armel and armhf. I put "armeb" as well because it exists in that file.</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px"><br></span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">I myself is confident that Android SDK is runnable on ARM and MIPS machines. From the AndroidConfig.h and some Android.mk elsewhere we can know that all programs and libraries can be built for the host version on x86, ARM and MIPS platforms, although Google only releases x86 SDK.</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px"><br></span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">For the hardening warnings, I think some deep bugs cause that because I applied the dpkg flags to the compiler and I can see the build commands are already set the proper flags. When I was packaging libnative-platform-java I was facing the same issue.</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px"><br></span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">Cheers,</span></div><div><span style="font-size:13.1999998092651px;line-height:19.7999992370605px">Kai-Chung Yan</span></div></div><br><div class="gmail_quote"><div dir="ltr">Hans-Christoph Steiner <<a href="mailto:hans@at.or.at">hans@at.or.at</a>> 於 2015年8月19日週三 上午1:23寫道：<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Hey seamlik,<br>
<br>
Looks good, nice work on debian/copyright, lots of stuff there.<br>
<br>
It looks like you removed 'armhf' support and added 'armeb' support, if I am<br>
following debian/<a href="http://detect-arch.mk" rel="noreferrer" target="_blank">detect-arch.mk</a> correctly.  'armeb' is dead.  'armhf' and<br>
'armel' are active, as far as I know:<br>
<br>
<a href="https://wiki.debian.org/ArmHardFloatPort" rel="noreferrer" target="_blank">https://wiki.debian.org/ArmHardFloatPort</a><br>
<a href="https://buildd.debian.org/" rel="noreferrer" target="_blank">https://buildd.debian.org/</a><br>
<br>
Also, there are some lintian warnings that should be dealt with:<br>
<br>
* postrm-has-useless-call-to-ldconfig - I did a lintian override to hide this<br>
warning on other packages like in android-platform-system-core.  I included<br>
the same one since it is the same issue.<br>
<br>
* hardening-no-fortify-functions - Other android-lib* packages have gotten<br>
this working, so it would be good to have them on this package too.<br>
<br>
Since Google only supports amd64 and i386 for the Android SDK, I went ahead an<br>
uploaded this package without the changes to the ARM names.  If you get around<br>
to fixing the issues here, then we can upload a new package version.<br>
<br>
.hc<br>
<br>
殷啟聰:<br>
> Hi,<br>
><br>
> I believe android-platform-external-libunwind is ready for review now. This<br>
> package is an AOSP fork of libunwind with modifications specific for<br>
> Android, so we need it packaged separated from the origianl libunwind. It<br>
> installs the libraries to /usr/lib/android so they won't affect the system.<br>
><br>
> Currently the only r-depends is libbacktrace of android-platform-system.<br>
><br>
> android-platform-external-libunwind:<br>
> <a href="http://anonscm.debian.org/cgit/android-tools/android-platform-external-libunwind.git" rel="noreferrer" target="_blank">http://anonscm.debian.org/cgit/android-tools/android-platform-external-libunwind.git</a><br>
><br>
> android-platform-system:<br>
> <a href="http://anonscm.debian.org/cgit/android-tools/android-platform-system.git" rel="noreferrer" target="_blank">http://anonscm.debian.org/cgit/android-tools/android-platform-system.git</a><br>
><br>
> Thank you!<br>
><br>
> Cheers,<br>
> Kai-Chung Yan<br>
><br>
</blockquote></div><div dir="ltr">-- <br></div><div dir="ltr"><p dir="ltr">殷啟聰 | Kai-Chung Yan<br>
一生只向真理與妻子低頭<br>
Full-time student of National Taichung University of Education<br>
LinkedIn: <<a href="https://linkedin.com/in/seamlik">https://linkedin.com/in/seamlik</a>><br>
Blog: <<a href="http://seamlik.logdown.com">http://seamlik.logdown.com</a>></p>
</div>