Skip to content
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

System library added to a group has empty path #488

Merged
merged 5 commits into from
Oct 10, 2019

Conversation

damirdavletov
Copy link
Contributor

Short description 📝

Adding system library to Frameworks group using PBXGroup.addFile() method causes BPFFileReference to have empty path

Solution 📦

Edit PBXGroup.addFile method to correctly create a file with path that was passed with sourceTree type sdkRoot and developerDir

pepicrft
pepicrft previously approved these changes Oct 10, 2019
@damirdavletov damirdavletov changed the title File add path fix System library added to a group has empty path Oct 10, 2019
@damirdavletov
Copy link
Contributor Author

Thanks for the quick review @pepibumur ! I've updated the changelog as well.

@pepicrft
Copy link
Contributor

Thank you for fixing it! We can merge it as soon as CI is happy.

@codecov
Copy link

codecov bot commented Oct 10, 2019

Codecov Report

Merging #488 into master will increase coverage by 0.13%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #488      +/-   ##
==========================================
+ Coverage    82.4%   82.53%   +0.13%     
==========================================
  Files         148      148              
  Lines        8052     8073      +21     
==========================================
+ Hits         6635     6663      +28     
+ Misses       1417     1410       -7
Impacted Files Coverage Δ
Sources/XcodeProj/Objects/Files/PBXGroup.swift 79.04% <100%> (+1.37%) ⬆️
...s/XcodeProjTests/Objects/Files/PBXGroupTests.swift 94.08% <100%> (+0.74%) ⬆️
Sources/XcodeProj/Objects/Targets/PBXTarget.swift 83.33% <0%> (+4.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea74aaf...d27b344. Read the comment docs.

@pepicrft pepicrft merged commit f9e804e into tuist:master Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants