[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v2 1/5] crypto/secret: rename to secret_interface.
From: |
Daniel P . Berrangé |
Subject: |
Re: [RFC PATCH v2 1/5] crypto/secret: rename to secret_interface. |
Date: |
Wed, 22 Apr 2020 16:32:33 +0100 |
User-agent: |
Mutt/1.13.3 (2020-01-12) |
On Thu, Apr 16, 2020 at 01:25:21AM +0300, Alexey Krasikov wrote:
> * Rename for future division into subclasses. Most part of the interface
> will remain in basic common class.
You don't need to put bullet points in the commit message, just
have the text.
>
> Signed-off-by: Alexey Krasikov <address@hidden>
> ---
> crypto/{secret.c => secret_interface.c} | 0
> include/crypto/{secret.h => secret_interface.h} | 0
> 2 files changed, 0 insertions(+), 0 deletions(-)
> rename crypto/{secret.c => secret_interface.c} (100%)
> rename include/crypto/{secret.h => secret_interface.h} (100%)
This breaks the build because Makefile.objs doesn't reference
the new filename, and likewise other files doing #include
don't work.
I don't think renaming actually makes sense in the first place,
because you then add the original files back again in a later
patch.
You need to just have a patch which introduces secret_interface.{ch}
without killing the original secret.{c,h} entirely. The key point is
that QEMU must successfully compile on each individual patch in the
series, otherwise it breaks "git bisect" usage.
Also, since the object is called "SecretCommon",the filenames
should match that "secret_common.{ch}"
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [RFC PATCH v2 1/5] crypto/secret: rename to secret_interface., Alexey Krasikov, 2020/04/15
- [RFC PATCH v2 5/5] test-crypto-secret: add 'syskey' object tests., Alexey Krasikov, 2020/04/15
- [RFC PATCH v2 4/5] crypto/linux_keyring: add 'syskey' secret object., Alexey Krasikov, 2020/04/15
- [RFC PATCH v2 2/5] crypto/secret_interface: conversion to common basic class., Alexey Krasikov, 2020/04/15
- [RFC PATCH v2 3/5] crypto/secret: add secret class files., Alexey Krasikov, 2020/04/15
- Re: [RFC PATCH v2 1/5] crypto/secret: rename to secret_interface., Markus Armbruster, 2020/04/16
- Re: [RFC PATCH v2 1/5] crypto/secret: rename to secret_interface.,
Daniel P . Berrangé <=