Re: Android port

[Eli Zaretskii]

> From: Po Lu <luangruo@yahoo.com>
> Cc: Angelo Graziosi <angelo.g0@libero.it>,  Eli Zaretskii <eliz@gnu.org>,
>   bruno@clisp.org,  eggert@cs.ucla.edu,  emacs-devel@gnu.org
> Date: Mon, 07 Aug 2023 16:44:46 +0800
> 
> Corwin Brust <corwin@bru.st> writes:
> 
> > On Mon, Aug 7, 2023 at 2:22 AM Po Lu <luangruo@yahoo.com> wrote:
> >>
> >> Angelo Graziosi <angelo.g0@libero.it> writes:
> >>
> >> > maybe we have to test master now. Right?
> >>
> >> It works, since the printf module has been removed.
> >>
> >
> > I'm not able to build the master branch starting from c71a52, here is
> > the relevant part of the log (attached), which starts with autogen &&
> > configure prior to the make, which failed:
> >
> >   GEN      ../../etc/charsets/CP1258.map
> >   CC       indent.o
> >   CC       search.o
> > fileio.c: In function 'internal_delete_file':
> > fileio.c:2601:36: warning: passing argument 1 of
> > 'internal_condition_case_2' from incompatible pointer type
> > [-Wincompatible-pointer-types]
> >  2601 |   tem = internal_condition_case_2 (Fdelete_file_internal, filename,
> >       |                                    ^~~~~~~~~~~~~~~~~~~~~
> >       |                                    |
> >       |                                    struct Lisp_X * (*)(struct 
> > Lisp_X *)
> > In file included from fileio.c:49:
> > lisp.h:4582:47: note: expected 'struct Lisp_X * (*)(struct Lisp_X *,
> > struct Lisp_X *)' but argument is of type 'struct Lisp_X * (*)(struct
> > Lisp_X *)'
> >  4582 | extern Lisp_Object internal_condition_case_2 (Lisp_Object (*)
> > (Lisp_Object, Lisp_Object), Lisp_Object, Lisp_Object, Lisp_Object,
> > Lisp_Object (*) (Lisp_Object));
> >       |
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > fileio.c:2602:40: warning: passing argument 4 of
> > 'internal_condition_case_2' from incompatible pointer type
> > [-Wincompatible-pointer-types]
> >  2602 |                                    Qt, internal_delete_file_1);
> >       |                                        ^~~~~~~~~~~~~~~~~~~~~~
> >       |                                        |
> >       |                                        struct Lisp_X *
> > (*)(struct Lisp_X *)
> > In file included from fileio.c:49:
> > lisp.h:4582:117: note: expected 'Lisp_Object' {aka 'struct Lisp_X *'}
> > but argument is of type 'struct Lisp_X * (*)(struct Lisp_X *)'
> >  4582 | extern Lisp_Object internal_condition_case_2 (Lisp_Object (*)
> > (Lisp_Object, Lisp_Object), Lisp_Object, Lisp_Object, Lisp_Object,
> > Lisp_Object (*) (Lisp_Object));
> >       |
> >                                                      ^~~~~~~~~~~
> > fileio.c:2601:9: error: too few arguments to function
> > 'internal_condition_case_2'
> >  2601 |   tem = internal_condition_case_2 (Fdelete_file_internal, filename,
> >       |         ^~~~~~~~~~~~~~~~~~~~~~~~~
> > In file included from fileio.c:49:
> > lisp.h:4582:20: note: declared here
> >  4582 | extern Lisp_Object internal_condition_case_2 (Lisp_Object (*)
> > (Lisp_Object, Lisp_Object), Lisp_Object, Lisp_Object, Lisp_Object,
> > Lisp_Object (*) (Lisp_Object));
> >       |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
> >   GEN      ../../etc/charsets/CP10007.map
> >   CC       regex-emacs.o
> >   GEN      ../../etc/charsets/CP720.map
> >   GEN      ../../etc/charsets/CP858.map
> >   CC       undo.o
> > make[1]: *** [Makefile:455: fileio.o] Error 1
> > make[1]: *** Waiting for unfinished jobs....
> >
> > Let me know if there's other information/digging I might provide.
> >
> > Note, I updated my script to add V=1 for make in the future.
> 
> I think this is a problem with the fileio stuff Eric installed earlier
> instead of the Android port.  Simply one of the many hazards of
> indiscriminately translating working C to Elisp...

Actually, it's partially my fault: I made a mistake in my fix of
Eric's changeset.  I hope it's fixed now on master.