-
-
Notifications
You must be signed in to change notification settings - Fork 517
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing header search paths of SPM dependencies #4513
Comments
Hey @ffittschen thanks for reporting! I guess we can start adding AppCenter to the Side note: I am note sure I got what this PR has to do with the problem 🤔 |
Hmm.. good question, it's probably not related as the problem with the lproj was a runtime issue and not a build error. Sorry for the confusion, I missed that they are not related. I updated the issue and removed the references to #4507 |
@danyf90 I started by adding AppCenter to the fixture on this branch. When setting a breakpoint in the lines mentioned above, the switch statement always falls into the |
Thanks @ffittschen |
Hola 👋, We want to inform you that the issue has been marked as stale. This means that there hasn't been any activity or updates on it for quite some time, and it's possible that it may no longer be relevant or actionable. |
Hola 👋, We want to inform you that we have decided to close this stale issue as there hasn't been any activity or response regarding it after marking it as stale. We understand that circumstances may have changed or priorities may have shifted, and that's completely understandable. If you still believe that this issue needs to be addressed, please feel free to reopen it and provide any necessary updates or additional information. We appreciate your understanding and look forward to your continued contributions to the project. Thank you. |
If anyone experiencing this issue, update Tuist to version
|
Describe the bug and the expected behavior
We are observing missing header search paths with the latest Tuist release (3.5.0) for some of our dependencies, e.g. AppCenter.
After checking out a few commits with related changes, I traced it back to this PR: #4456. Looking at the code changes, I'm assuming that the changes in these lines are causing the issues.
When comparing the generated AppCenter projects with xcdiff from the commit prior to #4456 with the merge-commit of #4456, you can see that in Tuist < 3.5.0, the
HEADER_SEARCH_PATHS
of theAppCenterCrashes
target contained$(SRCROOT)/../PLCrashReporter/include
and$(SRCROOT)/AppCenter/AppCenter/include
, which are missing with Tuist 3.5.0 causing the error shown in the attached screenshot.xcdiff result
Reproduction steps
Steps to reproduce the bug:
tuist fetch && tuist generate
Error log
Error screenshots
Additional context
The text was updated successfully, but these errors were encountered: