unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: dmoncayo@gmail.com
Cc: emacs-devel@gnu.org
Subject: Re: Trunk broken on MS-Windows
Date: Sat, 08 Nov 2014 22:40:52 +0200	[thread overview]
Message-ID: <83ppcxpgrv.fsf@gnu.org> (raw)
In-Reply-To: <83tx29pine.fsf@gnu.org>

> Date: Sat, 08 Nov 2014 22:00:21 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: emacs-devel@gnu.org
> 
> The build fails with these error messages:
> 
>   Warning: arch-independent data dir `%emacs_dir%/share/emacs/25.0.50/etc/': Permission denied
>   Warning: Lisp directory `./lisp': No such file or directory
>   Error: charsets directory not found:
>   d:/gnu/bzr/emacs/trunk/src/%emacs_dir%/share/emacs/25.0.50/etc/charsets
>   Emacs will not function correctly without the character map files.
>   Please check your installation!
>   Makefile:833: recipe for target `bootstrap-emacs.exe' failed
>   make[1]: *** [bootstrap-emacs.exe] Error 1
> 
> Looks like some fallout from commits in r118316..118318.

The problem is with editing nt/epaths.nt into src/epaths.in, and
specifically with editing of @SRC@.  Instead of substituting the
absolute file name of the top-level directory, Makefile now
substitutes just ".", so we end up with

  #define PATH_DUMPLOADSEARCH "./lisp"

instead of the correct

  #define PATH_DUMPLOADSEARCH "d:/foo/bar/baz/lisp"

This makes this portion of init_callproc fail:

  if (data_dir == 0)
    {
      Lisp_Object tem, tem1, srcdir;
      Lisp_Object lispdir = Fcar (decode_env_path (0, PATH_DUMPLOADSEARCH, 0));

      srcdir = Fexpand_file_name (build_string ("../src/"), lispdir);

      tem = Fexpand_file_name (build_string ("NEWS"), Vdata_directory);
      tem1 = Ffile_exists_p (tem);
      if (!NILP (Fequal (srcdir, Vinvocation_directory)) || NILP (tem1))
	{
	  Lisp_Object newdir;
	  newdir = Fexpand_file_name (build_string ("../etc/"), lispdir);
	  tem = Fexpand_file_name (build_string ("NEWS"), newdir);
	  tem1 = Ffile_exists_p (tem);
	  if (!NILP (tem1))
	    Vdata_directory = newdir;
	}
    }

because when lispdir is just "./lisp", expand-file-name yields
d:/foo/bar/baz/src/src, since it expands "." into the directory where
temacs runs.

IOW, the new editing of epaths.h will probably fail for anyone who
builds inside the source tree.




  reply	other threads:[~2014-11-08 20:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-08 20:00 Trunk broken on MS-Windows Eli Zaretskii
2014-11-08 20:40 ` Eli Zaretskii [this message]
2014-11-08 21:19   ` Dani Moncayo
2014-11-08 22:25     ` Dani Moncayo
2014-11-08 23:34       ` Dani Moncayo
2014-11-09  3:39     ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83ppcxpgrv.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=dmoncayo@gmail.com \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).