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