Problem using library-specific CPPFLAGS with new --automake-subdir option

2022-03-08 Thread John W. Eaton
First, thanks for working on the new --automake-subdir feature. We've been wanting to use a non-recursive build of gnulib with Octave for many years but have never been able to make it work completely with the previous non-recursive-gnulib-prefix-hack module. For the following, I updated to

Re: Re: imperative vs. descriptive style

2018-05-07 Thread John W. Eaton
On 05/06/2018 10:33 AM, Jim Meyering wrote: Here's the relevant section: (from https://www.gnu.org/prep/standards/html_node/GNU-Manuals.html) Whenever possible, please stick to the active voice, avoiding the passive, and use the present tense, not the future teste. For instance, write “The funct

Re: The non-recursive-gnulib-prefix-hack module

2017-02-01 Thread John W. Eaton
Is there anything I can do to help get this change accepted as part of gnulib? I'd really like to be able to use a completely non-recursive Makefile for Octave. jwe On 11/28/2016 12:40 AM, John W. Eaton wrote: On 11/27/2016 06:18 PM, Bruno Haible wrote: John, I

Small change needed for link.m4

2016-12-13 Thread John W. Eaton
The test in link.m4 needs to include to provide a declaration for the rename function. The test fails if building with GCC's -Werror=implicit-function-declaration option. This was reported as part of a bug with building Octave on Fedora, which now uses -Werror=implicit-function-declaration b

Re: The non-recursive-gnulib-prefix-hack module

2016-11-27 Thread John W. Eaton
On 11/27/2016 06:18 PM, Bruno Haible wrote: John, I added AC_CONFIG_LIBOBJ_DIR([libgnu]) to my configure.ac file. Maybe gnulib-tool could do this automatically if --non-recursive-makefile is specified? gnulib-tool does not rely on AC_LIBOBJ and friends, because these autoconf macros ass

Re: The non-recursive-gnulib-prefix-hack module

