Skip to content

[#465] GoogleUtilities 중복 링크로 인한 ObjC 클래스 충돌 경고를 제거한다#493

Merged
opficdev merged 1 commit into
developfrom
fix/#465-GoogleUtilites-objc
May 22, 2026
Merged

[#465] GoogleUtilities 중복 링크로 인한 ObjC 클래스 충돌 경고를 제거한다#493
opficdev merged 1 commit into
developfrom
fix/#465-GoogleUtilites-objc

Conversation

@opficdev
Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

모듈러 아키텍처 전환 후 GoogleUtilities Objective-C 클래스가 앱 번들에 중복 포함되며 실기기 콘솔에 경고가 발생하는 문제 해결

📝 작업 내용

📌 요약

  • DevLogApp 타깃의 GoogleSignIn 직접 링크 제거
  • Google Sign-In URL 처리 책임의 DevLogInfra 이관
  • GoogleUtilities 중복 포함 경고 제거

🔍 상세

  • DevLogApp.xcodeproj에서 GoogleSignIn-iOS 패키지 참조 및 GoogleSignIn product dependency 제거
  • AppDelegateGIDSignIn.sharedInstance.handle(url) 직접 호출 제거
  • DevLogInfraGoogleSignInURLHandler 추가 후 URL 처리 위임
  • Google Sign-In SDK 소유 위치의 DevLogInfra 단일화
  • 실기기 Xcode 콘솔에서 GoogleUtilities_GoogleUtilities_Logger_SWIFTPM_MODULE_BUNDLER_FINDER, GULLoggerWrapper 중복 경고 미발생 확인

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this May 22, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이번 PR은 Google Sign-In 의존성을 메인 앱 타겟에서 DevLogInfra 모듈로 이전하는 리팩터링을 수행합니다. 이를 위해 GoogleSignInURLHandler를 추가하여 SDK 호출 로직을 추상화하였으며, AppDelegate에서 해당 핸들러를 사용하도록 수정했습니다. 현재 별도의 리뷰 의견은 없습니다.

@opficdev opficdev merged commit 861cbac into develop May 22, 2026
1 check passed
@opficdev opficdev deleted the fix/#465-GoogleUtilites-objc branch May 22, 2026 16:07
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.

GoogleUtilities 중복 링크로 인한 ObjC 클래스 충돌 경고를 제거한다

1 participant