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

[Bazel] Add HeaderMaps to Build with Bazel #4

Open
wants to merge 1 commit into
base: remove_private_api_usage
Choose a base branch
from

Conversation

naik-sanju
Copy link
Collaborator

@naik-sanju naik-sanju commented Sep 6, 2023

Created a new branch for the same changes - #3

Description

  • Headermaps are required to build with Bazel, Incase of XBS Xcode creates this internally for us.
  • Headermap is a map data structure which contains mapping of header files to its path .

Ex: {'MyHeader.h' : '//path-to-MyHeader.h'}

… Bazel

This commit adds headermaps so Bazel can find where headers are located, Incase of XBS Xcode does this for us but for Bazel we need to provide headermap. We have setup example project to generate headermaps
@naik-sanju naik-sanju force-pushed the remove-private-api-usage-and-add-header-maps branch from ed58719 to a864075 Compare September 6, 2023 06:28
@naik-sanju
Copy link
Collaborator Author

Squashed commits and now will point to this branch in Gojek repo's Podfile , run pod install and merge Gojek MR

@asifmohd
Copy link

asifmohd commented Sep 6, 2023

Thanks mas, changes look good. Let's close #3

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