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

Remove import of deleted header files in NMapsMap. #168

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

Conversation

simp7
Copy link

@simp7 simp7 commented Jan 6, 2023

iOS의 NMapsMap 라이브러리가 3.16.1로 업데이트 되면서 일부 헤더파일(NMGLatLng.h, LMGLatLngBounds.h)이 NMapsGeometry로 옮겨졌습니다. 이로 인해 옮겨진 헤더파일의 기존 위치를 참조하여 iOS에서 빌드가 안되는 오류를 수정했습니다. #167
일단 수정된 라이브러리로 테스트하면 정상적으로 작동되는데 시간이 되면 리뷰 가능하신가요? 감사합니다.

@mintwoo
Copy link

mintwoo commented Jan 27, 2023

react-native-nmap (0.0.66)
NMapsMap (3.16.1)
../node_modules/react-native-nmap/ios/reactNativeNMap/RNNaverMapViewManager.m:15:9 'NMapsMap/NMGLatLng.h' file not found
오류가 납니다...🥲

@mintwoo
Copy link

mintwoo commented Jan 27, 2023

git-lfs install, pod install --repo-update 했지만 똑같아요..ㅠ

@simp7
Copy link
Author

simp7 commented Jan 27, 2023

git-lfs install, pod install --repo-update 했지만 똑같아요..ㅠ

아직 해당 pull request가 merge되지 않아서 버전에 반영이 안되어있을 겁니다.
당장 적용 가능한 해결책으로는 #167에서 언급한 임시 해결 방법을 쓴 후에 patch-package와 postinstall-postinstall패키지를 이용해 자체적으로 변경한 라이브러리를 프로젝트에 고정시키면 될 것 같습니다.

@mintwoo
Copy link

mintwoo commented Jan 27, 2023

적용되는데 얼마나 걸릴까요?

@simp7
Copy link
Author

simp7 commented Jan 27, 2023

적용되는데 얼마나 걸릴까요?

제가 관리자나 메인테이너가 아닌지라 잘 모르겠네요... pull request 자체는 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