Subject: bug#66507: closed (Re: bug#66507: 30.0.50; Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.)

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 13 Oct 2023 05:03:02 +0000

Your bug report

#66507: 30.0.50; Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
66507: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=66507
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#66507: 30.0.50; Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
From: Po Lu <[email-masked]>
To: Takesi Ayanokoji <[email-masked]>
Cc: [email-masked]
Date: Fri, 13 Oct 2023 13:01:58 +0800
List-Id: <debbugs-submit.debbugs.gnu.org>

Takesi Ayanokoji <[email-masked]> 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.

Subject: 30.0.50; Cannot build some modules without adding LOCAL_EXPORT_CFLAGS=-isystem ... on Android port.
From: Takesi Ayanokoji <[email-masked]>
To: [email-masked]
Date: Fri, 13 Oct 2023 06:17:50 +0900
List-Id: <debbugs-submit.debbugs.gnu.org>

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,