Daniel P. Berrangé wrote:
> > For what purpose is libvirt or QEMU using setgroups()? What goes wrong if
> > setgroups() fails?
>
> QEMU potentially needs access to files owned by a supplementary group.
> On Linux for example, /dev/kvm is often owned by 'kvm' group, but the
> 'qemu' user on Fedora
Daniel P. Berrangé wrote:
> > > FWIW I compiled libvirt without the setgroups code on Mac and it
> > > worked as expected. Not sure what the implications of that are though?
> >
> > OK, then the fix would be to not use setgroups on Mac, and nothing to do
> > in gnulib. Right?
>
> Not calling
Marcus Furlong wrote:
> FWIW I compiled libvirt without the setgroups code on Mac and it
> worked as expected. Not sure what the implications of that are though?
OK, then the fix would be to not use setgroups on Mac, and nothing to do
in gnulib. Right?
Bruno
--
libvir-list mailing list
Hi Eric,
> I wonder if gnulib could provide a workaround setgroups() that overcomes
> this issue
I don't see how a workaround could look like. The problem is not the value
of NGROUPS_MAX in user-space, but the same value NGROUPS_MAX in the kernel.
More precisely, in the Darwin kernel file
Daniel P. Berrange wrote:
> Tested-by: Daniel P. Berrange <berra...@redhat.com>
>
>
> Confirmed it fixes the failure on Fedora 28, and does not cause a regression
> on Fedora 26 with older glibc.
Thanks. Pushing it:
2017-10-09 Bruno Haible <br...@clisp.org>
Daniel P. Berrange wrote:
> From my own F28 rawhide install with glibc-2.26.90-16.fc28.x86_64
>
> >
> > 1) The output of
> > $ nm test-getopt-posix | grep getopt
>
> $ nm test-getopt-posix | grep getopt
> U getopt@@GLIBC_2.2.5
> 00400ab0 t getopt_loop.constprop.0
Hi,
Christian Ehrhardt wrote:
> > Fedora 26 only has glibc 2.25 - you need to have Fedora rawhide to get
> > the broken behaviour, as that has glibc 2.26.90
> As Daniel said at least glibc 2.26 as in Fedora rawhide or Ubuntu Artful.
This tip is not helpful: I spent two hours trying Fedora
Daniel P. Berrange wrote:
Try applying this patch to your source tree
diff --git a/configure.ac b/configure.ac
index 6b189db..4f906bb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2876,6 +2876,10 @@ test x$lv_cv_static_analysis = xyes t=1
AC_DEFINE_UNQUOTED([STATIC_ANALYSIS], [$t],
Daniel P. Berrange wrote:
If its better to just do it in libvirt config.h, then we
can do that too
Yes, doing '#define foo libvirt_foo' in config.h is the preferred way
of achieving a namespace clean shared library.
There are two ways to generate these #defines:
1) You collect manually, on
Hi Paul,
Daniel P. Berrange wrote:
While on x86_64-w64-mingw32 only I see:
In file included from ../../../gnulib/lib/regex.c:69:0:
../../../gnulib/lib/regcomp.c: In function 'parse_dup_op':
../../../gnulib/lib/regcomp.c:2624:39: warning: cast to pointer from integer
of different size
Hello Daniel,
CC fstat.lo
../../../gnulib/lib/fstat.c:27:0: warning: stat redefined [enabled by
default]
In file included from ./sys/stat.h:32:0,
from ../../../gnulib/lib/fstat.c:25:
/usr/x86_64-w64-mingw32/sys-root/mingw/include/sys/stat.h:258:0: note: this
is the
Hi Eric,
case 'v':
-/* FIXME - list a copyright blurb, as in GNU programs? */
-puts(VERSION);
+vshShowVersion(ctl);
exit(EXIT_SUCCESS);
The gnulib module closeout can automatically take care of this, but it
is currently
Hi Eric,
after looking into this deeper, it (happily) appears that I
may have been mistaken. It looks like the gnulib vasprintf module
_already_ performs printf parsing on mingw; and that as a virtue of that
printf parsing, %zu and %llu are already rewritten into modifiers
understood by
[removed bug-gnulib from CC]
Eric Blake wrote:
printf (% PRIuPTR, (uintptr_t) size_t_value);
But still needs the use of need-formatstring-macros, and gettext 0.16.1
or newer (whereas libvirt is stuck on 0.14.1 at the moment).
The uses of these formatstring macros require
- at package
Hi Eric,
My understanding is that on 64-bit windows,
sizeof(long)==4 but sizeof(void*)==8; and ... sizeof(size_t) is also 8.
Yes, correct.
Which means you _can't_ use %lu,(unsigned long)size_t_val.
You _can_ use this. It will work as long as each of your program's data
structures is less
Eric Blake wrote:
it still requires auditing code
and forbidding %zu in favor of %PRIuSIZE (or whatever other name we
settle on).
You could hack GCC, clang, or even xgettext to produce a warning when it
sees a 'z' size modifier in a format string.
Bruno
--
libvir-list mailing list
[This question ought to have been sent to bug-gettext, not bug-gnulib.]
Hi Eric,
Use of PRIuMAX in a translated string produces a .pot file that
refers to the special string PRIuMAX, and which requires the
need-formatstring-macros option passed to AM_GNU_GETTEXT in configure.ac
to be
Eric Blake wrote:
Here's where a cross-project change to GNU Coding Standards could be
helpful - if we all agree that gnulib should add the macro and gettext
should add the support for it at the same time, then it would be much
easier for all remaining GNU projects to take advantage of a
What a shame that POSIX omitted an inttypes.h PRIu* for size_t.
You can define it by yourself ...
Or use uintptr_t instead of size_t. By the definition of these types,
you can be sure that uintptr_t is at least as wide as size_t. Then use
printf (% PRIuPTR, (uintptr_t) size_t_value);
-
Jim Meyering wrote:
Thank you for the patch. That is indeed a typo.
I've attached a patch adding a ChangeLog entry,
for application to gnulib's git repository.
Thanks, applied.
[Bruno, to apply this, you can remove the two '' and then
run git am THIS_MESSAGE. ]
Which '' should I remove?
20 matches
Mail list logo