-
Notifications
You must be signed in to change notification settings - Fork 500
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
doc/ko: Translate README to Korean #986
Conversation
Please squash all the commits into a single commit first. |
squashed into one commit! |
What do you think about changing your name to a real one instead of 'kangtegong'? git commit --amend --author="Kang Minchul <[email protected]>" |
Amended the author name! :) |
@kangtegong I don't see it's changed properly. |
Forced Pushed the amended commit |
@henrychoi7 Could you please review together? |
피드백 반영하여 다시 commit 했습니다~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전체에 대한 리뷰는 아니지만 현재 요청드린 부분만 수정해서 rebase 후 업데이트 부탁 드립니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아직 다 보지 못 했지만, 몇 가지 표현 수정이 필요한 것 같습니다.
전체적으로 보진 못했지만 대부분 @henrychoi7 님의 의견에 동의합니다. 요청해주신대로 수정해주세요. @henrychoi7 님 감사합니다! |
변경 요청사항 반영해서 변경하신 부분이 있으면 각 요청마다 댓글로 수정했다고 댓글 달고, "Resolve" 버튼을 눌러주시면 리뷰에 많은 시간을 아낄 수 있습니다. 감사합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 아직 진행 중입니다. 내용 확인 부탁드립니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰를 마쳤습니다. 고생 많으셨습니다! 😄
모든 수정사항 반영하여 commit amend 했습니다! |
doc/ko/README.md
Outdated
프로그램 실행 중 각 함수들과 그 실행 소요시간을 추적한다. | ||
외부 라이브러리의 호출 또한 지원하지만, 보통 진입과 종료만 지원한다. | ||
다른 (중첩된) 외부 라이브러리의 호출 그리고 (혹은) 라이브러리 호출에서 | ||
내부 함수의 호출 추적이 선택적으로 가능하다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'선택적으로' 라는 표현이 약간 어색합니다. 차라리 '내부 함수의 호출을 추적하는 것도 가능하다' 정도로 표현하는 것이 어떨까요?
doc/ko/README.md
Outdated
For recording, the executable needs to be compiled with the `-pg` | ||
(or `-finstrument-functions`) option which generates profiling code | ||
(calling mcount or __cyg_profile_func_enter/exit) for each function. | ||
레코딩을 하려면, 각 함수에 프로파일링 코드 (calling mcount 혹은 __cyg_profile_func_enter/exit) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(mcount 혹은 __cyg_profile_func_enter/exit 함수를 호출)
doc/ko/README.md
Outdated
실험적으로 지원되고 있다. 또한 최근 컴파일러들 중 | ||
(여전히 사용자 프로그램을 재컴파일해야 하긴 하지만) 비슷한 방식으로 | ||
uftrace의 추적 과정에서 생기는 오버헤드를 줄이기 위한 옵션들을 제공하고 있다. | ||
[doc/uftrace-record.md](uftrace-record.md)파일을 참조하라. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
링크가 잘 동작하나요? 제 생각에는 (../uftrace-record.md)가 되어야 할 것 같네요.
doc/ko/README.md
Outdated
@@ -108,30 +105,30 @@ some options to help uftrace to reduce tracing overhead with similar way | |||
4.962 us [ 1892] | } /* a */ | |||
5.769 us [ 1892] | } /* main */ | |||
|
|||
For more analysis, you'd be better recording it first so that it can run | |||
analysis commands like replay, report, graph, dump and/or info multiple times. | |||
더 상세한 분석을 하려면, record를 통해 우선 레코딩을 하고 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
데이터를 기록하고
doc/ko/README.md
Outdated
`-R` options respectively. The following example records first argument and | ||
return value of 'fib' (fibonacci number) function. | ||
또한 함수의 인자와 반환 값을 각각 `-A`와 `-R`옵션으로 레코딩을 하고 보여줄 수 있다. | ||
이하 예제에서는 'fib'(피보나치 숫자) 함수의 첫 번째 인자와 리턴값을 레코드한다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기록한다.
doc/ko/README.md
Outdated
@@ -188,8 +184,8 @@ including its children (total time). | |||
2.194 us 2.194 us 1 __cxa_atexit | |||
|
|||
|
|||
The `graph` command shows function call graph of given function. In the above | |||
example, function graph of function 'main' looks like below: | |||
`graph` 명령어는 주어진 함수의 함수 호출 그래프를 보여준다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
주어진 함수의 호출 그래프를 보여준다.
doc/ko/README.md
Outdated
The `graph` command shows function call graph of given function. In the above | ||
example, function graph of function 'main' looks like below: | ||
`graph` 명령어는 주어진 함수의 함수 호출 그래프를 보여준다. | ||
위의 예제에서, main 함수의 함수 그래프는 아래와 같다: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
호출 그래프
doc/ko/README.md
Outdated
Below is a trace of clang (LLVM) compiling a small C++ template metaprogram. | ||
`dump` 명령어는 각 추적 레코드의 원시 출력 값을 보여준다. | ||
해당 데이터가 `uftrace dump --chrome`으로 실행되었다면, 크롬 브라우저에서 결과를 확인할 수 있다. | ||
이하는 작은 C++ template metaprogram을 컴파일하는 clang (LLVM)의 추적을 보여준다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dump 명령은 기록된 데이터를 그대로(raw) 출력하여 보여준다.
uftrace dump --chrome 명령을 사용하면 크롬 브라우저에서 결과를 확인할 수 있다.
이하는 작은 C++ template metaprogram을 컴파일하는 clang (LLVM)의 실행 과정을 보여준다.
doc/ko/README.md
Outdated
|
||
|
||
License | ||
저작권 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
License는 저작권이 아니고 '사용 허가'에 가까운 개념입니다. 더 좋은 번역이 있을지 모르겠네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@namhyung 외래어 표기법에 따라 라이선스
로 하는 건 어떨까요? 사용권, 허가 등이 있지만 특정 문장 내에서 쓰이는 경우가 아니면 보통 라이선스
로 표기하는 걸 봤습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 좋습니다
모든 수정사항을 반영하여 다시 Commit amend 했습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그동안 리뷰를 잘 하지 못하고 있었는데 몇가지만 더 수정이 되면 좋을것 같습니다.
doc/ko/README.md
Outdated
of the Linux kernel (especially function graph tracer) and supports | ||
userspace programs. It supports various kind of commands and filters | ||
to help analysis of the program execution and performance. | ||
uftrace는 C/C++ 로 작성된 프로그램을 추적(trace)하며 분석하는 도구이다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 아래와 같이 변경하면 어떨까요?
uftrace는 C/C++ 로 작성된 프로그램을 추적(trace)하며 분석하는 도구이다. | |
uftrace 는 C/C++ 로 작성된 프로그램의 실행 흐름을 추적(trace)하며 기록하고 분석하는 도구이다. |
doc/ko/README.md
Outdated
* Mailing list: [[email protected]](https://groups.google.com/forum/#!forum/uftrace) | ||
* 홈페이지: https://github.com/namhyung/uftrace | ||
* 튜토리얼: https://github.com/namhyung/uftrace/wiki/Tutorial | ||
* 채팅방: https://gitter.im/uftrace/uftrace |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
채팅방 링크는 아래와 같이 한글 채팅방으로 변경하면 어떨까요?
* 채팅방: https://gitter.im/uftrace/uftrace | |
* 채팅방: https://gitter.im/uftrace/ko |
doc/ko/README.md
Outdated
* 홈페이지: https://github.com/namhyung/uftrace | ||
* 튜토리얼: https://github.com/namhyung/uftrace/wiki/Tutorial | ||
* 채팅방: https://gitter.im/uftrace/uftrace | ||
* 메일 리스트: [[email protected]](https://groups.google.com/forum/#!forum/uftrace) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"메일 리스트" 는 "메일링 리스트"가 되어야 할 것 같습니다.
doc/ko/README.md
Outdated
프로그램 실행 중 각 함수들과 그 실행 소요시간을 추적한다. | ||
외부 라이브러리의 호출 또한 지원하지만, 보통 진입과 종료만 지원한다. | ||
다른 (중첩된) 외부 라이브러리의 호출 그리고 (혹은) 라이브러리 호출에서 | ||
내부 함수의 호출을 추적하는 것도 가능하다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"가능하다" 뒤에 마침표(.)를 찍어주세요.
doc/ko/README.md
Outdated
|
||
$ ./configure | ||
$ make | ||
$ sudo make install | ||
|
||
For more advanced setup, please refer [INSTALL.md](../../INSTALL.md) file. | ||
더 자세한 설치방법은, [INSTALL.md](../../INSTALL.md) 파일을 참조하라. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"파일을 참조하라." 보다는 "파일을 참조할 수 있다." 는 어떨까요?
doc/ko/README.md
Outdated
Also it can record and show function arguments and return value with `-A` and | ||
`-R` options respectively. The following example records first argument and | ||
return value of 'fib' (fibonacci number) function. | ||
또한 함수의 인자와 반환 값을 각각 `-A`와 `-R`옵션으로 레코딩을 하고 보여줄 수 있다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"레코딩" 을 전부 "기록" 으로 변경하면 좋을것 같습니다.
doc/ko/README.md
Outdated
@@ -250,22 +245,21 @@ The `info` command shows system and program information when recorded. | |||
# page fault : 0 / 172 (major / minor) | |||
# disk iops : 0 / 24 (read / write) | |||
|
|||
The `script` command allows user to run a custom script on a data recorded. | |||
Currently python (version 2.7) is supported only. | |||
`script` 명령어는 레코딩이 된 데이터에 사용자 정의 스크립트를 실행할 수 있게 한다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"레코딩" -> "기록"
doc/ko/README.md
Outdated
The `tui` command is for interactive text-based user interface using ncurses. | ||
It provides basic functionality of `graph`, `report` and `info` commands as of | ||
now. | ||
`tui` 명령어는 ncurses를 이용한 텍스트 기반 대화형 유저 인터페이스를 위한 명령어이다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"ncurses를" -> "ncurses 를"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리고 "유저" 보다는 "사용자" 로 변경하는게 좋겠습니다.
doc/ko/README.md
Outdated
It'll create uftrace.data directory that contains trace data files. | ||
Other analysis commands expect the directory exists in the current directory, | ||
but one can use another using `-d` option. | ||
record 명령어는 추적 데이터 파일을 포함하는 uftrace.data 디렉토리를 만든다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"디렉토리" 를 "디렉터리" 로 변경해 주세요. 다른 문서에서도 이렇게 통일하기로 했습니다.
doc/ko/README.md
Outdated
but one can use another using `-d` option. | ||
record 명령어는 추적 데이터 파일을 포함하는 uftrace.data 디렉토리를 만든다. | ||
다른 분석 명령어들은 그 디렉터리가 현재 경로에 있을 것으로 예상하지만, | ||
다른 디렉토리를 쓰기 위해서는 `-d` 옵션을 사용하면 된다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"디렉토리" -> "디렉터리"
doc/ko/README.md
Outdated
실험적으로 지원되고 있다. 또한 최근 컴파일러들 중 | ||
(여전히 사용자 프로그램을 재컴파일해야 하긴 하지만) 비슷한 방식으로 | ||
uftrace의 추적 과정에서 생기는 오버헤드를 줄이기 위한 옵션들을 제공하고 있다. | ||
[doc/uftrace-record.md](../uftrace-record.md)파일을 참조하라. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리고 이 부분은 다시 아래와 같이 변경이 필요해 보이네요. 현재 디렉토리의 record 페이지로 원문을 만든 이유는 한글로 번역된 man 페이지를 보여주기 위함입니다.
[doc/uftrace-record.md](../uftrace-record.md)파일을 참조하라. | |
[doc/uftrace-record.md](uftrace-record.md) 파일을 참조하라. |
@honggyukim 님 피드백을 바탕으로 수정한 README 다시 Commit 했습니다! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니다!
doc/ko/README.md
Outdated
On Linux distros, [misc/install-deps.sh](../../misc/install-deps.sh) installs required | ||
software(s) on your system. Those are for optional advanced features but highly | ||
recommend to install them together. | ||
리눅스 배포판에서, [misc/install-deps.sh](../../misc/install-deps.sh)스크립트는 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
공백이 하나 필요해 보이네요.
리눅스 배포판에서, [misc/install-deps.sh](../../misc/install-deps.sh)스크립트는 | |
리눅스 배포판에서, [misc/install-deps.sh](../../misc/install-deps.sh) 스크립트는 |
doc/ko/README.md
Outdated
======= | ||
The uftrace program is released under GPL v2. See [COPYING file](../../COPYING) for details. | ||
uftrace 는 GPL v2. 라이선스 하에 배포되며 자세한 내용은 [COPYING 파일](../../COPYING)에서 확인할 수 있다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사소한 변경이 하나 더 있으면 좋겠습니다.
uftrace 는 GPL v2. 라이선스 하에 배포되며 자세한 내용은 [COPYING 파일](../../COPYING)에서 확인할 수 있다. | |
uftrace 는 GPL v2. 라이선스 하에 배포되며 자세한 내용은 [COPYING](../../COPYING) 파일에서 확인할 수 있다. |
수정사항 반영해서 다시 커밋했습니다~ |
This commit translates README.md in Korean Signed-off-by: Minchul Kang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@namhyung 최종 확인 부탁드립니다. |
@namhyung 일단 merge 하고 더 수정이 필요하면 추가 업로드하도록 하겠습니다. |
dynamic tracing 이 AArch64 (ARM64) 를 지원하다는 내용과 script 명령에서 Lua 5.1 언어가 추가되었다는 내용을 원문과 번역 모두 업데이트하면 좋겠네요. |
Hello, I translated README.md to Korean!
https://github.com/kangtegong/uftrace/blob/README-ko/doc/ko/README.md
If you find any awkward expressions or mistakes,
please let me know! :)
Thank you
Signed-off-by: kangtegong [email protected]