cashew icon indicating copy to clipboard operation
cashew copied to clipboard

Modernising

Open tonyarnold opened this issue 7 years ago • 5 comments

This PR is a work in progress - I'm looking at bringing everything up to the following levels:

  • macOS 10.14 minimum target, building against the 10.14 SDK
  • Xcode 10.2 w/Swift 5.0

It's going to be big, it's going to be messy and it's probably going to have to be broken down into smaller PRs.

tonyarnold avatar Feb 06 '19 00:02 tonyarnold

One problem I found with this (not made by you, but I'm testing against your code)... I use a case sensitive file system (legacy unix development reasons) and HotKey is trying to link against "Appkit", not "AppKit". The framework needs to be fixed AND the xcconfig file under Pods needs to be fixed (I thought I could find that via xcode build settings, but I couldn't).

zenspider avatar Feb 07 '19 21:02 zenspider

@zenspider I've created a PR on the upstream dependency (soffes/HotKey#10).

tonyarnold avatar Feb 07 '19 21:02 tonyarnold

@tonyarnold Sorry for missing this! I wasn't getting any notifications from this repo and hadn't been working on Cashew so didn't see this PR. I haven't had a look at it yet—are you planning on adding more to it still?

bellebethcooper avatar Mar 08 '19 04:03 bellebethcooper

Not a problem at all @bellebethcooper!

I was planning on breaking it down quite a bit - I don't think this all needs to land in one go, I just wanted the work to be public. If you want to take it as-is or put it in a WIP branch then please, be my guest! 🌻

tonyarnold avatar Mar 08 '19 04:03 tonyarnold

@tonyarnold Breaking it down sounds good to me. Hit me up if/when you need anything from me :)

bellebethcooper avatar Mar 08 '19 04:03 bellebethcooper