Skip to content

Conversation

@fanquake
Copy link
Member

No description provided.

@laanwj
Copy link
Member

laanwj commented Jul 11, 2019

Concept ACK, the file is produced in the makefile (though, not by any umbrella rule), so it should be ignored

also code review ACK 290c5c6

@maflcko maflcko changed the title trivial: build: ignore osx_volname & add it to clean-local build: ignore osx_volname & add it to clean-local Jul 11, 2019
@fanquake fanquake force-pushed the ignore_osx_volname branch from 290c5c6 to 57fb62e Compare July 16, 2019 02:22
@fanquake fanquake changed the title build: ignore osx_volname & add it to clean-local build: ignore macOS make deploy artefacts & add them to clean-local Jul 16, 2019
@hebasto
Copy link
Member

hebasto commented Jul 19, 2019

Concept ACK

@practicalswift
Copy link
Contributor

utACK 57fb62e

*.plist

osx_volname
dist/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have any other files than the already ignored Bitcoin-Qt.app in here?

@jonasschnelli
Copy link
Contributor

ACK 57fb62e

@hebasto
Copy link
Member

hebasto commented Jul 22, 2019

@fanquake
On master testing make deploy on High Sierra 10.13.6. Cannot observe mentioned artefacts.
Which macOS version do you use?

@fanquake
Copy link
Member Author

@hebasto The artefacts occur when building for a macOS HOST on a Linux machine.

@hebasto
Copy link
Member

hebasto commented Aug 13, 2019

Tested master branch on Linux Mint 19.2:

cd depends
make HOST=x86_64-apple-darwin14 SDK_PATH=`pwd`/SDKs
cd ..
./configure --prefix=`pwd`/depends/x86_64-apple-darwin14
make deploy

Artefacts:

  • dist/ folder -- observed
  • dpi36.background.tiff -- observed
  • dpi72.background.tiff -- observed
  • osx_volname -- not observed

@fanquake fanquake force-pushed the ignore_osx_volname branch from 57fb62e to c84ff23 Compare August 14, 2019 00:07
@fanquake
Copy link
Member Author

@hebasto Thanks for testing. I've fixed your nit.

osx_volname -- not observed

You should see it if you run make osx_volname

@hebasto
Copy link
Member

hebasto commented Aug 14, 2019

ACK c84ff23, tested on Linux Mint 19.2:

  • make with targets osx_volname, deploydir and deploy creates the mentioned artefacts
  • make clean removes them

@DrahtBot
Copy link
Contributor

Gitian builds for commit a7aa809 (master):

Gitian builds for commit 5b4a866 (master and this pull):

@dongcarl
Copy link
Contributor

ACK c84ff23

fanquake added a commit that referenced this pull request Aug 21, 2019
… clean-local

c84ff23 build: ignore macOS make deploy artefacts & add them to clean-local (fanquake)

Pull request description:

ACKs for top commit:
  hebasto:
    ACK c84ff23, tested on Linux Mint 19.2:
  dongcarl:
    ACK c84ff23

Tree-SHA512: 5a6525a5e433a7279807cc02d3f47bb7f4bd5da8142a945b21d3bfa71938cac823569b34b071e21525a817a187fb02a562bb1c5a9761e2019a2b0900a51a8315
@fanquake fanquake merged commit c84ff23 into bitcoin:master Aug 21, 2019
@fanquake fanquake deleted the ignore_osx_volname branch August 21, 2019 00:03
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Aug 22, 2019
…them to clean-local

c84ff23 build: ignore macOS make deploy artefacts & add them to clean-local (fanquake)

Pull request description:

ACKs for top commit:
  hebasto:
    ACK c84ff23, tested on Linux Mint 19.2:
  dongcarl:
    ACK c84ff23

Tree-SHA512: 5a6525a5e433a7279807cc02d3f47bb7f4bd5da8142a945b21d3bfa71938cac823569b34b071e21525a817a187fb02a562bb1c5a9761e2019a2b0900a51a8315
kwvg added a commit to kwvg/dash that referenced this pull request Nov 3, 2021
kwvg added a commit to kwvg/dash that referenced this pull request Nov 4, 2021
kwvg added a commit to kwvg/dash that referenced this pull request Nov 11, 2021
pravblockc pushed a commit to pravblockc/dash that referenced this pull request Nov 18, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants