[libvirt] [PATCH] util: Add win32 version of virFileUnlink

[Martin Kletzander]

On Tue, Sep 08, 2015 at 09:12:11AM -0400, John Ferlan wrote:
>
>
>On 09/07/2015 04:25 AM, Martin Kletzander wrote:
>> Commit 35847860f65f Added the virFileUnlink function, but failed to add
>> a version for mingw build, causing the following error:
>>
>>   Cannot export virFileUnlink: symbol not defined
>>
>> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
>> ---
>>  src/util/virfile.c | 14 ++++++++++++++
>>  1 file changed, 14 insertions(+)
>>
>
>Ugh... Sorry about this one... Too bad there wasn't a way to have some
>sort of make check rule - it wasn't the first and won't be the last time
><sigh> ;-)
>

That's not easy without having such build.  I'm sure there are many of
those as the matrix of #ifs and #ifndefs etc. grows exponentially.  If
we had some kind of check for that, it would take way too much time.
And the mingw build is still broken because other things I'm trying to
investigate, so don't worry about that ;)

>John
>> diff --git a/src/util/virfile.c b/src/util/virfile.c
>> index 408d2d912f13..75819d9c8bd7 100644
>> --- a/src/util/virfile.c
>> +++ b/src/util/virfile.c
>> @@ -2637,6 +2637,20 @@ virDirCreate(const char *path ATTRIBUTE_UNUSED,
>>
>>      return -ENOSYS;
>>  }
>> +
>> +int
>> +virFileUnlink(const char *path,
>> +              uid_t uid ATTRIBUTE_UNUSED,
>> +              gid_t gid ATTRIBUTE_UNUSED)
>> +{
>> +    if (unlink(path) < 0) {
>> +        virReportSystemError(errno, _("Unable to unlink path '%s'"),
>> +                             path);
>> +        return -1;
>> +    }
>> +
>> +    return 0;
>> +}
>>  #endif /* WIN32 */
>>
>>  /**
>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150908/06adf2bc/attachment-0001.sig>