setup.nsi doesnt include full sources, only src directory #2819

issue TheBlueMatt opened this issue on July 9, 2013
  1. TheBlueMatt commented at 1:54 PM on July 9, 2013: member

    Probably should include doc/ contrib/ etc like the zip/linux tars do too.

  2. gavinandresen commented at 8:15 PM on July 9, 2013: contributor

    I think we should go the opposite way, and not ship any sources with the Windows setup.exe.

    My rationale: nobody is getting the setup.exe and then building from source

  3. jgarzik commented at 8:42 PM on July 9, 2013: contributor
  4. TheBlueMatt commented at 8:58 PM on July 9, 2013: member

    Fine by me, probably shouldnt include partial sources, though.

  5. sipa commented at 9:08 PM on July 9, 2013: member

    Agree about not including the sources in the Windows zip/install, but we should include the documents, which is what this issue is about.

  6. laanwj commented at 6:17 AM on July 10, 2013: member

    Right it's the wrong way around, it should include the docs but not the source, now it includes the source but not the docs.

  7. Diapolo commented at 11:15 AM on July 12, 2013: none

    What I like about inclusion of the src is I can easily do a diff between my current version and a new version and see changes in source. Perhaps I'm the only Win-user doing this ;).

  8. laanwj referenced this in commit abb6774f12 on Jan 21, 2014
  9. laanwj referenced this in commit 6c980c7d31 on Jan 21, 2014
  10. laanwj closed this on Jan 23, 2014

  11. MathyV referenced this in commit 3141799983 on Aug 5, 2014
  12. Bushstar referenced this in commit 377dd3b822 on Apr 5, 2019
  13. Bushstar referenced this in commit 8c58799d8d on Apr 8, 2020
  14. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 03:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me