<div dir="ltr">I forgot to say, that I put the full tree I worked on at <a href="http://www.contrib.andrew.cmu.edu/~rjkohler/">http://www.contrib.andrew.cmu.edu/~rjkohler/</a> for reference.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 13, 2014 at 6:17 PM, Ray Kohler <span dir="ltr"><<a href="mailto:ataraxia937@gmail.com" target="_blank">ataraxia937@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Package: src:android-tools<br>
Followup-For: Bug #738119<br>
<br>
Although the android-tools packaging is indeed rather bad, I've made an<br>
attempt to update it for 5.0.1. (Integrating it with<br>
android-platform-system-core was beyond my skill.)<br>
<br>
I'm attaching updates of the three custom makefiles supplied under<br>
debian/makefiles. Some comments on the changes I made:<br>
<br>
- AndroidConfig.h is no more, so the build-dep on android-system-dev is<br>
  no longer needed.<br>
<br>
- adb doesn't appear to use libzipfile any longer.<br>
<br>
- adb is currently built with -DWORKAROUND_BUG6558362 by upstream, so I<br>
  did likewise.<br>
<br>
- fastboot now depends on f2fs_utils, i.e., extras/f2fs_utils and<br>
  externals/f2fs-tools. (Debian has an f2fs-tools package, but<br>
  apparently no dev package to go with it.) f2fs_utils in turn pulls in<br>
  core/include/log and core/include/android.<br>
<br>
- fastboot (or one of its deps, I forgot which) needs<br>
  core/include/utils.<br>
<br>
- ext4_utils now needs the selinux headers from external/libselinux, as<br>
  Debian's libselinux1-dev doesn't contain android.h.<br>
<br>
- ext4_utils needs -DANDROID and -DHOST to compile correctly.<br>
<br>
There were also some problems with the actual source files, which will<br>
need patches:<br>
<br>
- libzipfile/centraldir.c needs an explicit #define _FILE_OFFSET_BITS 64<br>
  to compile.<br>
<br>
- extras/f2fs_utils/f2fs_sparseblock.c is missing includes for stdlib.h,<br>
  string.h, and errno.h.<br>
<br>
- extras/f2fs_utils/f2fs_utils.c is missing an include of string.h.<br>
<br>
As the internal deps continue to grow, it's even more true that the<br>
current build method of this package is a problem. However, since there<br>
are multiple deps on source files outside of the platform/core repo,<br>
integration with android-platform-system-core also feels wrong. Upstream<br>
doesn't make this easy.<br>
<br>
-- System Information:<br>
Debian Release: 8.0<br>
  APT prefers unstable<br>
  APT policy: (500, 'unstable')<br>
Architecture: amd64 (x86_64)<br>
<br>
Kernel: Linux 3.16.0-4-amd64 (SMP w/2 CPU cores)<br>
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)<br>
Shell: /bin/sh linked to /bin/dash<br>
Init: systemd (via /run/systemd/system)<br>
</blockquote></div><br></div>