unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#18383: Build failure
@ 2014-09-01 19:20 Angelo Graziosi
  2014-09-01 20:14 ` Andy Moreton
  0 siblings, 1 reply; 2+ messages in thread
From: Angelo Graziosi @ 2014-09-01 19:20 UTC (permalink / raw)
  To: 18383; +Cc: Dmitry Antipov

Current trun, rev. 117801, fails to build (I tried on MSYS2-MinGW64) 
with this compiler error:

gcc -std=gnu99  -c -mtune=generic   -DUSE_CRT_DLL=1 -I /tmp/emacs/nt/inc 
-Demacs  -I. -I. -I../lib -I../lib  -mtune=generic 
-IC:/msys64/mingw64/include/librsvg-2.0 
-IC:/msys64/mingw64/include/gdk-pixbuf-2.0 -pthread 
-IC:/msys64/mingw64/include/cairo -IC:/msys64/mingw64/include/glib-2.0 
-IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields 
-IC:/msys64/mingw64/include/pixman-1 -I/mingw64/include/libpng16 
-I/mingw64/include/harfbuzz  -IC:/msys64/mingw64/include 
-I/mingw64/include/freetype2 -IC:/msys64/mingw64/include/libpng16 
-IC:/msys64/mingw64/include/libxml2 
-IC:/msys64/mingw64/include/dbus-1.0 
-IC:/msys64/mingw64/lib/dbus-1.0/include           -MMD -MF 
deps/fileio.d -MP -IC:/msys64/mingw64/include/p11-kit-1 
-IC:/msys64/mingw64/include      -I/mingw64/include/noX -Ofast -g0 -pipe 
  fileio.c
fileio.c: In function 'Fexpand_file_name':
fileio.c:1345:26: error: 'aidr' undeclared (first use in this function)
       newdirlen = strlen (aidr);
                           ^
fileio.c:1345:26: note: each undeclared identifier is reported only once 
for each function it appears in


maybe this change was the origin of failure:

2014-09-01  Dmitry Antipov  <xyz@xyz>

	Avoid extra calls to strlen in filesystem I/O routines.
	* fileio.c (Fexpand_file_name): Avoid calls to strlen if
	the length of 'newdir' is known or may be precalculated.
	(file_accessible_directory_p): Prefer to pass Lisp_Object,
	not 'char *', and so use precalculated length.
	(Ffile_accessible_directory_p):
	[...]


Ciao,
  Angelo.





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#18383: Build failure
  2014-09-01 19:20 bug#18383: Build failure Angelo Graziosi
@ 2014-09-01 20:14 ` Andy Moreton
  0 siblings, 0 replies; 2+ messages in thread
From: Andy Moreton @ 2014-09-01 20:14 UTC (permalink / raw)
  To: 18383

On Mon 01 Sep 2014, Angelo Graziosi wrote:

> Current trun, rev. 117801, fails to build (I tried on MSYS2-MinGW64) with this
> compiler error:
>
> gcc -std=gnu99  -c -mtune=generic   -DUSE_CRT_DLL=1 -I /tmp/emacs/nt/inc
> -Demacs  -I. -I. -I../lib -I../lib  -mtune=generic
> -IC:/msys64/mingw64/include/librsvg-2.0
> -IC:/msys64/mingw64/include/gdk-pixbuf-2.0 -pthread
> -IC:/msys64/mingw64/include/cairo -IC:/msys64/mingw64/include/glib-2.0
> -IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields
> -IC:/msys64/mingw64/include/pixman-1 -I/mingw64/include/libpng16
> -I/mingw64/include/harfbuzz  -IC:/msys64/mingw64/include
> -I/mingw64/include/freetype2 -IC:/msys64/mingw64/include/libpng16
> -IC:/msys64/mingw64/include/libxml2 -IC:/msys64/mingw64/include/dbus-1.0
> -IC:/msys64/mingw64/lib/dbus-1.0/include           -MMD -MF deps/fileio.d -MP
> -IC:/msys64/mingw64/include/p11-kit-1 
> -IC:/msys64/mingw64/include      -I/mingw64/include/noX -Ofast -g0 -pipe
> fileio.c
> fileio.c: In function 'Fexpand_file_name':
> fileio.c:1345:26: error: 'aidr' undeclared (first use in this function)
>       newdirlen = strlen (aidr);
>                           ^
> fileio.c:1345:26: note: each undeclared identifier is reported only once for
> each function it appears in
>

It looks like a typo in r117801. This fixed the Mingw64 build for me:

=== modified file 'src/fileio.c'
--- src/fileio.c        2014-09-01 16:05:43 +0000
+++ src/fileio.c        2014-09-01 20:10:44 +0000
@@ -1342,7 +1342,7 @@
              memcpy (adir, SSDATA (tem), newdirlen + 1);
            }
          else
-           newdirlen = strlen (aidr);
+           newdirlen = strlen (adir);
          newdir = adir;
        }



HTH,

    AndyM






^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-09-01 20:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-01 19:20 bug#18383: Build failure Angelo Graziosi
2014-09-01 20:14 ` Andy Moreton

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).