Re: [PATCH 09/12] Revert "src/xenconfig: Xen-xl parser"

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

 



On 01/12/2015 07:51 AM, John Ferlan wrote:
> 
> 
> On 01/10/2015 12:03 AM, Jim Fehlig wrote:
>> This reverts commit 2c78051a14acfb7aba078d569b1632dfe0ca0853.
>>
>> Conflicts:
>> 	src/Makefile.am
>>
>> Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxx>
>> ---
>>  .gitignore                    |   1 -
>>  cfg.mk                        |   3 +-
>>  configure.ac                  |   1 -
>>  po/POTFILES.in                |   1 -
>>  src/Makefile.am               |  25 +--
>>  src/libvirt_xenconfig.syms    |   4 -
>>  src/xenconfig/xen_common.c    |   3 +-
>>  src/xenconfig/xen_xl.c        | 499 ------------------------------------------
>>  src/xenconfig/xen_xl.h        |  33 ---
>>  src/xenconfig/xen_xl_disk.l   | 256 ----------------------
>>  src/xenconfig/xen_xl_disk_i.h |  39 ----
>>  11 files changed, 4 insertions(+), 861 deletions(-)
>>
> 
> OK - so reverting is fine; however, xen_xl_disk.{c,h} still exist...
> Simple enough solution, but they will show up in someone's git status
> output since they are also removed from .gitignore.

It's a transient issue - someone that only checked out git at v1.2.11
won't see the generated files; the few of us that do incremental work
can modify .git/info/exclude manually to ignore leftover generated
files.  But if you also want to explicitly ignore the generated files in
.gitignore, go for it.

ACK 1-9, and I'm liking the initial work of 10-12 other than the
Coverity issues that it introduces.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]