This is the mail archive of the elfutils-devel@sourceware.org mailing list for the elfutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Update .gitignore file


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

Attachment: signature.asc
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]