[trivial] Add .DSYM to .gitignore #8119

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:ignore_debug changing 1 files +2 −0

fanquake commented at 11:29 AM on May 30, 2016: member

Add debug symbols to .gitignore
[trivial] Add .DSYM to .gitignore 9cf7923553
laanwj commented at 11:31 AM on May 30, 2016: member

What OS is this for?
btcdrak commented at 11:31 AM on May 30, 2016: contributor

This is platform specific (OSX) and should be in your local global gitignore.
laanwj commented at 1:03 PM on May 31, 2016: member

@btcdrak I think we have more OS-specific stuff in there, .e.g DS_store and *.exe. I think the criterion should be whether the file is generated while following our supported build process. If it is, it should be in our .gitignore. If not, and it is part of some IDE or other user specific tool, it belongs in that user's gitignore.

Apart from that, at some point in our we should encourage in our build documentation doing out-of-tree builds so that it is no longer an issue.
sipa commented at 4:37 PM on June 1, 2016: member

@laanwj That sounds like a good criterion. @fanquake Can you comment?
theuni commented at 4:43 PM on June 1, 2016: member

ACK @laanwj's comment.
fanquake commented at 1:36 AM on June 2, 2016: member

.DSYM are generated when running valgrind on OS X
laanwj commented at 1:52 PM on June 3, 2016: member

I'd say valgrind fits squarely into the 'other tools' category, it's not something we ever launch from the build system.
fanquake closed this on Jun 3, 2016
fanquake deleted the branch on Jun 3, 2016
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels