Re: [PATCH v4] util: Add virGettextInitialize, convert the code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/14/2016 01:18 PM, John Ferlan wrote:
> 
> 
> On 04/13/2016 08:21 AM, Cole Robinson wrote:
>> Take setlocale/gettext error handling pattern from tools/virsh-*
>> and use it for all standalone binaries via a new shared
>> virGettextInitialize routine. The virsh* pattern differed slightly
>> from other callers. All users now consistently:
>>
>> * Ignore setlocale errors. virsh has done this forever, presumably for
>>   good reason. This has been partially responsible for some bug reports:
>>
>>   https://bugzilla.redhat.com/show_bug.cgi?id=1312688
>>   https://bugzilla.redhat.com/show_bug.cgi?id=1026514
>>   https://bugzilla.redhat.com/show_bug.cgi?id=1016158
>>
>> * Report the failed function name
>> * Report strerror
>> ---
>> v4:
>>     sigh, forgot to squash in a cfg.mk change
>>
>>  cfg.mk                        | 13 +++++++++-
>>  daemon/libvirtd.c             |  6 ++---
>>  src/Makefile.am               |  2 ++
>>  src/libvirt_private.syms      |  4 ++++
>>  src/locking/lock_daemon.c     |  6 ++---
>>  src/locking/sanlock_helper.c  |  9 ++-----
>>  src/logging/log_daemon.c      |  6 ++---
>>  src/lxc/lxc_controller.c      |  6 ++---
>>  src/network/leaseshelper.c    | 12 +++-------
>>  src/security/virt-aa-helper.c | 12 +++-------
>>  src/storage/parthelper.c      |  9 ++-----
>>  src/util/iohelper.c           | 13 +++-------
>>  src/util/virgettext.c         | 56 +++++++++++++++++++++++++++++++++++++++++++
>>  src/util/virgettext.h         | 25 +++++++++++++++++++
>>  tools/virsh.c                 | 15 ++----------
>>  tools/virt-admin.c            | 15 ++----------
>>  tools/virt-host-validate.c    | 15 ++----------
>>  tools/virt-login-shell.c      | 14 ++---------
>>  tools/vsh.c                   |  2 --
>>  19 files changed, 128 insertions(+), 112 deletions(-)
>>  create mode 100644 src/util/virgettext.c
>>  create mode 100644 src/util/virgettext.h
>>
> 
> Minor nit in virgettext.c below
> 
> ACK -
> 
> John
> 
> [...]
> 
>> +++ b/src/util/virgettext.c
>> @@ -0,0 +1,56 @@
>> +/*
>> + * virgettext.c: gettext helper routines
>> + *
>> + * Copyright (C) 2016 Red Hat, Inc.
>> + *
>> + * This library is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU Lesser General Public
>> + * License as published by the Free Software Foundation; either
>> + * version 2.1 of the License, or (at your option) any later version.
>> + *
>> + * This library is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>> + * Lesser General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU Lesser General Public
>> + * License along with this library.  If not, see
>> + * <http://www.gnu.org/licenses/>.
>> + */
>> +
>> +#include <config.h>
>> +
>> +#include <locale.h>
>> +#include <stdio.h>
>> +
>> +#include "configmake.h"
>> +#include "internal.h"
>> +#include "virgettext.h"
>> +
>> +
>> +/**
>> + * virGettextInit:
> 
> ^^  virGettextInitialize

Good catch :) Fixed and pushed

Thanks,
Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]