Giter Site home page Giter Site logo

Comments (13)

fbp93 avatar fbp93 commented on August 25, 2024 1

@cipolleschi can you have a look when you have some time? thanks 🙏

from react-native.

cipolleschi avatar cipolleschi commented on August 25, 2024 1

Shouldn't the header search path link to the headers in the pods?

nope, because with frameworks, the items that are copied to the bundle and used for linking are the content of the .framework folder.

Have you been able to make the audio manager build by that change ?

Unfortunately not, once I saw that the setup was not standard and the error was different, I stopped.
I think we should first understand why we are seeing two different errors.

Unfortunatly I have a lots of them with the current setup, so it's not something I was plan to do immediatly, but I'll definitely consider it

😞 But I understand. Working with legacy projects is tough. I'm going to be on PTO next week, but I plan to resume the work on this when I'm back.

from react-native.

fbp93 avatar fbp93 commented on August 25, 2024 1

thanks! It seems working now (at least this building phase). I'll late you know if I have any other issue while migrating to the latest version.
I'm gonna close the issue

from react-native.

fbp93 avatar fbp93 commented on August 25, 2024

I also checked and it's breaking starting from 0.74.0

from react-native.

max-quirk avatar max-quirk commented on August 25, 2024

Also getting this issue :'(

from react-native.

cipolleschi avatar cipolleschi commented on August 25, 2024

Looking into it.
We probably make a mistake in configuring the RCTDeprecation podspec.

from react-native.

cipolleschi avatar cipolleschi commented on August 25, 2024

@fbp93 I tried the app and the main target builds fine with static libraries and static frameworks.

AudioManager fails, but it fails because React/RCTBridgeModule.h' file not found, not RCTDeprecation not found
Screenshot 2024-06-27 at 17 57 37

I think that the problem is in the header search paths. If you look at the search paths for the main project, they look like this:
Screenshot 2024-06-27 at 18 06 38

The header search paths of the AudioManager looks like this:
Screenshot 2024-06-27 at 18 07 05

The structure of React-Core on disk is like this:
Screenshot 2024-06-27 at 18 07 35

And, as you can see, there is no folder called Public after Headers.

This type of setup, anyway, is not something we recommend: it would be better to create a proper pod for an external library, to benefit from all the automations we are building for autolinking.

That said, @fbp93 and @max-quirk, if you can provide a reproducer where I can see the RCTDeprecation.h file not found happen, I'll be glad to dig into it more closely!

from react-native.

fbp93 avatar fbp93 commented on August 25, 2024

And, as you can see, there is no folder called Public after Headers.

@cipolleschi Shouldn't the header search path link to the headers in the pods? Because there I see the public folder. Have you been able to make the audio manager build by that change ? 🤔
Screenshot 2024-06-28 at 09 35 32

This type of setup, anyway, is not something we recommend: it would be better to create a proper pod for an external library, to benefit from all the automations we are building for autolinking.

Unfortunatly I have a lots of them with the current setup, so it's not something I was plan to do immediatly, but I'll definitely consider it

from react-native.

fbp93 avatar fbp93 commented on August 25, 2024

nope, because with frameworks, the items that are copied to the bundle and used for linking are the content of the .framework folder.

the weird thing is that the same setup was working fine with the previous react and even if I try to remove the public it's not working anyway 🤔

Unfortunately not, once I saw that the setup was not standard and the error was different, I stopped.
I think we should first understand why we are seeing two different errors.

Actually it seems that sometimes I get your same error and sometimes the RCTDeprecation one, so I think it's kind of related. Probably if we are able to make it build we may fix both or understand more 🤔

from react-native.

cipolleschi avatar cipolleschi commented on August 25, 2024

@fbp93 any news on this?

In 0.73, we didn't have the RCTDeprecation module, so that's probably why it was working.

from react-native.

cipolleschi avatar cipolleschi commented on August 25, 2024

@fbp93 I investigated this a little bit more.

First of all, my bad. I was trying to build with use_frameworks, but after reading the issue again, I can see that that was not the case.
Without frameworks, the search path you were using ($(SRCROOT)/../../ios/Pods/Headers/Public/React-Core) is correct.

I added another search path there: $(SRCROOT)/../../ios/Pods/Headers/Public.
Screenshot 2024-07-09 at 15 42 33

and after that, it builds correctly.

The problem is that the Audio Manager was not finding the header in the available search paths.
Starting from Public, the RCTDeprecation.h header is in the path RCTDeprecation/RCTDeprecation.h that is exactly the same import you have in the file.

This is not a standard and officially supported setup, so I don't think we can/should modify core to automate this.
I hope this is enough to understand how these search paths work and to fix them forward for all the modules.

from react-native.

amoiseenko-sc avatar amoiseenko-sc commented on August 25, 2024

@fbp93 How did you resolve the issue?

from react-native.

fbp93 avatar fbp93 commented on August 25, 2024

@fbp93 How did you resolve the issue?

I changed the header path as suggested in the above message

from react-native.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.