Review Request 127205: Add stubs to allow compilation on Android.

Aleix Pol Gonzalez aleixpol at kde.org
Tue Mar 1 00:26:16 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127205/#review92959
-----------------------------------------------------------




src/lib/CMakeLists.txt (line 114)
<https://git.reviewboard.kde.org/r/127205/#comment63395>

    Why does pthread make a difference?



src/lib/util/kuser_unix.cpp (line 37)
<https://git.reviewboard.kde.org/r/127205/#comment63394>

    In glib they solved it using
    __BIONIC__
    https://mail.gnome.org/archives/commits-list/2012-November/msg07198.html
    
    If you look at the pwd.h file Android offers, you can see it says:
    ```
    #if 0 /* MISSING FROM BIONIC */
    int getpwnam_r(const char*, struct passwd*, char*, size_t, struct passwd**);
    int getpwuid_r(uid_t, struct passwd*, char*, size_t, struct passwd**);
    struct passwd* getpwent(void);
    int setpwent(void);
    #endif /* MISSING */
    ```
    So it might make sense.


- Aleix Pol Gonzalez


On Feb. 28, 2016, 11:14 a.m., Andreas Cord-Landwehr wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127205/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2016, 11:14 a.m.)
> 
> 
> Review request for KDE Frameworks, Aleix Pol Gonzalez and Alex Richardson.
> 
> 
> Repository: kcoreaddons
> 
> 
> Description
> -------
> 
> Android's libc implementation lacks several features required by the Unix backend implementation. This patch allows compilation by adding stubs for all missing struct members and functions. Result of this patch is that on Android KUser returns empty GUID and UID lists.
> 
> 
> Diffs
> -----
> 
>   src/lib/CMakeLists.txt cf57f0947f13b126f658774209363480013a2e2a 
>   src/lib/util/kuser_unix.cpp de5acde07f4cb8425f03c455bc55d03dfd2579e9 
> 
> Diff: https://git.reviewboard.kde.org/r/127205/diff/
> 
> 
> Testing
> -------
> 
> Compilation tested on Linux and Android.
> 
> 
> Thanks,
> 
> Andreas Cord-Landwehr
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160301/b3e8ad0c/attachment.html>


More information about the Kde-frameworks-devel mailing list