On Tue 15 Apr 2014 14:34:42 Mark Wielaard wrote: > On Mon, 2014-04-14 at 11:12 +0200, Florian Weimer wrote: > > .gitignore | 99 > > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file > > changed, 99 insertions(+) > > Hmmm. This is all for building with srcdir == builddir isn't it? > I really don't like that. IMHO we should advise people to always build > with srcdir != buildir. But apparently I am in the minority. Mike > Frysinger (added to CC) also keeps trying to push something similar > every couple of months. Last here: > https://lists.fedorahosted.org/pipermail/elfutils-devel/2014-January/thread. > html#3682 > > Are you really sure this is a good idea?
i have no problem with encouraging people to build out of tree (i love that functionality). sometimes though, it's easier to just build in tree to do quick tests. i also don't think it's a big deal to support, or to just take patches tweaking the gitignore file even if people themselves don't want to maintain it. > If so could you double check that your lists are equal? if you want to merge Florian's version, i'd be happy to rebase > > +/src/addr2line > > +/src/ar > > +/src/elfcmp > > +/src/elflint > > +/src/findtextrel > > +/src/ld > > +/src/ldlex.c > > +/src/ldscript.c > > +/src/ldscript.h > > +/src/make-debug-archive > > +/src/nm > > +/src/objdump > > +/src/ranlib > > +/src/readelf > > +/src/size > > +/src/stack > > +/src/strings > > +/src/strip > > +/src/unstrip > > +/tests/*.log > > +/tests/*.trs > > +/tests/addrcfi > > +/tests/addrscopes > > +/tests/alldts > > +/tests/allfcts > > +/tests/allregs > > +/tests/arextract > > +/tests/arls > > +/tests/arsymtest > > +/tests/asm-tst1 > > +/tests/asm-tst2 > > +/tests/asm-tst3 > > +/tests/asm-tst4 > > +/tests/asm-tst5 > > +/tests/asm-tst6 > > +/tests/asm-tst7 > > +/tests/asm-tst8 > > +/tests/asm-tst9 > > +/tests/backtrace > > +/tests/backtrace-child > > +/tests/backtrace-data > > +/tests/backtrace-dwarf > > +/tests/dwarf-getmacros > > +/tests/dwarf-getstring > > +/tests/dwfl-addr-sect > > +/tests/dwfl-bug-addr-overflow > > +/tests/dwfl-bug-fd-leak > > +/tests/dwfl-bug-getmodules > > +/tests/dwfl-bug-report > > +/tests/dwfl-report-elf-align > > +/tests/dwfllines > > +/tests/dwflmodtest > > +/tests/dwflsyms > > +/tests/early-offscn > > +/tests/ecp > > +/tests/find-prologues > > +/tests/funcretval > > +/tests/funcscopes > > +/tests/get-aranges > > +/tests/get-files > > +/tests/get-lines > > +/tests/get-pubnames > > +/tests/hash > > +/tests/line2addr > > +/tests/low_high_pc > > +/tests/md5-sha1-test > > +/tests/msg_tst > > +/tests/newfile > > +/tests/newscn > > +/tests/rdwrmmap > > +/tests/rerequest_tag > > +/tests/saridx > > +/tests/scnnames > > +/tests/sectiondump > > +/tests/show-abbrev > > +/tests/show-die-info > > +/tests/showptable > > +/tests/test-elf_cntl_gelf_getshdr > > +/tests/test-flag-nobits > > +/tests/test-nlist > > +/tests/typeiter > > +/tests/typeiter2 > > +/tests/update1 > > +/tests/update2 > > +/tests/update3 > > +/tests/update4 > > +/tests/varlocs might be easier to keep these lists in src/.gitignore or tests/.gitignore. i'm fine with either. -mike
signature.asc
Description: This is a digitally signed message part.
