search

QuadFlask / react-native-naver-map

🗺️naver map for react-native
MIT License
160 stars 146 forks source link

Remove import of deleted header files in NMapsMap. #168

Open simp7 opened 1 year ago

simp7 commented 1 year ago

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

mintwoo commented 1 year ago

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 commented 1 year ago

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

simp7 commented 1 year ago

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

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

mintwoo commented 1 year ago

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

simp7 commented 1 year ago

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

제가 관리자나 메인테이너가 아닌지라 잘 모르겠네요... pull request 자체는 3주 전에 했습니다.