On 07/09/2020 21.43, Philippe Mathieu-Daudé wrote: > On 9/7/20 7:42 PM, Thomas Huth wrote: >> Now that we always do out-of-tree builds (and the in-tree builds are >> faked via a "build" directory), we can simplify out .gitignore file >> quite a bit. >> >> Signed-off-by: Thomas Huth <th...@redhat.com> >> --- >> .gitignore | 158 ----------------------------------------------------- >> 1 file changed, 158 deletions(-) >> >> diff --git a/.gitignore b/.gitignore >> index 4ccb9ed975..bb916594eb 100644 >> --- a/.gitignore >> +++ b/.gitignore >> @@ -1,165 +1,7 @@ >> /GNUmakefile >> /build/ >> -/.doctrees >> -/config-devices.* >> -/config-all-devices.* >> -/config-all-disas.* >> -/config-host.* >> -/config-target.* >> -/config.status >> -/config-temp >> -/tools/virtiofsd/50-qemu-virtiofsd.json >> -/elf2dmp >> -/trace-events-all >> -/trace/generated-events.h >> -/trace/generated-events.c >> -/trace/generated-helpers-wrappers.h >> -/trace/generated-helpers.h >> -/trace/generated-helpers.c >> -/trace/generated-tcg-tracers.h >> -/ui/shader/texture-blit-frag.h >> -/ui/shader/texture-blit-vert.h >> -/ui/shader/texture-blit-flip-vert.h >> -/ui/input-keymap-*.c.inc >> -*-timestamp >> -/*-softmmu >> -/*-darwin-user >> -/*-linux-user >> -/*-bsd-user >> -/ivshmem-client >> -/ivshmem-server >> -/libdis* >> -/libuser >> -/linux-headers/asm >> -/qga/qapi-generated >> -/qapi-gen-timestamp >> -/qapi/qapi-builtin-types.[ch] >> -/qapi/qapi-builtin-visit.[ch] >> -/qapi/qapi-commands-*.[ch] >> -**/qapi/qapi-commands.[ch] >> -**/qapi/qapi-emit-events.[ch] >> -/qapi/qapi-events-*.[ch] >> -**/qapi/qapi-events.[ch] >> -**/qapi/qapi-init-commands.[ch] >> -**/qapi/qapi-introspect.[ch] >> -/qapi/qapi-types-*.[ch] >> -**/qapi/qapi-types.[ch] >> -/qapi/qapi-visit-*.[ch] >> -!/qapi/qapi-visit-core.c >> -**/qapi/qapi-visit.[ch] >> -**/qapi/qapi-doc.texi >> -/qemu-edid >> -/qemu-img >> -/qemu-nbd >> -/qemu-options.def >> -/qemu-options.texi >> -/qemu-img-cmds.texi >> -/qemu-img-cmds.h >> -/qemu-io >> -/qemu-ga >> -/qemu-bridge-helper >> -/qemu-keymap >> -/qemu-monitor.texi >> -/qemu-monitor-info.texi >> -/qemu-storage-daemon >> -/qemu-version.h >> -/qemu-version.h.tmp >> -/module_block.h >> -/scsi/qemu-pr-helper >> -/vhost-user-scsi >> -/vhost-user-blk >> -/vhost-user-gpu >> -/vhost-user-input >> -/fsdev/virtfs-proxy-helper >> -*.tmp > > I'm not sure about this one.
According to git blame, it has been added in commit b8bd2f598b7 for tests/vm images. >> -*.[1-9] >> -*.a >> -*.aux >> -*.cp >> -*.exe >> -*.msi >> -*.dll >> -*.so >> -*.fn >> -*.ky > > Need to double-check where this come from. Looking at 0cb3fb1e30a7789881e18 and some other commits, it seems like these were once used for the qemu-doc PDF file generation. Should be ok to remove them now since we moved to Sphinx. >> -*.log >> -*.pdf >> -*.pod >> -*.cps >> -*.fns >> -*.kys >> -*.pg >> -*.pyc >> -*.toc >> -*.tp >> -*.vr >> -*.d >> -!/.gitlab-ci.d >> -!/scripts/qemu-guest-agent/fsfreeze-hook.d >> -*.o >> -.sdk > > Need to double-check where this come from. It's used for the Win32 SDK symlink in "configure": symlink "$vss_win32_sdk/inc" "$source_path/.sdk/vss/inc" Indeed, it might be necessary to keep this entry. >> -*.gcda >> -*.gcno >> -*.gcov >> -/pc-bios/bios-pq/status >> -/pc-bios/edk2-*.fd > > I'm not sure about this one but will check. Are the pc-bios/edk* file still built in-tree? >> -/pc-bios/vgabios-pq/status >> -/pc-bios/optionrom/linuxboot.asm >> -/pc-bios/optionrom/linuxboot.bin >> -/pc-bios/optionrom/linuxboot.raw >> -/pc-bios/optionrom/linuxboot.img >> -/pc-bios/optionrom/linuxboot_dma.asm >> -/pc-bios/optionrom/linuxboot_dma.bin >> -/pc-bios/optionrom/linuxboot_dma.raw >> -/pc-bios/optionrom/linuxboot_dma.img >> -/pc-bios/optionrom/pvh.asm >> -/pc-bios/optionrom/pvh.bin >> -/pc-bios/optionrom/pvh.raw >> -/pc-bios/optionrom/pvh.img >> -/pc-bios/optionrom/multiboot.asm >> -/pc-bios/optionrom/multiboot.bin >> -/pc-bios/optionrom/multiboot.raw >> -/pc-bios/optionrom/multiboot.img >> -/pc-bios/optionrom/kvmvapic.asm >> -/pc-bios/optionrom/kvmvapic.bin >> -/pc-bios/optionrom/kvmvapic.raw >> -/pc-bios/optionrom/kvmvapic.img >> -/pc-bios/s390-ccw/s390-ccw.elf >> -/pc-bios/s390-ccw/s390-ccw.img >> -/docs/built >> -/docs/interop/qemu-ga-qapi.texi >> -/docs/interop/qemu-ga-ref.html >> -/docs/interop/qemu-ga-ref.info* >> -/docs/interop/qemu-ga-ref.txt >> -/docs/interop/qemu-qmp-qapi.texi >> -/docs/interop/qemu-qmp-ref.html >> -/docs/interop/qemu-qmp-ref.info* >> -/docs/interop/qemu-qmp-ref.txt >> -/docs/version.texi >> -/contrib/vhost-user-gpu/50-qemu-gpu.json >> -*.tps > > Need to double-check where this come from. According to 56e8bdd46a8 this is related to the QAPI doc generation ... I hope this is done properly out-of-tree now, so it should be ok to remove it. >> -.stgit-* > > Need to double-check where this come from. I assume that this is related to temporary files from "Stacked Git" ... if anybody is still using that tool, I think that line should rather go into their global ~/.gitignore file instead. >> .git-submodule-status >> cscope.* >> tags >> TAGS >> -docker-src.* >> *~ >> -*.ast_raw >> -*.depend_raw > > Coccinelle generated in source tree, please keep. Ok. Thomas
