tom-seddon/b2

Sort out keyboard monitoring state for b2 and b2 Debug

Opened this issue · 2 comments

At the moment you can only add one, it seems? Should be able to add both! Maybe their Info.plists aren't unique enough.

(Would be nice to have them both share the same setting, but that's probably impossible, especially if not notarized...)

Note:

tmbp ~/github/b2 % diff -u ./build/Xcode/src/b2/RelWithDebInfo/b2.app/Contents/Info.plist ./build/Xcode/src/b2/Final/b2.app/Contents/Info.plist
tmbp ~/github/b2 %

The release process uses ditto to simultaneously copy b2.app (RelWithDebInfo config) and rename it:

copy_darwin_app("r",mount,"b2 Debug.app")

Should try doing this at the build stage, so that b2.app (RelWithDebInfo config) actually has a fundamentally different name. Will that help?

Hmm, looks like set_target_properties doesn't support generator expressions, so per-config Info.plist contents will be difficult to arrange.

Could work around this by taking advantage of the Ninja generator being single config? Not sure I fancy that, though. It'd be nice to have this work for everything (and there's a multi-config Ninja generator too).

A reasonable option: separate b2 vs b2 Debug from the cmake config mechanism. So no more Final vs RelWithDebInfo, you now just have Debug and RelWIthDebInfo and there's a (b2,b2 Debug) * (Debug,RelWithDebInfo) matrix.

But I think this is actually just telling me to JFDI regarding #125