frida / frida-swift

Frida Swift bindings
Other
141 stars 37 forks source link

Reference to `FridaDeviceManager` is ambiguous #5

Closed klmitchell2 closed 4 years ago

klmitchell2 commented 4 years ago

Hey 👋 ,

I am trying to install frida-swift but I receive this error when I attempt to run the tests. Building the framework does not produce errors, but I am getting this compile error: image

This is happening for FridaDevice, FridaSession, and FridaScript.

oleavr commented 4 years ago

Hey!

This means the namespace we chose on the Swift side is unfortunuate, as it results in some of the Objective-C interop header types clashing with the C types. Either we will have to change our Swift namespace so this doesn't happen, or configure a different namespace for the Objective-C side only (if possible).

As a workaround you can probably disable generation of the Objective-C interop header.

klmitchell2 commented 4 years ago

What would be the best way to disable the generation of the Objective-C inter header? I'm unsure if this a flag at make core-macos-thin or when generating the dev kit?

oleavr commented 4 years ago

It would be an Xcode project setting in frida-swift. I haven't investigated this but suspect it's in there somewhere.

klmitchell2 commented 4 years ago

Found the setting in Build Settings:

image

Changing from Yes to No fixed the issue.

oleavr commented 4 years ago

Yay. Could you open a PR to change this? Don't think we're missing out on anything by omitting this.