Merge ~vicamo/libhybris/+git/libhybris:for-libhybris-maintainers/fix-wifi-h-not-found into ~libhybris-maintainers/libhybris/+git/libhybris:master
Proposed by
You-Sheng Yang
Status: | Superseded |
---|---|
Proposed branch: | ~vicamo/libhybris/+git/libhybris:for-libhybris-maintainers/fix-wifi-h-not-found |
Merge into: | ~libhybris-maintainers/libhybris/+git/libhybris:master |
Diff against target: |
27 lines (+3/-3) 2 files modified
hybris/tests/test_wifi.c (+2/-2) hybris/wifi/wifi.c (+1/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Simon Fels | Approve | ||
Review via email: mp+280546@code.launchpad.net |
This proposal has been superseded by a proposal from 2016-01-14.
Description of the change
wifi: fix hardware_
Android headers were supposed to be installed to /usr/include. When it's not the case, even with a correct C include path set for Android headers, the build still fails with "hardware_
This patch removes the redundant level of directory from wifi related include directives.
To post a comment you must log in.
Actually prefixing a package name as namespace in the include directives is a safe policy that we should really enforce, but that takes several parties, pkg-config, down-stream projects to cooperate. This patch simply removes android/ as others do.
It should be fine for now to enforce such prefix policy in libhybris source code if that's what we decide to head to. And the decision here may affect the layout of android-headers package as well. See https:/ /code.launchpad .net/~vicamo/ android- headers/ +git/android- headers/ +merge/ 278111