Page MenuHomePhabricator

ignore OSX make deploy generate files
AbandonedPublic

Authored by andychen on Apr 3 2018, 12:20.

Details

Reviewers
jasonbcox
matiu
deadalnix
Group Reviewers
Restricted Project
Restricted Owners Package(Owns No Changed Paths)
Summary

OSX make deploy generate files which should be ignored

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
feature/ignore_osx_make_deploy_files
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2231
Build 2605: Bitcoin ABC Buildbot (legacy)
Build 2604: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Apr 3 2018, 12:20
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptApr 3 2018, 12:20
deadalnix requested changes to this revision.Apr 4 2018, 21:40
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
.gitignore
80

Remove irrelevent files.

95

What happens with core on these ? Do they generate them ? If not why ?

This revision now requires changes to proceed.Apr 4 2018, 21:40
.gitignore
80

which irrelevent files?

95

make deploy on OSX generate these files which I think should be ignored

jasonbcox requested changes to this revision.Sep 20 2018, 21:24

You should be building out of tree rather than in the same repo to avoid us having to maintain a long list of generated files to ignore. Please either fix up this diff or abandon, as I'd like to cleanup our review board.