On Tue, Mar 15, 2011 at 10:06:23AM -0400, Laine Stump wrote:
> On 03/15/2011 08:32 AM, Daniel P. Berrange wrote:
> >* configure.ac: Check for geteuid()
> >* src/util/util.h: Compat for geteuid()
> >---
> > configure.ac | 2 +-
> > src/util/util.h | 4 ++++
> > 2 files changed, 5 insertions(+), 1 deletions(-)
> >
> >diff --git a/configure.ac b/configure.ac
> >index a58ee4e..e2b2b24 100644
> >--- a/configure.ac
> >+++ b/configure.ac
> >@@ -120,7 +120,7 @@ AC_MSG_RESULT([$have_cpuid])
> > dnl Availability of various common functions (non-fatal if missing),
> > dnl and various less common threadsafe functions
> > AC_CHECK_FUNCS_ONCE([cfmakeraw regexec sched_getaffinity getuid getgid \
> >- initgroups posix_fallocate mmap kill \
> >+ geteuid initgroups posix_fallocate mmap kill \
> > getmntent_r getgrnam_r getpwuid_r])
> >
> > dnl Availability of pthread functions (if missing, win32 threading is
> >diff --git a/src/util/util.h b/src/util/util.h
> >index 31c3a33..276e259 100644
> >--- a/src/util/util.h
> >+++ b/src/util/util.h
> >@@ -269,6 +269,10 @@ const char *virEnumToString(const char *const*types,
> > static inline int getuid (void) { return 0; }
> > # endif
> >
> >+# ifndef HAVE_GETUID
>
> This should be HAVE_GETEUID, not HAVE_GETUID ;-)
Yes, I'll push without the stupid typo !
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list