Skip to content
This repository has been archived by the owner on May 25, 2021. It is now read-only.

add *.d to .gitignore #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robertkowalski
Copy link
Member

they are generated by the dependencies in priv/

they are generated by the dependencies in priv/
@kxepal
Copy link
Member

kxepal commented Jan 12, 2015

Hm, I don't have such. How does they looks like?

@davisp
Copy link
Member

davisp commented Mar 24, 2015

Which dependencies in priv? *.d seems a bit overly general, perhaps limit to the directory they affect?

@robertkowalski
Copy link
Member Author

file list

    priv/couch_ejson_compare/couch_ejson_compare.d
    priv/couch_js/http.d
    priv/couch_js/main.d
    priv/couch_js/utf8.d
    priv/couch_js/util.d
    priv/icu_driver/couch_icu_driver.d

@robertkowalski
Copy link
Member Author

example file content:

(01:12:21) [robert@tequila-osx] ~/apache/couchdb/src/couch (master) $ cat priv/couch_js/utf8.d
priv/couch_js/utf8.o: priv/couch_js/utf8.c /usr/local/include/js/jsapi.h \
  /usr/local/include/js/js-config.h /usr/local/include/js/jspubtd.h \
  /usr/local/include/js/jstypes.h /usr/local/include/js/jsautocfg.h \
  /usr/local/include/js/jsinttypes.h /usr/local/include/js/jsotypes.h \
  /usr/local/include/js/jscompat.h /usr/local/include/js/jslong.h \
  /usr/local/include/js/jsval.h /usr/local/include/js/jsutil.h \
  /usr/local/include/js/jsproto.tbl /usr/local/include/js/jsversion.h \
  priv/couch_js/config.h

@eiri
Copy link
Member

eiri commented May 18, 2016

@kxepal @davisp : Bump for a great justice.
*.d files in question appears to be "deps files" generated by rebar's port_compiler. rebar 2.6.1 has a habit to leave them behind.

@robertkowalski I'd suggest to make it priv/**/*.d, just to be on a safe side

@davisp
Copy link
Member

davisp commented May 18, 2016

+1 to making it priv/*/.d

@davisp
Copy link
Member

davisp commented May 18, 2016

Erp! Formatting

priv/**/*.d

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants