Re: Native-comp branch compilation error on Windows

["Liāu, Kiong-Gē 廖宮毅" <gongyi.liao@gmail.com>]

It seems the case, here's the partial content of epaths.h:

/* Together with PATH_SITELOADSEARCH, this gives the default value of
   load-path, which is the search path for the Lisp function "load".
   Configure (using "make epaths-force") sets this to
   ${standardlisppath}, which typically has a value like:
   <datadir>/emacs/VERSION/lisp where datadir is eg /usr/local/share.
*/
#define PATH_LOADSEARCH "%emacs_dir%/share/emacs/28.0.50/lisp"

/* Like PATH_LOADSEARCH, but contains the non-standard pieces.
   These are the site-lisp directories.  Configure sets this to
   ${locallisppath}, which typically defaults to something like:
   <datadir>/emacs/VERSION/site-lisp:<datadir>/emacs/site-lisp
   but can be overridden by the --enable-locallisppath argument.
   This is combined with PATH_LOADSEARCH to make the default load-path.
   If the --no-site-lisp option is used, this piece is excluded.
*/
#define PATH_SITELOADSEARCH
"%emacs_dir%/share/emacs/28.0.50/site-lisp;%emacs_dir%/share/emacs/site-lisp"

/* Like PATH_LOADSEARCH, but used only during the build process
   when Emacs is dumping.  Configure (using "make epaths-force-w32") sets
   this to $buildlisppath, which normally has the value: <srcdir>/lisp.
*/
#define PATH_DUMPLOADSEARCH
"C:/msys64/home/VWinUser0/Downloads/emacs/native-comp/src/lisp"

It seems the %emacs_dir% part is not expanded

Kiong-Ge.





On Thu, Dec 17, 2020 at 2:43 PM Eli Zaretskii <eliz@gnu.org> wrote:
>
> > Date: Thu, 17 Dec 2020 19:51:19 +0000
> > Cc: emacs-devel@gnu.org
> > From: Andrea Corallo via "Emacs development discussions." <emacs-devel@gnu.org>
> >
> > > comp.c: In function 'Fcomp_el_to_eln_filename':
> > > comp.c:4110:36: error: expected ')' before 'PATH_REL_LOADSEARCH'
> > >  4110 |    Fregexp_quote (build_string ("/" PATH_REL_LOADSEARCH "/")));
> > >       |                                    ^~~~~~~~~~~~~~~~~~~~
> > >       |                                    )
> > > comp.c: In function 'Fcomp__compile_ctxt_to_file':
> > > comp.c:4458:12: warning: unused variable 'oldset' [-Wunused-variable]
> > >  4458 |   sigset_t oldset;
> > >       |            ^~~~~~
> >
> > Hi Liāu,
> >
> > please open a bug with M-x report-emacs-bug adding into the subject
> > [feature/native-comp].
> >
> > Also if you could attach to the bug your src/epaths.h this will be
> > helpful.
>
> I don't see that the branch updated nt/epaths.nt (and in top-level
> Makefile.in), without which nothing useful can happen with
> PATH_REL_LOADSEARCH.
>
> And if PATH_REL_LOADSEARCH is defined on MS-Windows like I think it
> should, you will probably have problems using it in a regex, because
> it will include the unexpanded %emacs_dir% part?