set TARGET = bitcoin-qt in project file #1623

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:pro-file-set-target changing 1 files +1 −1
  1. Diapolo commented at 6:10 AM on July 23, 2012: none

    From the documentation:

    <pre> TARGET This specifies the name of the target file. For example: TEMPLATE = app TARGET = myapp SOURCES = main.cpp The project file above would produce an executable named myapp on unix and 'myapp.exe' on windows. </pre>

    I checked the makefile, no changes, but TARGET should not be empty IMHO.

  2. set TARGET = bitcoin-qt in project file 6fe375c3bd
  3. sipa commented at 5:24 PM on July 31, 2012: member

    Won't hurt, but not sure what the effect is, if nothing observable happens...

  4. Diapolo commented at 5:25 AM on August 1, 2012: none

    The effect is we don't have an empty field there ;).

  5. BitcoinPullTester commented at 11:14 PM on August 8, 2012: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/6fe375c3bd558cc257e72c04ac5d94dc86be4a19 for binaries and test log.

  6. laanwj commented at 6:21 AM on August 13, 2012: member

    I'm not seeing much of an advantage here either. But I suppose it's good to be explicit (especially as the target name is overridden later for MACOSX).

  7. laanwj referenced this in commit b2ce93fe16 on Aug 13, 2012
  8. laanwj merged this on Aug 13, 2012
  9. laanwj closed this on Aug 13, 2012

  10. DrahtBot 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-21 18:16 UTC

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