GNU bug report logs - #66507
30.0.50; Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 66507 in the body.
You can then email your comments to 66507 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50;
 Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem
 ... on Android port.
Date: Fri, 13 Oct 2023 06:17:50 +0900

[Message part 1 (text/plain, inline)]

Hello,

Duaring building Emacs Android port, I encountered some modules that could
not build without specifying system include header file by -isystem.

These modules are

libxml2:
add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/include
 to its Android.mk.

lbjansson:
add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/src -I
$(LOCAL_PATH)/android
 to its Android.mk

core/libpackagelistparser:
add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/../include
(I installed it from core/libpackagelistparser)

Thanks,

[Message part 2 (text/html, inline)]

Message #8 received at 66507 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com>
Cc: 66507 <at> debbugs.gnu.org
Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding
 LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
Date: Fri, 13 Oct 2023 09:06:11 +0800

Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:

> Hello,
>
> Duaring building Emacs Android port, I encountered some modules that could not build without specifying system include header file by -isystem.
>
> These modules are
>
> libxml2:
> add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/include
>  to its Android.mk.
>
> lbjansson:
> add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/src -I $(LOCAL_PATH)/android
>  to its Android.mk
>
> core/libpackagelistparser:
> add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/../include
> (I installed it from core/libpackagelistparser)
>
> Thanks,

IIRC all three of these changes are mentioned in java/INSTALL, but
please correct me if I'm wrong.

Message #11 received at 66507 <at> debbugs.gnu.org (full text, mbox):

From: Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 66507 <at> debbugs.gnu.org
Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding
 LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
Date: Fri, 13 Oct 2023 13:30:19 +0900

[Message part 1 (text/plain, inline)]

Hello Po- san.

Unfortunately it doesn't seem to be mentioned in the java/INSTALL.

Thanks,

2023年10月13日(金) 午前10:06 Po Lu <luangruo <at> yahoo.com>:

> Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:
>
> > Hello,
> >
> > Duaring building Emacs Android port, I encountered some modules that
> could not build without specifying system include header file by -isystem.
> >
> > These modules are
> >
> > libxml2:
> > add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/include
> >  to its Android.mk.
> >
> > lbjansson:
> > add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/src -I
> $(LOCAL_PATH)/android
> >  to its Android.mk
> >
> > core/libpackagelistparser:
> > add LOCAL_EXPORT_CFLAGS := -isystem $(LOCAL_PATH)/../include
> > (I installed it from core/libpackagelistparser)
> >
> > Thanks,
>
> IIRC all three of these changes are mentioned in java/INSTALL, but
> please correct me if I'm wrong.
>

[Message part 2 (text/html, inline)]

Message #16 received at 66507-done <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com>
Cc: 66507-done <at> debbugs.gnu.org
Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding
 LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
Date: Fri, 13 Oct 2023 13:01:58 +0800

Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:

> Hello Po- san.
>
> Unfortunately it doesn't seem to be mentioned in the java/INSTALL.

I've now repaired these ommissions; that being said, I opted to
recommend modifications to the Makefiles different from those you
mentioned, for include paths are customarily exported through the
variable LOCAL_EXPROT_C_INCLUDE_DIRS, not variables meant for exporting
CFLAGS.

Message #19 received at 66507-done <at> debbugs.gnu.org (full text, mbox):

From: Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 66507-done <at> debbugs.gnu.org
Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding
 LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
Date: Fri, 13 Oct 2023 15:40:27 +0900

[Message part 1 (text/plain, inline)]

Hello,

I will try LOCAL_EXPORT_C_INCLUDE_DIRS instead.

Thanks,

2023年10月13日(金) 午後2:02 Po Lu <luangruo <at> yahoo.com>:

> Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:
>
> > Hello Po- san.
> >
> > Unfortunately it doesn't seem to be mentioned in the java/INSTALL.
>
> I've now repaired these ommissions; that being said, I opted to
> recommend modifications to the Makefiles different from those you
> mentioned, for include paths are customarily exported through the
> variable LOCAL_EXPROT_C_INCLUDE_DIRS, not variables meant for exporting
> CFLAGS.
>

[Message part 2 (text/html, inline)]

Message #22 received at 66507-done <at> debbugs.gnu.org (full text, mbox):

From: ayatakesi <ayanokoji.takesi <at> gmail.com>
To: Po Lu <luangruo <at> yahoo.com>
Cc: 66507-done <at> debbugs.gnu.org
Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding
 LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
Date: Mon, 16 Oct 2023 02:35:32 +0900

Hello, again

I tried LOCAL_EXPORT_C_INCLUDE_DIRS, and worked fine
 except core/libpackagelistparser.

When I build libpackagelistparser, make failed with

/home/runner/work/my-emacs/my-emacs/my_sub_modules/core/libpackagelistparser//packagelistparser.c:30:10: fatal error: 'utils/Log.h' file not found
#include <utils/Log.h>
         ^~~~~~~~~~~~~

I cloned core package from Google, and I specify such bellow

directory structure:
core
|-libpackagelistparser
    Android.mk
|-utils
  |-include
      Log.h

in Android.mk:
LOCAL_EXPORT_C_INCLUDE_DIRS += $(LOCAL_PATH)/../include

But LOCAL_EXPORT_C_INCLUDE_DIRS and LOCAL_EXPORT_C_INCLUDES
 did not work except LOCAL_EXPORT_CFLAGS.


Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:

> Hello,
>
> I will try LOCAL_EXPORT_C_INCLUDE_DIRS instead.
>
> Thanks,
>
> 2023年10月13日(金) 午後2:02 Po Lu <luangruo <at> yahoo.com>:
>
>  Takesi Ayanokoji <ayanokoji.takesi <at> gmail.com> writes:
>
>  > Hello Po- san.
>  >
>  > Unfortunately it doesn't seem to be mentioned in the
>  java/INSTALL.
>
>  I've now repaired these ommissions; that being said, I
>  opted to
>  recommend modifications to the Makefiles different from
>  those you
>  mentioned, for include paths are customarily exported
>  through the
>  variable LOCAL_EXPROT_C_INCLUDE_DIRS, not variables
>  meant for exporting
>  CFLAGS.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.