[PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
4 index 000..e353ebf --- /dev/null +++ b/lib/fsync.c @@ -0,0 +1,62 @@ +/* Emulate fsync on platforms which lack it, primarily Windows and + cross-compilers like MinGW. + + This is derived from sqlite3 sources and is in the public domain. + + Written by Richard W.M. Jones +*/ + +#include

Re: [PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
On Wed, Oct 01, 2008 at 06:06:31PM +0200, Jim Meyering wrote: > To make gnulib-tool's --with-tests option work, you can add > this to your change set: OK, I tried your suggested patch and the tests passed under Linux and under the Fedora cross-compiler / Wine environment. I'm unable to test it on

Re: [PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora >From 2326ca168fa4826a64d933f3491ca94be6808f9b Mon Sep 17 00:00:00 2001 From: Richard W.M. Jones <[EMAIL PROTECTED]> Date: Wed, 1 Oct 2008 13:54:44 +0100 Subject: [PATCH] fsync implementation. --- M

Re: [PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora >From a1552e630163ceadd92728ec10339c7411191e33 Mon Sep 17 00:00:00 2001 From: Richard W.M. Jones <[EMAIL PROTECTED]> Date: Wed, 1 Oct 2008 13:54:44 +0100 Subject: [PATCH] fsync imple

Re: [PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
On Wed, Oct 01, 2008 at 05:51:23PM +, Eric Blake wrote: > Richard W.M. Jones redhat.com> writes: > > > > > Like the earlier patch, but this relicenses to LGPLv2+, includes > > license text, and assigns everything to the FSF. > > Do you have copyright on

Re: [PATCH] Implementation of fsync for Windows

2008-10-01 Thread Richard W.M. Jones
I have a general question: Does Gnulib get any sort of nightly build/test? On [real] Windows machines? And a general comment: Some work colleagues and I are working on a Windows cross-compiler project for Fedora[1]. The number one big porting problem has been lack of flock/lockf/fcntl file locki

Preliminary patch for flock (just for discussion)

2008-10-02 Thread Richard W.M. Jones
out even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program; if not, write to the Free Software Foundation, +

Re: Preliminary patch for flock (just for discussion)

2008-10-03 Thread Richard W.M. Jones
On Fri, Oct 03, 2008 at 02:25:42AM +0200, Bruno Haible wrote: > Richard W.M. Jones wrote: > > here is a preliminary patch for flock support for Windows. > > Three comments. > > * About the function. Why flock() and not lockf()? > There are three APIs for locking: &

Re: Preliminary patch for flock (just for discussion)

2008-10-03 Thread Richard W.M. Jones
On Fri, Oct 03, 2008 at 02:19:05PM +0200, Bruno Haible wrote: > Very good point. You can write flock() in terms of fcntl()? Also, in glibc > we have code for lockf() in terms of fcntl. I've written an flock-in-terms-of-fcntl in the latest version. Coming soon ... I'm working around some tricky b

[PATCH] flock implementation (version 2)

2008-10-03 Thread Richard W.M. Jones
/dev/null +++ b/lib/flock.c @@ -0,0 +1,244 @@ +/* Emulate flock on platforms that lack it, primarily Windows and MinGW. + + This is derived from sqlite3 sources. + http://www.sqlite.org/cvstrac/rlog?f=sqlite/src/os_win.c + http://www.sqlite.org/copyright.html + + Written by Richard

Re: [PATCH] flock implementation (version 2)

2008-10-03 Thread Richard W.M. Jones
disk stats, logging, etc. http://et.redhat.com/~rjones/virt-top >From 2f5d884b626a6410662826d0748f03ba297e76b2 Mon Sep 17 00:00:00 2001 From: Richard W.M. Jones <[EMAIL PROTECTED]> Date: Fri, 3 Oct 2008 14:58:14 +0100 Subject: [PATCH] Implement flock. * MODULES.html.sh: Add to list of mod

Re: [PATCH] random_r: new module

2008-10-23 Thread Richard W.M. Jones
On Wed, Oct 22, 2008 at 10:06:28PM +0200, Jim Meyering wrote: > I saw that there was no random-number generating module in gnulib, > and certainly nothing reentrant, so took glibc's random_r.c and did this: I need random() and drand48() to port some Unix program to Windows. Win32 doesn't have eith

[PATCH] Implementations of random, srandom, initstate, setstate, rand, srand

2008-10-23 Thread Richard W.M. Jones
VE_SETENV=1; AC_SUBST([HAVE_SETENV]) diff --git a/modules/random b/modules/random new file mode 100644 index 000..ab33752 --- /dev/null +++ b/modules/random @@ -0,0 +1,25 @@ +Description: +random number generator + +Files: +lib/random.c +m4/random.m4 + +Depends-on: +stdlib +rand

[PATCH] New module rand48

2008-10-23 Thread Richard W.M. Jones
BST([GNULIB_RANDOM_R]) GNULIB_RPMATCH=0; AC_SUBST([GNULIB_RPMATCH]) GNULIB_SETENV=0;AC_SUBST([GNULIB_SETENV]) @@ -42,6 +43,7 @@ AC_DEFUN([gl_STDLIB_H_DEFAULTS], HAVE_MALLOC_POSIX=1;AC_SUBST([HAVE_MALLOC_POSIX]) HAVE_MKDTEMP=1; AC_SUBST([HAVE_MKDTEMP]) HAVE_REALLOC_

[PATCH] Compile popt under Windows using gnulib

2008-10-25 Thread Richard W.M. Jones
This patch allows popt from CVS to be cross-compiled for Windows using MinGW and Win32 API (not Cygwin). It uses Gnulib to supply the missing functionality -- mainly glob(3) and globfree(3). Note that the patch isn't "complete". As you can see from the modification to autogen.sh you are expecte

Re: [PATCH] Compile popt under Windows using gnulib

2008-10-25 Thread Richard W.M. Jones
On Sat, Oct 25, 2008 at 11:31:49AM -0400, Jeff Johnson wrote: > FYI: patches to <[EMAIL PROTECTED]>, please. I'll take > patches however they are sent however. *shrug* > > I can't tie builds of popt to gnulib, even for cross-compiles, there's > way too many > projects that depend on popt. Gnulib

[PATCH] Implementations of random, srandom, initstate, setstate, rand, srand. [rebased]

2008-10-29 Thread Richard W.M. Jones
_SETENV]) diff --git a/modules/random b/modules/random new file mode 100644 index 000..ab33752 --- /dev/null +++ b/modules/random @@ -0,0 +1,25 @@ +Description: +random number generator + +Files: +lib/random.c +m4/random.m4 + +Depends-on: +stdlib +random_r + +configure.ac: +gl_FUNC_RANDOM +gl_STDL

[PATCH] New module rand48 [rebased]

2008-10-29 Thread Richard W.M. Jones
AULTS], HAVE_MALLOC_POSIX=1; AC_SUBST([HAVE_MALLOC_POSIX]) HAVE_MKDTEMP=1;AC_SUBST([HAVE_MKDTEMP]) HAVE_REALLOC_POSIX=1; AC_SUBST([HAVE_REALLOC_POSIX]) + HAVE_RAND48=1; AC_SUBST([HAVE_RAND48]) HAVE_RANDOM_R=1; AC_SUBST([HAVE_RANDOM

Re: [PATCH] Implementations of random, srandom, initstate, setstate, rand, srand

2008-10-29 Thread Richard W.M. Jones
On Wed, Oct 29, 2008 at 11:22:08AM +0100, Jim Meyering wrote: [random, random_r, RAND_MAX] > However, what if an application wants only random_r? > We shouldn't require that they import all of the non-thread-safe > functions, too, so I think random really does belong separate. > Of course, it depen

Re: [PATCH] New module rand48 [rebased]

2008-10-29 Thread Richard W.M. Jones
Sorry, ignore this one. I realized that Ralf Wildenhues sent some feedback which I haven't incorporated. I'll post an updated patch in a moment. Rich. -- Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones Read my OCaml programming blog: http://camltastic.blogspot.com/

Re: [PATCH] New module rand48 [rebased]

2008-10-29 Thread Richard W.M. Jones
B_SETENV=0;AC_SUBST([GNULIB_SETENV]) @@ -45,6 +46,7 @@ AC_DEFUN([gl_STDLIB_H_DEFAULTS], HAVE_MALLOC_POSIX=1; AC_SUBST([HAVE_MALLOC_POSIX]) HAVE_MKDTEMP=1;AC_SUBST([HAVE_MKDTEMP]) HAVE_REALLOC_POSIX=1; AC_SUBST([HAVE_REALLOC_POSIX]) + HAVE_RAND48=1; AC_SUBST([H

Gauge interest in an XDR implementation

2008-10-29 Thread Richard W.M. Jones
I maintain portableXDR (http://et.redhat.com/~rjones/portablexdr/) which is an XDR library for Windows and Macs. As far as I'm aware it is only used by libvirt (http://libvirt.org/), and only on platforms which lack any XDR library (ie. Windows) or a usable XDR library (Macs and other Unix systems

undefined reference to `_rpl_optarg' / '_rpl_optind'

2008-10-31 Thread Richard W.M. Jones
Does anyone recognize this error at all, perhaps from an older version of gnulib? (See also the follow-up comments). https://bugzilla.redhat.com/show_bug.cgi?id=467398#c1 Rich. -- Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones Read my OCaml programming blog: http:

Re: undefined reference to `_rpl_optarg' / '_rpl_optind'

2008-10-31 Thread Richard W.M. Jones
On Fri, Oct 31, 2008 at 11:18:30AM +, Richard W.M. Jones wrote: > Does anyone recognize this error at all, perhaps from an older version > of gnulib? (See also the follow-up comments). > > https://bugzilla.redhat.com/show_bug.cgi?id=467398#c1 Sorry for the noise -- ign

Re: [PATCH] Implementations of random, srandom, initstate, setstate, rand, srand

2008-11-06 Thread Richard W.M. Jones
On Tue, Nov 04, 2008 at 03:26:55AM +0100, Bruno Haible wrote: > Hi, Hi - sorry, I went on holiday just after you posted this, hence the late response ... > Richard W.M. Jones wrote: > > This patch adds a 'random' module which implements: > > > > - random

Re: fix test-flock.c

2008-11-15 Thread Richard W.M. Jones
On Sat, Nov 15, 2008 at 01:33:33PM +0100, Bruno Haible wrote: > Very clear: It uses O_RDWR but does not include . Also it lacks > an include of , needed for all code that uses gnulib. Fixing it > like this: Yup, looks like a clear fix. Rich. -- Richard Jones, Emerging Technologies, Red Hat htt

Re: Request: srand48/ drand48

2009-03-23 Thread Richard W.M. Jones
On Sun, Mar 22, 2009 at 10:48:51PM +0100, Jim Meyering wrote: > Stefan Bienert wrote: > > I need a fallback implementation of srand48 and drand48 for my GPL project > > to compile with mingw32 for a certain operating system. > > > > Is there a chance that this can be put into a near-by-future revis

Re: [Libguestfs] Hivex licensing question

2010-11-26 Thread Richard W.M. Jones
On Fri, Nov 26, 2010 at 11:01:53AM -0800, Yandell, Henri wrote: > Taking that as a hint not to go charging off onto bug-gnulib@, and looking > at the bug-gnulib archive, Bruno Haible says in > http://www.mail-archive.com/bug-gnulib@gnu.org/msg20323.html: > > "The header is updated automatically wh

Re: [PATCH 1/2] pread: new module

2009-11-25 Thread Richard W.M. Jones
I'm getting the following error when cross-compiling with --host=i686-pc-mingw32: make[4]: Entering directory `/home/rjones/d/libguestfs-mingw/daemon/lib' GENconfigmake.h CC pread.o pread.c:34: error: conflicting types for 'pread' ./unistd.h:705: note: previous declaration of

gettime.c:36: undefined reference to `clock_gettime'

2009-11-25 Thread Richard W.M. Jones
Latest gnulib from git, Fedora 12 (64 bit) host: lib/libgnu.a(gettime.o): In function `gettime': /home/rjones/d/libguestfs/daemon/lib/gettime.c:37: undefined reference to `clock_gettime' I spent a little bit of time trying to work out what you'd have to link against in order to get this symbol,

Re: gettime.c:36: undefined reference to `clock_gettime'

2009-11-25 Thread Richard W.M. Jones
On Wed, Nov 25, 2009 at 04:54:21PM +, Richard W.M. Jones wrote: > Latest gnulib from git, Fedora 12 (64 bit) host: > > lib/libgnu.a(gettime.o): In function `gettime': > /home/rjones/d/libguestfs/daemon/lib/gettime.c:37: undefined reference to > `clock_gettime' Fa

[PATCH] set_winsock_errno

2009-11-26 Thread Richard W.M. Jones
Currently any socket functions that are replaced by Gnulib on Win32 call set_winsock_errno. This function reads the error from winsock (WSAGetLastError) and sets errno to some corresponding Unix-ish approximation. The vast majority of functions (non-socket ones) still require you to deal with Ge

Re: [PATCH] set_winsock_errno

2009-11-26 Thread Richard W.M. Jones
On Thu, Nov 26, 2009 at 08:22:47PM +0100, Paolo Bonzini wrote: > On 11/26/2009 05:52 PM, Richard W.M. Jones wrote: >> >> Currently any socket functions that are replaced by Gnulib on Win32 >> call set_winsock_errno. This function reads the error from winsock >> (WSAGetL

Re: [PATCH] set_winsock_errno

2009-11-26 Thread Richard W.M. Jones
On Thu, Nov 26, 2009 at 08:29:38PM +0100, Paolo Bonzini wrote: >> No I don't - can you apply it for me? > > Done. Excellent, thanks. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones New in Fedora 11: Fedora Windows cross-compiler. Compile Windows programs,

Error handling in Win32 + gnulib

2009-11-27 Thread Richard W.M. Jones
I'm confused by errno and error handling in gnulib, and what the strategy for this is supposed to be, versus how it seems to work now. Win32 has two methods to return an error code from a syscall: DWORD WSAGetLastError(void); // for all socket functions DWORD GetLastError(); // for al

Re: Error handling in Win32 + gnulib

2009-11-30 Thread Richard W.M. Jones
e correct me because this is the most important bit of this email. Any deviation from the policy above would be a bug. Bruno Haible wrote: > Richard W.M. Jones wrote: > > error codes aren't compatible. > > Although Gnulib tries to define its own error numbers which are > &g

Re: Error handling in Win32 + gnulib

2009-11-30 Thread Richard W.M. Jones
On Mon, Nov 30, 2009 at 09:16:04AM +, Richard W.M. Jones wrote: > My take from this is that we need a 'set_errno' function which is Jim pointed out to me that calling it 'set_errno' is likely to be confused with the glibc macro __set_errno. Call it 'win32_set_err

Re: Error handling in Win32 + gnulib

2009-11-30 Thread Richard W.M. Jones
On Fri, Nov 27, 2009 at 03:57:13PM +, Richard W.M. Jones wrote: > Win32 itself doesn't use or define a variable called 'errno', but > gnulib does provide this variable when linked to Win32 programs. This statement is inaccurate: Win32 does define errno, but only uses it f

Re: [PATCH] xstrtoll: new module

2010-01-25 Thread Richard W.M. Jones
On Sat, Jan 23, 2010 at 12:13:51PM +0100, Jim Meyering wrote: > diff --git a/lib/xstrtol.h b/lib/xstrtol.h > index 95475f0..3a94a9c 100644 > --- a/lib/xstrtol.h > +++ b/lib/xstrtol.h > @@ -46,6 +46,11 @@ _DECLARE_XSTRTOL (xstrtoul, unsigned long int) > _DECLARE_XSTRTOL (xstrtoimax, intmax_t) > _D

Re: [PATCH] xstrtoll: new module

2010-01-25 Thread Richard W.M. Jones
On Mon, Jan 25, 2010 at 12:39:02PM +0100, Jim Meyering wrote: > Richard W.M. Jones wrote: > > On Sat, Jan 23, 2010 at 12:13:51PM +0100, Jim Meyering wrote: > >> diff --git a/lib/xstrtol.h b/lib/xstrtol.h > >> index 95475f0..3a94a9c 100644 > >> --- a/li

Gnulib setenv on Mac OS X bug?

2010-03-21 Thread Richard W.M. Jones
There seems to be a bug in Gnulib's setenv module on Mac OS X. At configure time it says: checking whether setenv validates arguments ... no (The test program it is running is: | int | main () | { | |if (setenv ("", "", 0) != -1) return 1; |if (errno != EINVAL) return 2; |

Re: Gnulib setenv on Mac OS X bug?

2010-03-21 Thread Richard W.M. Jones
On Sun, Mar 21, 2010 at 11:18:16PM +0100, Bruno Haible wrote: > If you do > $ grep LIBOBJ config.status > you should see something like this: > S["gl_LTLIBOBJS"]=" setenv.lo" > S["gl_LIBOBJS"]=" setenv.o" $ grep LIBOBJ config.status | grep -i setenv s,@gltests_LIBOBJS@,|#_!!_#| getugroups.o

Re: Gnulib setenv on Mac OS X bug?

2010-03-21 Thread Richard W.M. Jones
On Sun, Mar 21, 2010 at 11:24:38PM +0100, Bruno Haible wrote: > Richard W.M. Jones wrote: > > (The test program it is running is: > > > > | int > > | main () > > | { > > | > > |if (setenv ("", "", 0) != -1) return 1; > &

Re: Gnulib setenv on Mac OS X bug?

2010-03-21 Thread Richard W.M. Jones
On Mon, Mar 22, 2010 at 12:04:58AM +0100, Bruno Haible wrote: > This is all as expected. Now you have to turn to your Makefile, and see > whether these gl_LIBOBJS are correctly reflected in that Makefile, and > why the setenv.o is not being built. Part of the reason why setenv.o isn't being built

test-getlogin_r segfaults with glibc 2.12

2010-05-05 Thread Richard W.M. Jones
$ EXEEXT='' srcdir='.' LOCALE_FR='fr_FR' LOCALE_FR_UTF8='fr_FR.UTF-8' LOCALE_JA='ja_JP' LOCALE_ZH_CN='zh_CN.GB18030' LOCALE_FR_UTF8='fr_FR.UTF-8' LOCALE_FR='fr_FR' LOCALE_FR_UTF8='fr_FR.UTF-8' LOCALE_JA='ja_JP' LOCALE_ZH_CN='zh_CN.GB18030' gdb --args ./test-getlogin_r GNU gdb (GDB) Fedora (7.0-

Re: test-getlogin_r segfaults with glibc 2.12

2010-05-05 Thread Richard W.M. Jones
On Wed, May 05, 2010 at 04:35:49PM +0100, Richard W.M. Jones wrote: > Program received signal SIGSEGV, Segmentation fault. > 0x77ae77fe in __strncpy_sse2 (s1=, > s2=, n=) at ./strncpy.c:83 > 83 *++s1 = '\0'; > (gdb) bt > #0 0x77ae77fe in __

gettext 0.18 grumbles

2010-06-03 Thread Richard W.M. Jones
I just updated gnulib in libguestfs. This pulls in gettext 0.18 -- I don't directly depend on the gettext module in gnulib, but doubtless it's being pulled in as a dependency of something. There's no gettext 0.18 in Fedora, even in Rawhide, and it seems non-trivial to build. My workaround has b

Re: gettext 0.18 grumbles

2010-06-03 Thread Richard W.M. Jones
On Thu, Jun 03, 2010 at 08:13:32PM +0200, Paolo Bonzini wrote: > There aren't many modules that depend on gettext > > $ grep -x gettext * > acl:gettext > bison-i18n:gettext > quotearg-tests:gettext The issue in libguestfs seems to be quotearg-tests, pulled in via quotearg, and quotearg being pulle

Re: gettext 0.18 grumbles

2010-06-03 Thread Richard W.M. Jones
On Thu, Jun 03, 2010 at 08:40:40PM +0200, Bruno Haible wrote: > Non-trivial to build? It is essential for GNU packages to be buildable > with "configure && make && make install". Did you try that? It does have > some dependencies, but you should be able to install these dependencies > with a packag

Re: gettext 0.18 grumbles

2010-06-03 Thread Richard W.M. Jones
On Thu, Jun 03, 2010 at 08:10:47PM +0100, Richard W.M. Jones wrote: > On Thu, Jun 03, 2010 at 08:40:40PM +0200, Bruno Haible wrote: > > Non-trivial to build? It is essential for GNU packages to be buildable > > with "configure && make && make install"

Re: source builds vs. RPMs (was: Re: gettext 0.18 grumbles)

2010-06-04 Thread Richard W.M. Jones
On Thu, Jun 03, 2010 at 10:04:28PM +0200, Bruno Haible wrote: > Let me repeat it: We have GNU standards that guarantee you that > ./configure > make > make install > must work everywhere. This *is* the "easy install" that anyone must be > able to use. > > Why did you choose to ignore the con

Re: gettext 0.18 grumbles

2010-06-04 Thread Richard W.M. Jones
On Thu, Jun 03, 2010 at 02:29:44PM -0600, Eric Blake wrote: > On 06/03/2010 12:59 PM, Jim Meyering wrote: > > Richard W.M. Jones wrote: > >> I just updated gnulib in libguestfs. This pulls in gettext 0.18 -- I > >> don't directly depend on the gettext module in

Re: source builds vs. RPMs

2010-06-04 Thread Richard W.M. Jones
On Fri, Jun 04, 2010 at 11:05:56AM +0200, Bruno Haible wrote: > Where does the thinking come from that "there must be only one instance > of every package"? From Microsoft Windows? I've hardly ever used MS Windows, but I do know they have no packaging system to speak of and often install multiple,

./getopt.h:195: error: redefinition of 'struct option'

2010-06-04 Thread Richard W.M. Jones
make[4]: Entering directory `/home/rjones/d/libguestfs/gnulib/lib' CC xstrtol.lo In file included from xstrtol.h:23, from xstrtol.c:32: ./getopt.h:195: error: redefinition of 'struct option' In file included from xstrtol.h:23, from xstrtol.c:32: ./getopt.h:24

Re: ./getopt.h:195: error: redefinition of 'struct option'

2010-06-04 Thread Richard W.M. Jones
On Fri, Jun 04, 2010 at 02:17:22PM +0200, Bruno Haible wrote: > Hi Richard, > > Last time we saw this error message, it was due to a wrong config.h. The results below are after 'rm config.h; ./configure ...; make clean' > Richard W.M. Jones wrote: > > make[4]: Enteri

Re: ./getopt.h:195: error: redefinition of 'struct option'

2010-06-04 Thread Richard W.M. Jones
On Fri, Jun 04, 2010 at 03:16:38PM +0200, Bruno Haible wrote: > The first one is fine. The second one should not be there. Yes, I see now that gnulib/lib/getopt.h is present on the machine which fails and absent on the other one. > Conclusion: You had a getopt.h that was necessary for one platfo

Re: ./getopt.h:195: error: redefinition of 'struct option'

2010-06-04 Thread Richard W.M. Jones
Indeed 'make distclean' does fix this. Sorry for the noise .. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 80 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_st

test-stdalign.c breaks on i686 on Fedora Rawhide (gcc 4.7)

2012-01-26 Thread Richard W.M. Jones
CC test-stdalign.o test-stdalign.c:70:1: error: static assertion failed: "verify (alignof (int64_t) == offsetof (int64_t_helper, slot2))" test-stdalign.c:73:1: error: static assertion failed: "verify (alignof (double) == offsetof (double_helper, slot2))" make[4]: *** [test-stdalign.o] Error

Re: test-stdalign.c breaks on i686 on Fedora Rawhide (gcc 4.7)

2012-01-27 Thread Richard W.M. Jones
On Fri, Jan 27, 2012 at 12:57:20AM +0100, Bruno Haible wrote: > Richard W.M. Jones wrote: > > CC test-stdalign.o > > Please, can you show the entire compiler command? Run "make V=1" so that > 'make' prints the real command. Sure, it's: depbase=

gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
With gnulib from around May 25th, I get this error when compiling on RHEL 5. bash ../../libtool-kill-dependency_libs.sh ../../libtool --tag=CC --mode=link gcc -g -O2 -o test-futimens test-futimens.o libtests.a ../../gnulib/lib/libgnu.la libtests.a -lrt gcc -g -O2 -o test-futimens test-fut

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 10:26:57AM +0100, Richard W.M. Jones wrote: > > With gnulib from around May 25th, I get this error when compiling > on RHEL 5. > > bash ../../libtool-kill-dependency_libs.sh ../../libtool --tag=CC --mode=link > gcc -g -O2 -o test-futimens test-fut

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 11:53:01AM +0200, Jim Meyering wrote: > Richard W.M. Jones wrote: > > With gnulib from around May 25th, I get this error when compiling > > on RHEL 5. > > > > bash ../../libtool-kill-dependency_libs.sh ../../libtool --tag=CC > > --mode=link

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 12:09:41PM +0200, Jim Meyering wrote: > That suggests that it linked. > > In what context did you see the link failure? > While running tests for some other package? Yes, this is the attempt to get libguestfs working again on RHEL 5. It could also be because of 'libtool-k

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 12:33:36PM +0200, Jim Meyering wrote: > [libtool-kill-dependency_libs.sh] > That looks promising. > Can you easily refrain from using that when building gnulib's own tests? I've removed that now. Still happens. However I think this could be a bug in libtool (from RHEL 5).

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 11:44:08AM +0100, Richard W.M. Jones wrote: > On Mon, Jun 18, 2012 at 12:33:36PM +0200, Jim Meyering wrote: > > [libtool-kill-dependency_libs.sh] > > That looks promising. > > Can you easily refrain from using that when building gnulib's own tes

Re: gnulib/tests/utimecmp.c:183: undefined reference to `hash_insert'

2012-06-18 Thread Richard W.M. Jones
On Mon, Jun 18, 2012 at 12:01:01PM +0100, Richard W.M. Jones wrote: > On Mon, Jun 18, 2012 at 11:44:08AM +0100, Richard W.M. Jones wrote: > > On Mon, Jun 18, 2012 at 12:33:36PM +0200, Jim Meyering wrote: > > > [libtool-kill-dependency_libs.sh] > > > That looks prom

test-getlogin_r is annoyingly random

2012-07-19 Thread Richard W.M. Jones
When building in Koji (the Fedora build system), the test is usually skipped: Skipping test: stdin is not a tty. SKIP: test-getlogin_r except when it doesn't skip and instead it fails: test-getlogin_r.c:62: assertion failed /bin/sh: line 5: 11074 Aborted EXEEXT='' srcdir

Re: test-getlogin_r is annoyingly random

2012-07-19 Thread Richard W.M. Jones
Here is the complete log from a failed build: http://koji.fedoraproject.org/koji/getfile?taskID=4253206&name=build.log The only other thing of note is that this is glibc 2.16-2, and it should of course already have getlogin_r so we shouldn't be using the gnulib replacement function. Rich. -- R

Re: test-getlogin_r is annoyingly random

2012-07-19 Thread Richard W.M. Jones
On Thu, Jul 19, 2012 at 01:04:16PM +0200, Jim Meyering wrote: > A quick workaround is to use gnulib's --avoid option > to tell it you don't want the getlogin_r tests: > > --avoid=getlogin_r-tests Thanks. I pushed this workaround upstream: https://github.com/libguestfs/libguestfs/commit/6e1b

test-cloexec.c:97: assertion failed

2012-10-08 Thread Richard W.M. Jones
With Linux 3.7.0 I'm seeing: make[4]: Entering directory `/home/rjones/d/libguestfs/gnulib/tests' PASS: test-accept PASS: test-accept4 PASS: test-alloca-opt PASS: test-argmatch PASS: test-arpa_inet PASS: test-binary-io.sh PASS: test-bind PASS: test-bitrotate PASS: test-btowc1.sh PASS: test-btowc2

Re: test-cloexec.c:97: assertion failed

2012-10-08 Thread Richard W.M. Jones
Turning off coredumps reveals a load more problems. I wonder if the latest glibc in Fedora Rawhide is broken? test-dup2.c:173: assertion failed # ASSERT (!is_inheritable (fd + 1)); test-dup3.c:108: assertion failed # ASSERT (dup3 (fd, fd, o_flags) == -1); test-fcntl.c:291: assertion fai

Re: test-cloexec.c:97: assertion failed

2012-10-08 Thread Richard W.M. Jones
On Mon, Oct 08, 2012 at 07:53:26PM +0200, Jim Meyering wrote: > Richard W.M. Jones wrote: > > Turning off coredumps reveals a load more problems. I wonder if the > > latest glibc in Fedora Rawhide is broken? > > > > test-dup2.c:173: assertion failed > > #

Re: test-cloexec.c:97: assertion failed

2012-10-08 Thread Richard W.M. Jones
On Mon, Oct 08, 2012 at 10:50:30PM +0100, Richard W.M. Jones wrote: [.. discussion on gnulib test-cloexec test snipped ..] > I'm suspicious this is a kernel bug: > > creat("test-cloexec.tmp", 0600) = 3 > fcntl(3, F_GETFD)

Re: [regression] F_DUPFD_CLOEXEC breakage fix

2012-10-09 Thread Richard W.M. Jones
On Mon, Oct 08, 2012 at 11:21:58PM +0100, Al Viro wrote: > On Mon, Oct 08, 2012 at 10:53:25PM +0100, Richard W.M. Jones wrote: > > On Mon, Oct 08, 2012 at 10:50:30PM +0100, Richard W.M. Jones wrote: > > [.. discussion on gnulib test-cloexec test snipped ..] > > > I'

test-dup3.c:108: assertion failed (was: Re: test-cloexec.c:97: assertion failed)

2012-10-09 Thread Richard W.M. Jones
The F_DUPFD_CLOEXEC fix that Al Viro posted fixes 3/4 of the bugs, but I'm still investigating this one: > test-dup3.c:108: assertion failed > # ASSERT (dup3 (fd, fd, o_flags) == -1); Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones New in Fedora 11: F

Re: test-dup3.c:108: assertion failed (was: Re: test-cloexec.c:97: assertion failed)

2012-10-09 Thread Richard W.M. Jones
On Tue, Oct 09, 2012 at 09:54:45AM +0100, Richard W.M. Jones wrote: > The F_DUPFD_CLOEXEC fix that Al Viro posted fixes 3/4 of the > bugs, but I'm still investigating this one: > > > test-dup3.c:108: assertion failed > > # ASSERT (dup3 (fd, fd, o_flags) == -1);

Re: test-dup3.c:108: assertion failed (was: Re: test-cloexec.c:97: assertion failed)

2012-10-09 Thread Richard W.M. Jones
I'm testing the following patch. Rich. From 0944e30e12dec6544b3602626b60ff412375c78f Mon Sep 17 00:00:00 2001 From: "Richard W.M. Jones" Date: Tue, 9 Oct 2012 14:42:45 +0100 Subject: [PATCH] dup3: Return an error when oldfd == newfd. The following co

Re: test-dup3.c:108: assertion failed

2012-10-09 Thread Richard W.M. Jones
On Tue, Oct 09, 2012 at 08:18:21AM -0600, Eric Blake wrote: > On 10/09/2012 03:05 AM, Richard W.M. Jones wrote: > > On Tue, Oct 09, 2012 at 09:54:45AM +0100, Richard W.M. Jones wrote: > >> The F_DUPFD_CLOEXEC fix that Al Viro posted fixes 3/4 of the > >> bugs, but I

Another test-dup3 assertion with the new kernel

2012-10-10 Thread Richard W.M. Jones
Fixing the other two kernel bugs, leaves me with two further assertions. This time I'm not so sure that this is a kernel bug. /* The destination must be valid. */ errno = 0; ASSERT (dup3 (fd, -2, o_flags) == -1); ASSERT (errno == EBADF); <--- here

Re: Another test-dup3 assertion with the new kernel

2012-10-10 Thread Richard W.M. Jones
On Wed, Oct 10, 2012 at 11:05:54AM -0600, Eric Blake wrote: > On 10/10/2012 07:23 AM, Richard W.M. Jones wrote: > > > > Fixing the other two kernel bugs, leaves me with two further > > assertions. This time I'm not so sure that this is a kernel bug. > > >

Request to relax the licenses of quotearg, quotearg-simple, xalloc, xalloc-die, xalloc-oversized to LGPLv2+

2012-10-17 Thread Richard W.M. Jones
libguestfs (an LGPLv2+ library) would like to use the useful quotearg functions to ensure we can run system(3)-style external commands while safely quoting shell arguments. However this would require relaxation of the license to LGPLv2+ for at least the following modules [*]: quotearg quotear

Rawhide error test-fcntl-h.c:25:47: error: '__O_SYNC' undeclared here (not in a function)

2012-11-02 Thread Richard W.M. Jones
On current Rawhide: CC test-fcntl-h.o test-fcntl-h.c:25:47: error: '__O_SYNC' undeclared here (not in a function) I suspect this is actually a glibc bug. The gnulib source uses O_SYNC and doesn't mention __O_SYNC at all. Rich. -- Richard Jones, Virtualization Group, Red Hat http://peo

test-getaddrinfo failures in Fedora Rawhide

2013-04-23 Thread Richard W.M. Jones
Not looked into this in any detail yet, but just a note that the getaddrinfo test is now failing in Fedora Rawhide. This failure started to happen within the last week. system error: Connection refused system error: Connection refused system error: Connection refused system error: Connec

Request to relicense hash gnulib module to LGPLv2+

2013-08-28 Thread Richard W.M. Jones
libguestfs (an LGPLv2+ library) uses the 'hash' module, which turns out to be "GPL". Actually this happened because we started to use it in a separate GPL'd utility program, but later on included this functionality in the core library, copying the same code from the utility but not checking the li

Re: relicense module spawn-pipe

2013-08-29 Thread Richard W.M. Jones
It's easier to see your patches if you use the git send-email command. It has a long and horrible command line syntax so what people usually do is wrap it up in a shell script. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Fedora Windows cross-compiler.

Re: Request to relicense hash gnulib module to LGPLv2+

2013-09-09 Thread Richard W.M. Jones
On Wed, Aug 28, 2013 at 06:51:11PM +0100, Richard W.M. Jones wrote: > libguestfs (an LGPLv2+ library) uses the 'hash' module, which turns > out to be "GPL". > > Actually this happened because we started to use it in a separate > GPL'd utility program, but

Re: manywarnings: -Walloc-size-larger-than & 32bit architectures

2017-07-31 Thread Richard W.M. Jones
On Mon, Jul 31, 2017 at 02:15:16PM +0200, Pino Toscano wrote: > Hi, > > when using the warnings module, after commit > 5e22aee79f9d02ac37f40f1d18f5696114c3c3c9 also > -Walloc-size-larger-than=9223372036854775807 is added to the CFLAGS, > if supported (which seems the case with newer GCC versions).

Re: fseeko broken by Fedora rawhide glibc / git master (ie future 2.28)

2018-03-07 Thread Richard W.M. Jones
On Wed, Mar 07, 2018 at 03:23:10PM +, Daniel P. Berrangé wrote: > On Wed, Mar 07, 2018 at 09:05:59AM -0600, Eric Blake wrote: > > On 03/05/2018 01:25 PM, Paul Eggert wrote: > > > On 03/05/2018 11:20 AM, Daniel P. Berrangé wrote: > > > > Yes, this worked on rawhide when I tested with libvirt. >

Testing gnulib on riscv64 - replacement modules

2018-03-08 Thread Richard W.M. Jones
Platform: riscv64-unknown-linux-gnu gnulib @ db0b059ae4f489d0c83b785b259eb965a181d0d0 (yesterday) https://sourceware.org/glibc/wiki/Testing/Gnulib says: > "Verify that Gnulib has built no replacement/workaround code > (gllib/*.o files)" $ (cd gllib; ls -1 $(${GNULIB_CHECKOUT}/posix-modules | se

Re: Testing gnulib on riscv64 - replacement modules

2018-03-08 Thread Richard W.M. Jones
Attached are the results of the test suite. I also have core dumps from all the failing tests, but unfortunately we don't have 'gdb' working yet so I can't easily convert them into a stack trace. However most of the failures appear to be assert-fails so it doesn't look as if a stack trace would

Re: Testing on riscv64

2018-03-14 Thread Richard W.M. Jones
On Wed, Mar 14, 2018 at 12:09:40PM +0100, Bruno Haible wrote: > Hi Richard, > > > Platform: riscv64-unknown-linux-gnu > > gnulib @ db0b059ae4f489d0c83b785b259eb965a181d0d0 (yesterday) > > Prompted by your mail, I'm trying to run a Linux/riscv64 machine in a qemu > VM, in order to look at the gnul

Re: Testing on riscv64

2018-03-14 Thread Richard W.M. Jones
On Wed, Mar 14, 2018 at 12:09:40PM +0100, Bruno Haible wrote: > -append "console=ttyS0 ro root=/dev/vda init=/init" \ Oh I see, the problem is the init=... See https://fedorapeople.org/groups/risc-v/disk-images/readme.txt for the latest and correct instructions. I tried qemu from upstream no

Re: Testing gnulib on riscv64 - replacement modules

2018-03-14 Thread Richard W.M. Jones
On Wed, Mar 14, 2018 at 10:24:27PM +0100, Bruno Haible wrote: > FAIL: test-expf > FAIL: test-fmod > FAIL: test-hypot > FAIL: test-sqrt > FAIL: test-sqrtf > > Various floating-point issues, possibly due to the soft-float emulation. > > FAIL: test-strtod > > This one could be more worrisome. I now

Testing gnulib on riscv64 - replacement modules (update)

2018-05-16 Thread Richard W.M. Jones
Back in March I was trying to test gnulib on RISC-V using qemu: https://lists.gnu.org/archive/html/bug-gnulib/2018-03/threads.html#00022 We now have real hardware so I am trying to do the tests again, I still get stuck on the "replacement modules" step in the instructions. "Verify that Gnulib

Silent test failures when no French, Japanese, Chinese locales installed

2018-11-15 Thread Richard W.M. Jones
I'm using gnulib 6ccfbb4ce5d4fa79f7afb48f3648f2e0401523c3 from a few days ago. After upgrading to Fedora 29, the glibc-langpack-fr locale is no longer installed by default. This causes many tests to fail silently: ./gnulib/tests/test-suite.log:FAIL: test-btowc1.sh ./gnulib/tests/test-suite.log:F

Re: Silent test failures when no French, Japanese, Chinese locales installed

2018-11-15 Thread Richard W.M. Jones
On Thu, Nov 15, 2018 at 11:03:59AM +, Richard W.M. Jones wrote: > I'm using gnulib 6ccfbb4ce5d4fa79f7afb48f3648f2e0401523c3 from a > few days ago. > > After upgrading to Fedora 29, the glibc-langpack-fr locale is no > longer installed by default. This causes many te

Re: Silent test failures when no French, Japanese, Chinese locales installed

2018-11-16 Thread Richard W.M. Jones
On Fri, Nov 16, 2018 at 12:04:36AM +0100, Bruno Haible wrote: > Hi Richard, > > > After upgrading to Fedora 29, the glibc-langpack-fr locale is no > > longer installed by default. This causes many tests to fail silently: > > > > ./gnulib/tests/test-suite.log:FAIL: test-btowc1.sh > > ./gnulib/tes

Re: Silent test failures when no French, Japanese, Chinese locales installed

2018-11-16 Thread Richard W.M. Jones
On Fri, Nov 16, 2018 at 08:38:16AM +, Richard W.M. Jones wrote: > > $ grep LOCALE_FR config.status > > > > S["LOCALE_FR_UTF8"]="fr_FR.UTF-8" > > S["LOCALE_FR"]="fr_FR" > > Since I'm using gnulib from libgues

git://git.savannah.gnu.org/gnulib.git down for anyone?

2018-12-11 Thread Richard W.M. Jones
$ git pull fatal: read error: Connection reset by peer Does anyone know if this is a temporary service interruption or if the URL changed? Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.

Re: git://git.savannah.gnu.org/gnulib.git down for anyone?

2018-12-11 Thread Richard W.M. Jones
On Tue, Dec 11, 2018 at 01:33:52PM -0600, Eric Blake wrote: > On 12/11/18 1:20 PM, Richard W.M. Jones wrote: > > > >$ git pull > >fatal: read error: Connection reset by peer > > > >Does anyone know if this is a temporary service interruption or if the > >URL

  1   2   >