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 <alex-krasi...@yandex-team.ru> > --- > 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 :|