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

최신 이더리움 소스를 유지 방안 #8

Open
hackmod opened this issue Feb 14, 2018 · 4 comments
Open

최신 이더리움 소스를 유지 방안 #8

hackmod opened this issue Feb 14, 2018 · 4 comments

Comments

@hackmod
Copy link
Member

hackmod commented Feb 14, 2018

시범삼아, 지금 했던 작업을 기반으로, 커밋을 더 쪼갠 후에 이더리움 마스터 브랜치 위에 패치를 올려보았습니다.

쪼개놓은 패치들은 크게 세개이며 다음과 같습니다.

  • go 소스코드 import 레포지터리 수정.
  • ESC 미니멀 수정
  • VERSIONparams/version.go 갱신 커밋

ESC 미니멀 수정 단독으로만 master 브랜치로 cherry-pick을 하니 conflict가 최소화 되었으며,
go 소스코드 import 레포지터리 수정 부분은 간단히 awk 스크립트를 통해서 변경 (prepare.awk)

conflict난 부분을 수정하고, go소스 import 변환하여 얻은 master브랜치 + ESC 최소 변경을 컴파일하니 정상적으로 컴파일이 수행됨을 확인한 상태입니다.

모 사이트 포크 문제로 제가 이렇게 수정한 사항은 당분간 비공개로 두겠습니다. 혹시 작업하는데 참고하고 싶으시다면 말씀해주시기 바랍니다.

@ghost
Copy link

ghost commented Feb 16, 2018

@hackmod 혹시 go 테스트는 돌려 보신건가요?
설마 컴파일하고 끝은 아니겠죠?

@hackmod
Copy link
Member Author

hackmod commented Feb 16, 2018

@akx20000a // 당연하죠. 지금 몇몇 테스트 오류 수정중입니다. 기존에 있던 오류도 있고,
제 소스는 geth를 gesc로 바꾸지 않고 유지하려고 하고 있어서 다른 오류가 섞여있을 것 같지만요

@ghost
Copy link

ghost commented Feb 18, 2018

@hackmod 거기까지만 해도 되는걸 사기라고 펌훼하시면 저희도 가만 안 있습니다. 하시는거 뭐든지 간에 저희가 당한거랑 똑같이 해드릴게요 😄 해킹당한 거래소가 하는 프로젝트가 뭐가 좋아서 졸졸 따라다니는지 모르겠지만요 자존심도 없으시나 ㅋㅋㅋㅋㅋ

@ghost
Copy link

ghost commented Feb 18, 2018

@hackmod 그렇게 영어 쓰면 뭔가 될 줄 알았죠?
MyEtherWallet/ethereum-lists#451

hackmod pushed a commit to hackmod/go-esn that referenced this issue Jun 6, 2020
* added tests for new abi encoding features (Ethersocial#4)

* added tests from bytes32[][] and string[]

* added offset to other types

* formatting

* Abi/dynamic types (Ethersocial#5)

* Revert "Abi/dynamic types (Ethersocial#5)" (Ethersocial#6)

This reverts commit dabca31d797623d43bd780721fc0ad461d24be53.

* Abi/dynamic types (Ethersocial#7)

* some cleanup

* Apply suggestions from code review

apply suggestions

Co-Authored-By: vedhavyas <vedhavyas.singareddi@gmail.com>

* added better formatting (Ethersocial#8)

* review chnages

* better comments
hackmod added a commit to hackmod/go-esn that referenced this issue Jun 6, 2020
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

No branches or pull requests

1 participant