[0.17] don't do mac travis build, no support

[instagibbs]

No description provided.

[stevenroose]

utACK

[instagibbs]

last build appears to be failing for taking too long. Doubt it's our fault.

[jtimon]

Let's keep this open, but can't we leave it without merging for as long as we can?

[instagibbs]

Why? You'd have to get the OSX image thing that's proprietary.

…

On Fri, Oct 12, 2018, 8:15 PM Jorge Timón ***@***.***> wrote: Let's keep this open, but can't we leave it without merging for as long as we can? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#428 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFgC0wTqn_vARpqF1UMATVjc6bWfQUNyks5ukTC3gaJpZM4XZqcJ> .

[jtimon]

That is a very goo point, but isn't only travis who does got to get it?

[jtimon]

I guess my point and my vote is: let's keep as many tests from bitcoin core living for as long as we can until it is convenient for a given change to just ignore those tests for unsupported platforms. Yes we can run test for things we don't support. Perhaps what we really need is some doc:

WARNING: We don't plan to support this build and don't dream about signing it, but we will keep running tests with this piece of proprietary software just in case it burns differently in an interesting way.

Or something of the sort.

[instagibbs]

My point is currently the test isn't living. If we can make it live later, great. Until then I'm disabling it.

[stevenroose]

I'm not really seeing the problem. All 0.17 PRs seem to pass this build in CI..

[instagibbs]

oh hm it was failing before, and previous branches definitely didn't work.