Skip to content

Commit

Permalink
Add os version
Browse files Browse the repository at this point in the history
  • Loading branch information
noahsmartin committed Nov 30, 2023
1 parent f5aaea9 commit 97f73a3
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 6 deletions.
1 change: 1 addition & 0 deletions ETTrace/ETTraceRunner/ResponseModels/ResponseModel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import Symbolicator

struct ResponseModel: Decodable {
let osBuild: String
let osVersion: String?
let isSimulator: Bool
let libraryInfo: LibraryInfo
let cpuType: String
Expand Down
6 changes: 3 additions & 3 deletions ETTrace/ETTraceRunner/RunnerHelper.swift
Original file line number Diff line number Diff line change
Expand Up @@ -84,12 +84,12 @@ class RunnerHelper {
} else {
arch = ""
}
var osVersion = responseData.osBuild
osVersion.removeAll(where: { !$0.isLetter && !$0.isNumber })
var osBuild = responseData.osBuild
osBuild.removeAll(where: { !$0.isLetter && !$0.isNumber })

let threadIds = responseData.threads.keys
let threads = threadIds.map { responseData.threads[$0]!.stacks }
let symbolicator = StackSymbolicator(isSimulator: isSimulator, dSymsDir: dsyms, osVersion: osVersion, arch: arch, verbose: verbose)
let symbolicator = StackSymbolicator(isSimulator: isSimulator, dSymsDir: dsyms, osBuild: osBuild, osVersion: responseData.osVersion, arch: arch, verbose: verbose)
let flamegraphs = FlamegraphGenerator.generate(
events: responseData.events,
threads: threads,
Expand Down
8 changes: 5 additions & 3 deletions ETTrace/Symbolicator/Symbolicator.swift
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,15 @@ public class StackSymbolicator {

let isSimulator: Bool
let dSymsDir: String?
let osVersion: String
let osBuild: String
let osVersion: String?
let arch: String
let verbose: Bool

public init(isSimulator: Bool, dSymsDir: String?, osVersion: String, arch: String, verbose: Bool) {
public init(isSimulator: Bool, dSymsDir: String?, osBuild: String, osVersion: String?, arch: String, verbose: Bool) {
self.isSimulator = isSimulator
self.dSymsDir = dSymsDir
self.osBuild = osBuild
self.osVersion = osVersion
self.arch = arch
self.verbose = verbose
Expand Down Expand Up @@ -225,7 +227,7 @@ public class StackSymbolicator {
let searchFolder = "\(FileManager.default.homeDirectoryForCurrentUser.path)/Library/Developer/Xcode/iOS DeviceSupport"
let directories = (try? FileManager.default.contentsOfDirectory(atPath: searchFolder)) ?? []

for folder in directories where folder.contains(osVersion) && folder.hasSuffix(arch){
for folder in directories where folder.contains(osBuild) && folder.hasSuffix(arch){
return "\(searchFolder)/\(folder)/Symbols\(libPath)"
}
return nil
Expand Down
2 changes: 2 additions & 0 deletions ETTrace/Tracer/EMGTracer.mm
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,12 @@ + (NSDictionary *)getResults {

const NXArchInfo *archInfo = NXGetLocalArchInfo();
NSString *cpuType = [NSString stringWithUTF8String:archInfo->description];
NSOperatingSystemVersion version = [NSProcessInfo processInfo].operatingSystemVersion;
return @{
@"libraryInfo": EMGLibrariesData(),
@"isSimulator": @([self isRunningOnSimulator]),
@"osBuild": [self osBuild],
@"osVersion": [NSString stringWithFormat:@"%ld.%ld.%ld", (long)version.majorVersion, (long)version.minorVersion, (long)version.patchVersion],
@"cpuType": cpuType,
@"device": [self deviceName],
@"threads": threads,
Expand Down

0 comments on commit 97f73a3

Please sign in to comment.