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

webrick 의존성 추가 #131

Merged
merged 5 commits into from
Aug 5, 2023
Merged

webrick 의존성 추가 #131

merged 5 commits into from
Aug 5, 2023

Conversation

bnbong
Copy link
Collaborator

@bnbong bnbong commented Jul 27, 2023

Related issue: #130

Ruby 3.0 버전 부터는 webrick 라이브러리가 표준 라이브러리에서 제외되었기 때문에 Ruby 3.0 이상의 버전으로 빌드를 수행하면 다음과 같은 오류가 발생하면서 빌드가 수행되지 않습니다.

스크린샷 2023-07-27 오후 12 54 17

Gemfile, Gemfile.lock 파일에 webrick의존성을 추가함으로써 해당 문제를 해결하였습니다.

@bnbong
Copy link
Collaborator Author

bnbong commented Jul 27, 2023

@microsoft-github-policy-service agree

Gemfile.lock Outdated
minima (2.5.1)
jekyll (>= 3.5, < 5.0)
jekyll-feed (~> 0.9)
jekyll-seo-tag (~> 2.1)
minitest (5.18.0)
multipart-post (2.1.1)
nokogiri (1.13.8-arm64-darwin)
racc (~> 1.4)
nokogiri (1.13.8-x64-mingw32)
Copy link
Collaborator Author

@bnbong bnbong Jul 27, 2023

Choose a reason for hiding this comment

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

�기존에는 OS 별로 nokogiri 라이브러리의 설치 버전이 명시되어 있었으나, 의존성을 업데이트 하는 과정중에 nokogiri 라이브러리가 모두 1.13.8버전으로 통합되었습니다.

각기 다른 OS(Window - mingw, Linux - x86 architecture)에서 빌드를 수행했을 때, 이 부분으로 인한 빌드 충돌이 없는지 확인을 해봐야 할 것 같습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

다른 OS 에서도 정말 문제가 없는지 확인이 필요하겠군요. 일이 약간 커졌네요 😄
가능하시면 직접 관련 OS 환경에서 확인해보면 좋겠지만, 어렵다면 확인이 될때까지 보류해야할 것 같아요

Copy link
Collaborator

Choose a reason for hiding this comment

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

@bnbong: Would you re-consider this, because this comment part was already addressed through #116 ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@bnbong: Would you re-consider this, because this comment part was already addressed through #116 ?

@ianychoi I have checked the contents. Reflecting #116, I have rewritten Gemfile.lock because it seems better to restore the parts related to nokigiri library dependency to the original content.

@bnbong bnbong changed the title webrick 의존성 추가 #130 webrick 의존성 추가 Jul 27, 2023
Gemfile.lock Outdated
minima (2.5.1)
jekyll (>= 3.5, < 5.0)
jekyll-feed (~> 0.9)
jekyll-seo-tag (~> 2.1)
minitest (5.18.0)
multipart-post (2.1.1)
nokogiri (1.13.8-arm64-darwin)
racc (~> 1.4)
nokogiri (1.13.8-x64-mingw32)
Copy link
Contributor

Choose a reason for hiding this comment

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

다른 OS 에서도 정말 문제가 없는지 확인이 필요하겠군요. 일이 약간 커졌네요 😄
가능하시면 직접 관련 OS 환경에서 확인해보면 좋겠지만, 어렵다면 확인이 될때까지 보류해야할 것 같아요

@@ -269,12 +267,13 @@ GEM
unf (0.1.4)
unf_ext
unf_ext (0.0.8.1)
unf_ext (0.0.8.1-x64-mingw32)
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시, 이 부분도 nokogiri 통합되면서 그런걸까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

개인 Window Desktop 및 Virtual Machine을 활용해서 Windows, Linux 머신에서의 환경에서 확인을 해 볼 계획입니다!

unf_ext 라이브러리부분은 bundle add webrick 명령어로 의존성 파일이 재작성되는 과정에서 자동으로 삭제된 것 같습니다. 제 머신이 Mac 환경이라 불필요한 window 의존성을 설치받지 않았기 때문에 없어진 것 처럼 보이는데 mingw라이브러리도 다시 복구해야할 것 같습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

여기 부분은 살짝 다른 내용이네요,
PR 제목이나 설명을 수정하시거나, 별도 PR 으로 생성하여 정리하면 더 좋을 것 같습니다

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

다른 이슈의 컨트리뷰션 내용인데 이번 PR에도 반영이 되어 버린 모양입니다ㅠㅠ 브랜치를 분기하는 방식 등으로 분리해서 기여를 했었어야하는데 실수를 했습니다. 조언 정말 감사드립니다!

@ianychoi
Copy link
Collaborator

Please resolve conflicts (e.g., with #116 ) and only include webrick dependency thanks

@bnbong
Copy link
Collaborator Author

bnbong commented Jul 30, 2023

Please resolve conflicts (e.g., with #116 ) and only include webrick dependency thanks

@ianychoi Gemfile.lock file has been rewritten to reflect this. Thank you for the comment, May I ask you to check it?

@ianychoi
Copy link
Collaborator

It still shows to resolve conflicts - please double check this thanks

image

@bnbong
Copy link
Collaborator Author

bnbong commented Aug 2, 2023

It still shows to resolve conflicts - please double check this thanks

image

Conflicts has resolved! Thanks.

@ianychoi
Copy link
Collaborator

ianychoi commented Aug 5, 2023

Thanks!

@ianychoi ianychoi merged commit d928522 into Azure:main Aug 5, 2023
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.

3 participants