2016-11-27 Thread John W. Eaton
On 11/27/2016 05:02 PM, Bruno Haible wrote: * I would add a check to make sure that --non-recursive-makefile is only used together with the --makefile-name option. I cannot see how/why someone would use --non-recursive-makefile without --makefile-name. This simplies some of the logic (e.g.

Re: The non-recursive-gnulib-prefix-hack module

2016-11-27 Thread John W. Eaton
On 11/27/2016 04:29 PM, Bruno Haible wrote: When you specify --makefile-name=module.mk, gnulib-tool knows that you don't want a top-level Makefile.am and does not generate AUTOMAKE_OPTIONS and generates augmentations (+=) of the other variables instead of assignments. OK, I didn't realize that

Re: The non-recursive-gnulib-prefix-hack module

2016-11-27 Thread John W. Eaton
On 11/27/2016 07:52 AM, John W. Eaton wrote: On 10/22/2016 03:04 PM, Bruno Haible wrote: The idea would be to have gnulib-tool emit the correct code for the {Bison,coreutils,Octave} case right away, triggered by some command line option. If you want to help us here, please use the *current

Re: gethostname on 64-bit Windows

2016-06-26 Thread John W. Eaton
On 06/24/2016 11:01 PM, Tatsuro MATSUOKA wrote: - Original Message - From: John W. Eaton To: gnulib Cc: j...@octave.org Date: 2016/6/25, Sat 04:46 Subject: gethostname on 64-bit Windows I needed the attached change to cross compile Octave for i686-w64-mingw32. Without it

gethostname on 64-bit Windows

2016-06-24 Thread John W. Eaton
I needed the attached change to cross compile Octave for i686-w64-mingw32. Without it, rpl_gethostname was undefined because w64 has a gethostname function so gethostname.c is not added to the list of files to build but gethostname was still being redefined to rpl_gethostname. jwe diff -uNr

Adding files to gnulib_extra_files in bootstrap.conf?

2016-06-21 Thread John W. Eaton
The bootstrap script has # Extra files from gnulib, which override files from other sources. test -z "${gnulib_extra_files}" && \ gnulib_extra_files=" build-aux/install-sh build-aux/mdate-sh build-aux/texinfo.tex build-aux/depcomp build-aux/config.guess

Re: Canonicalize bug?

2015-08-31 Thread John W. Eaton
On 08/31/2015 09:54 AM, Paul Eggert wrote: Juan Pablo Carbajal wrote: Is this a bug in gnulib? Do you have file systems where '/' does not work to separate file name components? Like VMS or something? If so, it's a problem, though I'm not sure it's a bug, as most of gnulib doesn't claim to p

rpl_gethostname undefined on MinGW system when compiling Octave

2013-03-15 Thread John W. Eaton
I'm getting an error about rpl_gethostname undefined when compiling Octave for MinGW. HAVE_GETHOSTNAME is 1, so the AC_LIBOBJ macro from the module/gethostname file is not used, but unistd.in.h is checking GNULIB_GETHOSTNAME, which is also defined to 1. So the macros there are expanded and I

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread John W. Eaton
On 02/19/2013 05:46 PM, Paul Eggert wrote: On 02/19/13 14:04, John W. Eaton wrote: and also to use _putenv in the normal case when the argument string to putenv contains "VAR=VAL". Ouch, thanks for catching that; it suggests further fixes. Could you please try this patch to gnul

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread John W. Eaton
On 02/18/2013 10:40 PM, Paul Eggert wrote: Thanks, how about this change instead? It's a bit more conservative about always using _putenv and setting errno. And it uses WIN32_LEAN_AND_MEAN. I need the attached additional changes to provide declarations for putenv_result and putenv_error in th

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-17 Thread John W. Eaton
On 02/15/2013 03:16 PM, Michael Goffioul wrote: On Fri, Feb 15, 2013 at 2:10 PM, Paul Eggert mailto:egg...@cs.ucla.edu>> wrote: On 02/15/2013 07:10 AM, Michael Goffioul wrote: > I guess the only solution to this problem is to always _putenv and not manipulate "environ" directly.

secure_getenv declaration error

2013-02-09 Thread John W. Eaton
After updating gnulib in the Octave sources, I started seeing the following error: In file included from /usr/include/c++/4.7/cstdlib:66:0, from /usr/include/c++/4.7/bits/stl_algo.h:61, from /usr/include/c++/4.7/algorithm:63, from /home/jw

Re: Re: problem with unistd.h on mingw systems

2013-01-17 Thread John W. Eaton
On 01/16/2013 06:49 PM, Paul Eggert wrote: Thanks, I pushed this: Thanks for the quick response. Do you expect this to continue to work for older versions of mingw? I don't care about that myself, I'm just wondering whether that is something gnulib should do. Or is the target for gnulib l

problem with unistd.h on mingw systems

2013-01-16 Thread John W. Eaton
While trying to build Octave for mingw, I hit the following error: ../../liboctave/system/oct-env.cc: In member function 'std::string octave_env::do_get_host_name() const': ../../liboctave/system/oct-env.cc:455:20: error: 'gethostname' is not a member of 'gnulib' ../../liboctave/system/oct

Re: Problem with stdbool.h redefining true/false even in C++

2012-08-29 Thread John W. Eaton
On 29-Aug-2012, Bastien ROUCARIES wrote: | n Wed, Aug 29, 2012 at 7:36 PM, John W. Eaton wrote: | > On 29-Aug-2012, Paul Eggert wrote: | > | > | On 08/29/2012 10:00 AM, John W. Eaton wrote: | > | > Why define true, false, and bool to anything when using C++? | > | | > | M

Re: Problem with stdbool.h redefining true/false even in C++

2012-08-29 Thread John W. Eaton
On 29-Aug-2012, Paul Eggert wrote: | On 08/29/2012 10:00 AM, John W. Eaton wrote: | > Why define true, false, and bool to anything when using C++? | | Maybe it's for '#ifdef bool' but to be honest I'm just doing what | GCC does. Perhaps you can ask the GCC developers wh

Re: Problem with stdbool.h redefining true/false even in C++

2012-08-29 Thread John W. Eaton
On 29-Aug-2012, John W. Eaton wrote: | Michael, how is stdbool.h being included in Octave C++ sources? Maybe | that's a mistake that should be fixed. Oh, I see it ends up being included in data.cc because we include base64.h from gnulib there. jwe

Re: Problem with stdbool.h redefining true/false even in C++

2012-08-29 Thread John W. Eaton
On 29-Aug-2012, Michael Goffioul wrote: | On Wed, Aug 29, 2012 at 4:44 PM, Michael Goffioul | wrote: | | Thanks, that should indeed redefining true and false... | | | I meant: that should indeed avoid redefining true and false... Why define true, false, and bool to anything when using C++

apparent complexity of mkdir-p module (was: Re: dirchownmod and savewd modules uses unavailable functions under) Windows

2012-06-07 Thread John W. Eaton
On 7-Jun-2012, Paul Eggert wrote: | Much of gnulib has never been ported to MSVC / MingW | and apparently octave is using that part. I suggest | using Cygwin. If it were up to me, I'd go a step further and say, "don't use Windows," but the reality is that that's not going to fly with many of Oc

Re: Ordering of "-I" flags in CPPFLAGS can lead to gnulib headers being skipped

2012-02-23 Thread John W. Eaton
On 23-Feb-2012, Paul Eggert wrote: | On 02/23/2012 02:08 AM, Michael Goffioul wrote: | > 2) in liboctave/Makefile.am, @CPPFLAGS@ should be moved to the end, or | > after "-I../libgnu" | | That's what I'd do. Seems reasonable to me. I'm not sure why I put @CPPFLAGS@ first. I'll make this change.

