Conversation
kirb
left a comment
There was a problem hiding this comment.
This is great stuff, thanks! There is a slight problem though, changing the keys to kMGQ rather than kMG is a breaking change, don’t really want to do that. Could we change them back?
for sure! for some reason I totally missed that lol |
dlevi309
left a comment
There was a problem hiding this comment.
switched kMGQ back to kMG
|
@kirb Just to put this out there, I looked into why I originally formatted the keys as when the header was originally written, the case may have been otherwise (either way is fine with me, just wanted to document this in case you thought reformatting was worth the trouble to keep in sync with Apple) |
|
This PR removes the following keys, which is a breaking change: static const CFStringRef kMGAllDeviceCapabilities = CFSTR("AllDeviceCapabilities");
static const CFStringRef kMGAppleInternalInstallCapability = CFSTR("AppleInternalInstallCapability");
static const CFStringRef kMGCarrierBundleInfo = CFSTR("CarrierBundleInfoArray");
static const CFStringRef kMGDeviceSupportsFaceTime = CFSTR("DeviceSupportsFaceTime");
static const CFStringRef kMGDiagnosticsData = CFSTR("DiagData");
static const CFStringRef kMGDieID = CFSTR("DieId");
static const CFStringRef kMGHWModel = CFSTR("HWModelStr");
static const CFStringRef kMGHasAllFeaturesCapability = CFSTR("HasAllFeaturesCapability");
static const CFStringRef kMGRegulatoryIdentifiers = CFSTR("RegulatoryIdentifiers");
static const CFStringRef kMGTotalSystemAvailable = CFSTR("TotalSystemAvailable");I have discussed this with @dlevi309, who responded:
I'm open to not indicating the availability of these values because these values are difficult to determine in the first place and getting them for enough iOS versions to determine availability is more challenging than I think is worth-while for this repo. In either case, dlevi309 and I agree that the current contents of this PR should not be merged and for this reason, I'm moving the status of this PR to draft. |
… the missing keys sooner then later
|
FWIW, the recent commits of MGKeys repo now indicate the introduced (and removed) iOS version of each key. |
What does this implement/fix? Explain your changes.
Complete the list of MG keys (or list about 760 keys)
Checklist
@import MyLibrary;in ObjC, orimport MyLibraryin Swift.Does this close any currently open issues?
…
Any relevant logs, error output, etc?
Any other comments?
…
Where has this been tested?
Operating System: …
Platform: …
Target Platform: …
Toolchain Version: …
SDK Version: …