Remove recursive header search path

corymsmith / react-native-fabric

A React Native library for Fabric, Crashlytics and Answers

MIT License

1.31k stars 236 forks source link

Remove recursive header search path #191

Open bitcrumb opened 6 years ago

bitcrumb commented 6 years ago

It resulted in an "Argument list too long: recursive header expansion failed." error when archiving (via Xcode) an app that has react-native-fabric as a dependency.

isaaclem commented 6 years ago

Hi is this merged? Because I'm still hitting this issues

bitcrumb commented 6 years ago

No hasn't been merged yet. I've fixed this issue in May already and opened a PR then. @corymsmith hasn't responded since. So we're effectively using our own fork at the moment.

I'm seeing some recent activity on this repo by @sibelius, maybe he can have a look at this PR?

castaway2000 commented 6 years ago

Still having this issue on my own build as well please merge this ASAP

Twinbird24 commented 5 years ago

I'm having this exact issue as well, this PR fixes the issue for me. Is there a reason this isn't being merged?