[bug #35580] autogen.sh is not portable to OpenBSD

2012-02-21 Thread John W. Eaton
Follow-up Comment #5, bug #35580 (project octave): OK. You aren't running autogen.sh for a released version of Octave, are you? It is not needed except to build sources from the mercurial archive. If you have a tarball release, you just need to use the included configure script. __

[bug #35580] autogen.sh is not portable to OpenBSD

2012-02-21 Thread John W. Eaton
Follow-up Comment #3, bug #35580 (project octave): The error is also coming from the bootstrap script, which we should be using unmodified from gnulib. So the right place to fix this is in gnulib so that all projects using gnulib will get the fix, not just Octave. ___

Re: #define FOO rpl_FOO in config.h

2011-11-30 Thread John W. Eaton
On 29-Nov-2011, Eric Blake wrote: | On 11/29/2011 01:01 AM, John W. Eaton wrote: | > While trying to cross compile Octave for MinGW, I hit the following | > errors: | > | > In file included from /home/jwe/src/octave/liboctave/oct-time.h:26:0, | >from /home

#define FOO rpl_FOO in config.h

2011-11-29 Thread John W. Eaton
While trying to cross compile Octave for MinGW, I hit the following errors: In file included from /home/jwe/src/octave/liboctave/oct-time.h:26:0, from /home/jwe/src/octave/liboctave/file-stat.h:28, from /home/jwe/src/octave/liboctave/file-ops.cc:43: /usr/i

Re: Dependencies not compiled in

2011-08-26 Thread John W. Eaton
On 26-Aug-2011, Michael Goffioul wrote: | On Fri, Aug 26, 2011 at 1:44 PM, Bruno Haible wrote: | > Hi, | > | > Michael Goffioul wrote: | >> While compiling Octave, I got undefined references within the gnulib | >> module. | >> Namely: | >> - chdir_long undefined, referenced in save-cwd.c | > | >

Unable to override value of AR set by gnulib macro?

2011-08-25 Thread John W. Eaton
GNU Octave's configure script used to allow users to set AR using ./configure AR=... but that stopped working recently. It looks like the problem is the following change to gnulib: http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=2b14869442bce932cf1d805387928484d4914d59 Since this c

Re: fseek failing without fflush?

2011-08-09 Thread John W. Eaton
On 9-Aug-2011, Eric Blake wrote: | I'm thinking this may be a bug in gnulib. I also know that glibc 2.14 | fixed one fclose() bug in how fclose() relates to fflush(), but at the | expense of introducing another, and that gnulib does not yet detect and | work around that glibc bug. I'm assumi

fseek failing without fflush?

2011-08-09 Thread John W. Eaton
I discovered the following surprising behavior recently when adding a few more gnulib:: namespace tags to the Octave sources. Eventually I was able to generate the relatively small example attached below that shows the same behavior and is independent of Octave. The problem seems to be that fseek

Re: Fortran95 binding for posix/gnulib

2011-07-20 Thread John W. Eaton
On 3-Jul-2011, Bruno Haible wrote: | subdirectory modules/fortran/ -, and that you follow the GNU conventions for | Fortran code (whatever these may be - you can find out by looking at octave | in octave-3.4.2/libcruft/ [3]). I wouldn't recommend trying to follow the style of the Fortran code in

RE : Re: Fortran95 binding for posix/gnulib

2011-07-20 Thread John W. Eaton
On 20-Jul-2011, Bastien ROUCARIES wrote: | I agree libcraft is really ugly. | | Could you review the binding ? I don't use Fortran much these days. I don't write anything new using it, so I'm probably not the right person to review it. | Do you have use in octave? Not really. The Fortran cod

Re: cloning gnulib with --depth=2 option hangning

2011-04-21 Thread John W. Eaton
On 21-Apr-2011, Bruno Haible wrote: | Hi John, | | John W. Eaton wrote: | > When I execute | > | > git clone --depth=2 git://git.sv.gnu.org/gnulib | > | > as is done when running the bootstrap script for the first time, the | > git process hangs. | | Yes, for me too it h

cloning gnulib with --depth=2 option hangning

2011-04-21 Thread John W. Eaton
When I execute git clone --depth=2 git://git.sv.gnu.org/gnulib as is done when running the bootstrap script for the first time, the git process hangs. Several people have reported this problem when building Octave from a Mercurial checkout, which does not start out with a gnulib subdirectory b

cloning gnulib with --depth=2 option hangning

2011-04-21 Thread John W. Eaton
When I execute git clone --depth=2 git://git.sv.gnu.org/gnulib as is done when running the bootstrap script for the first time, the git process hangs. Several people have reported this problem when building Octave from a Mercurial checkout, which does not start out with a gnulib subdirectory b

[PATCH] doc: update users.txt

2011-02-10 Thread John W. Eaton
On 10-Feb-2011, Jim Meyering wrote: | FYI, another "user": You could also add Octave to that list with the URL http://hg.savannah.gnu.org/hgweb/octave. Thanks, jwe

Problem with including

2011-01-27 Thread John W. Eaton
On 27-Jan-2011, Benjamin Lindner wrote: | So currently, it would require unistd.h to be either included twice, | or be included before winsock2.h to work properly. I think it would be OK if we had to include unistd.h before winsock2.h. However, I did not know that was a requirement. If it is th

Re: default gnulib_path in build-aux/bootstrap script

2011-01-14 Thread John W. Eaton
On 14-Jan-2011, Jim Meyering wrote: | John W. Eaton wrote: | | > I updated Octave's copy of the bootstrap script from gnulib's version | > today and hit an error because gnulib_path was set to ''. The code in | > the bootstrap script for this is | > | > git_

default gnulib_path in build-aux/bootstrap script

2011-01-14 Thread John W. Eaton
I updated Octave's copy of the bootstrap script from gnulib's version today and hit an error because gnulib_path was set to ''. The code in the bootstrap script for this is git_modules_config () { test -f .gitmodules && git config --file .gitmodules "$@" } gnulib_path=`git_modules_conf

filemode with C++

2011-01-13 Thread John W. Eaton
Hi, I'd like to use the filemode module in a C++ file but found that it does not have extern "C" magic. I started to submit a patch, but stopped when I saw that filemode.h has # if HAVE_DECL_STRMODE # include /* FreeBSD, OpenBSD */ # include /* NetBSD */ # else void strmode (mode_t

Re: fnmatch sometimes defined to posix_fnmatch

2011-01-09 Thread John W. Eaton
On 10-Jan-2011, Bruno Haible wrote: | These are simply the old replacement idioms that are not C++ safe. | We introduced the C++ safe idioms for most header files in March 2010, | but the old idioms are still used in a couple of places: | | m4/eaccess.m4:[AC_DEFINE([eaccess], [access], | m4/f

fnmatch sometimes defined to posix_fnmatch

2011-01-09 Thread John W. Eaton
I received a bug report for Octave that was the result of fnmatch being defined to posix_fnmatch in config.h. I think this is due to AC_DEFINE_UNQUOTED([fnmatch], [${gl_fnmatch_required_lowercase}_fnmatch], [Define to a replacement function name for fnmatch().]) in gnulib's fnmatch.m4

copyright year ranges and README.gnulib

2011-01-01 Thread John W. Eaton
On 1-Jan-2011, Karl Berry wrote: | Subject: Re: [PATCH 1/2] maint: new rule to update copyright year ranges | | This brings up a related topic that belatedly crossed my mind recently. | rms stated an extra requirement of making a statement in the README | about copyright ranges when they are

Re: on msvc

2010-04-27 Thread John W. Eaton
On 28-Apr-2010, Bruno Haible wrote: | The way to do that is that you provide a good implementation of said | functions opendir, readdir, and closedir. Sure, I understand that. Sorry if my earlier message sounded like I was expecting someone else to do this part. | Projects that use gnulib have

Re: [PATCH] ftruncate: mark module as obsolete; even MinGW provides it, now

2010-04-27 Thread John W. Eaton
On 9-Apr-2010, Paolo Bonzini wrote: | On 04/09/2010 11:04 AM, Bruno Haible wrote: | > Indeed. But since mingw has it but MSVC doesn't, this raises the | > question: how important is the MSVC porting platform (use Microsoft's | > compiler and include files [proprietary but downloadable at zero cos

Re: [PATCH] ftruncate: mark module as obsolete; even MinGW provides it, now

2010-04-09 Thread John W. Eaton
On 9-Apr-2010, Jim Meyering wrote: | Paolo Bonzini wrote: | > On 04/09/2010 12:48 PM, Jim Meyering wrote: | >> Of course, if you have some precise -- and useful enough to count as a | >> reasonable porting target -- development environment in mind for which | >> this particular replacement is req

#defines in unistd.h

2010-03-23 Thread John W. Eaton
The current gnulib unistd.in.h file includes the following section: #if @GNULIB_GETHOSTNAME@ /* Get all possible declarations of gethostname(). */ # if @UNISTD_H_HAVE_WINSOCK2_H@ # if !defined _GL_SYS_SOCKET_H # undef socket # define socket socket_used_without_inclu

Re: status of new 'c++defs' module?

2010-03-16 Thread John W. Eaton
An additional problem showed up on OS X systems: libtool: compile: g++-4.2 -DHAVE_CONFIG_H -I. -I.. -I/sw/lib/flex/include -I/sw/include -O0 -g -m32 -I/sw/include/freetype2 -I/sw/include/qhull -I/usr/include -I../libgnu -I../libgnu -I../libcruft/misc -I../liboctave -I../liboctave -I. -I. -I/

Re: status of new 'c++defs' module?

2010-03-16 Thread John W. Eaton
On 13-Mar-2010, Bruno Haible wrote: | Good point. I'm adding this as an extra check in the testsuite: Thanks for the additional changes. I modified the Octave sources so that GNULIB_NAMESPACE is defined to gnulib and tagged all the uses that were reported by GCC warnings. Everything worked fine

Re: new module 'c++defs'

2010-03-07 Thread John W. Eaton
On 6-Mar-2010, Bruno Haible wrote: | John W. Eaton wrote: | > How can one easily find all the places where the GNULIB_NAMESPACE tag | > is needed? Is there some way we can get the compiler to help with | > this job? | | Good point. I'm adding a macro _GL_CXXALIASWARN, so that

status of new 'c++defs' module?

2010-03-02 Thread John W. Eaton
Is there any reason not to commit the changes for the new c++defs module? Earlier, I wrote: | How can one easily find all the places where the GNULIB_NAMESPACE tag | is needed? Is there some way we can get the compiler to help with | this job? | | The reason I liked the idea of having the gnuli

Re: new module 'c++defs'

2010-02-23 Thread John W. Eaton
On 23-Feb-2010, Eric Blake wrote: | According to John W. Eaton on 2/23/2010 7:51 AM: | > The only other error I'm seeing is | > | > libtool: compile: g++ -DHAVE_CONFIG_H -I. -I/home/jwe/src/octave/liboctave -I.. -I../libgnu -I/home/jwe/src/octave/libgnu -I/home/jwe/src/octave

Re: new module 'c++defs'

2010-02-23 Thread John W. Eaton
On 23-Feb-2010, Bruno Haible wrote: | John W. Eaton wrote: | | > The next problem I encountered was | > | > libtool: compile: g++ -DHAVE_CONFIG_H -I. -I/home/jwe/src/octave/liboctave -I.. -I../libgnu -I/home/jwe/src/octave/libgnu -I/home/jwe/src/octave/libcruft/misc -ggdb3 -DHAVE

Re: new module 'c++defs'

2010-02-22 Thread John W. Eaton
On 23-Feb-2010, Bruno Haible wrote: | John W. Eaton wrote: | > ../libgnu/stdio.h:1087: error: previous declaration of 'int _gl_cxxalias_dummy' with 'C++' linkage | > ../libgnu/signal.h:246: error: conflicts with new declaration with 'C' linkage | | Thank

new module 'c++defs'

2010-02-22 Thread John W. Eaton
On 22-Feb-2010, Bruno Haible wrote: | Here come the proposed changes to allow C++ developers to choose among | the default mode with many | #define func rpl_func | macro definitions and a "namespace mode", which is more in line with the | way C++ programs are built. | | I handled all header fil

Re: gnulib and C++

2010-02-20 Thread John W. Eaton
On 20-Feb-2010, Bruno Haible wrote: | And when I change the first line to | | namespace system { | #include | } | using namespace system; | | then there is a different error message: | | $ g++ -S foo.cc | foo.cc: In function ‘int bar_uses_gnulib_implicitly()’: | foo.cc:37: error: reference to

Re: gnulib and C++

2010-02-19 Thread John W. Eaton
On 19-Feb-2010, I wrote: | To handle all the cases, I find that I need to write | | #ifdef __cplusplus | namespace gnulib | { | int (*const open) (const char *filename, int flags, ...) = | # if @GNULIB_OPEN@ | # if @REPLACE_OPEN@ | ::rpl_open; | # else | ::open; |

Re: gnulib and C++

2010-02-19 Thread John W. Eaton
On 18-Feb-2010, Bruno Haible wrote: | Ralf proposes: | > I suggest you could | > produce a helper header to #undef them again; you could even do that | > automatically during bootstrap with above. Then only a portability | > layer would need to avoid using these symbols in another namespace. | |

Re: gnulib and C++

2010-02-18 Thread John W. Eaton
On 18-Feb-2010, Ralf Wildenhues wrote: | * John W. Eaton wrote on Thu, Feb 18, 2010 at 09:22:39PM CET: | > On 18-Feb-2010, Ralf Wildenhues wrote: | > | $ autoconf '--trace=AC_DEFINE:$1:$2' | grep ':rpl_[^:]*$' | > | > Running this with the Octave

Re: gnulib and C++

2010-02-18 Thread John W. Eaton
On 18-Feb-2010, Ralf Wildenhues wrote: | Hello John, | | * John W. Eaton wrote on Thu, Feb 18, 2010 at 08:37:30PM CET: | > However, there is one issue that has caused a bit of trouble as I | > have started using more gnulib modules. | > | > Octave is mostly written in C++ and am

gnulib and C++

2010-02-18 Thread John W. Eaton
Hi, I'm trying to use gnulib with GNU Octave and am overall very pleased, as it is allowing me to remove a lot of cruft from Octave's source files and just assume that a POSIX environment is available without having to use a lot of #ifdefs. To me, that's a significant improvement. However, there

Re: missing dependencies?

2010-02-08 Thread John W. Eaton
On 7-Feb-2010, Bruno Haible wrote: | John W. Eaton wrote: | > and I've received reports about the following undefined references | > when linking: | > | > .libs/liboctave_la-lo-cutils.o:lo-cutils.c:(.text+0x4f): undefi

missing dependencies?

2010-02-06 Thread John W. Eaton
I have the following modules listed in the bootstrap.conf file for Octave: # gnulib modules used by this package. gnulib_modules=" c-strcase crypto/md5fcntl fnmatch getcwdgethostname getopt-gnu gettimeofday glob link lstat mkdir mkfifo s

Re: glob on Windows systems

2010-01-13 Thread John W. Eaton
On 9-Jan-2010, Bruno Haible wrote: | Hi John, | | John W. Eaton wrote: | > Looking at lib/glob.c in the gnulib sources, there is some | > Windows-specific code, so it looks like it is intended to work on | > Windows systems | | > Is glob.c expected to work on Windows systems? | |

extern "C" for glob.h

2010-01-07 Thread John W. Eaton
the attached patch. Thanks, jwe >From 1641dbca79baee5b57a3bc691f2050d3da6dd135 Mon Sep 17 00:00:00 2001 From: John W. Eaton Date: Thu, 7 Jan 2010 13:08:33 -0500 Subject: [PATCH] glob.in.h: define __BEGIN_DECLS and __END_DECLS for C++ --- ChangeLog |5 + lib/glob.in.h |

extern "C" for wctype.h?

2010-01-06 Thread John W. Eaton
#include statement in the Octave sources with extern "C". jwe >From 9c5eb4db02eaae3b8d2dc65a2a2c7e42b8aa8694 Mon Sep 17 00:00:00 2001 From: John W. Eaton Date: Thu, 7 Jan 2010 02:03:16 -0500 Subject: [PATCH] wctype.in.h: add extern "C" block for C++ --- ChangeLog

glob on Windows systems

2010-01-06 Thread John W. Eaton
Looking at lib/glob.c in the gnulib sources, there is some Windows-specific code, so it looks like it is intended to work on Windows systems, but there are some things that don't look quite right. For example, it does not seem to uniformly use backslash as a directory separator, and if WINDOWS32 i

getlogin on Windows systems

2010-01-06 Thread John W. Eaton
I'm using the glob module in GNU Octave and a Windows user complained that linking failed with an undefined reference to getlogin. The problem appears to be that getlogin_r is called from glob, and the gnulib replacement for getlogin_r calls getlogin unconditionally, but getlogin doesn't exist on

should mkdir depend on xalloc?

2010-01-05 Thread John W. Eaton
Although lib/mkdir.c includes xalloc.h, the mkdir module doesn't depend on xalloc. Should it? If so, how about the following change? Thanks, jwe >From 78ed1c4cd10f5ef982ab09a2f19d31a786bf4cbe Mon Sep 17 00:00:00 2001 From: John W. Eaton Date: Tue, 5 Jan 2010 03:50:42 -0500 Subject

Re: Recognize Mercurial in build-aux/bootstrap

2009-11-14 Thread John W. Eaton
On 14-Nov-2009, Jim Meyering wrote: | John W. Eaton wrote: | > Would the following patch be OK for recognizing Mercurial in the | > build-aux/bootstrap script? It seems to work for me. Maybe there is | > a better way of recognizing that a file is tracked, but I'm not sure

Re: extern "C" for strftime.h?

2009-11-13 Thread John W. Eaton
On 14-Nov-2009, Jim Meyering wrote: | John W. Eaton wrote: | > Is there any reason that the strftime.h header in gnulib does not use | > extern "C"? If not, then would the following patch be OK? | | No one asked for it. | I've pushed this: Thanks! jwe

Recognize Mercurial in build-aux/bootstrap

2009-11-13 Thread John W. Eaton
n FILE does not exist. jwe commit 17efaa64f21d1834ab75904bc423a77b098c670d Author: John W. Eaton Date: Sat Nov 14 02:26:15 2009 -0500 build-aux/bootstrap: Recognize Mercurial for version control. diff --git a/build-aux/bootstrap b/build-aux/bootstrap index 8ac13b2..799ad68 100755

extern "C" for strftime.h?

2009-11-13 Thread John W. Eaton
Is there any reason that the strftime.h header in gnulib does not use extern "C"? If not, then would the following patch be OK? Thanks, jwe commit bf7fb16a0253f59bb1ec31e61bc92b1574e78633 Author: John W. Eaton Date: Sat Nov 14 00:02:23 2009 -0500 strftime.h: wrap funtion decl