| *** Taehee_Jang has joined #openstack-ko | 05:29 | |
| *** ChJR_ has joined #openstack-ko | 06:32 | |
| ChJR_ | ls | 06:32 |
|---|---|---|
| ChJR_ | ianychoi 님 계십니까? | 06:33 |
| *** ChJR_ has quit IRC | 06:52 | |
| *** Taehee_Jang has quit IRC | 07:46 | |
| *** ChJR has joined #openstack-ko | 13:02 | |
| ChJR | 안녕하세요. | 13:02 |
| ChJR | ianychoi 님 계신가요? | 13:02 |
| ianychoi | ChJR, 아 네 지금 있습니다 ㅎ | 13:21 |
| ChJR | 자세한 내용은 슬랙에 적어두었습니다만, 혹시 대화 가능하신가요? | 13:22 |
| ianychoi | 넵넵 ㅎㅎ | 13:22 |
| ChJR | 그냥 여기서 말씀드리면 되나요? 아니면 따로 말씀드리면 되나요? | 13:23 |
| ianychoi | 네 여기도 괜찮습니다. 8번 이슈 관련해.. 해결된 부분 혹시 git review로 올려주실 수 있으셔요? | 13:24 |
| ChJR | 잠시만요... | 13:25 |
| ChJR | 런치패드 등록 없이도 되나보네요... | 13:25 |
| ChJR | 올리기 전에 tox -epep8 케이스 두 개에 대해서만 돌리고요... | 13:26 |
| ianychoi | 넵넵 런치패드 없이도 가능합니다 :) | 13:26 |
| ChJR | git 시퀀스가 git commit 먼저 하고 git review 후에 git change-id를 고치는 거였나요? | 13:31 |
| ianychoi | dksy | 13:37 |
| ianychoi | 아뇨.. git review -s 해서 sync를 한 다음에요 | 13:37 |
| ianychoi | git commit 하시면.. change-id가 자동으로 생깁니다 | 13:37 |
| ChJR | ianychoi 님 올렸습니다. 커밋 메시지 좀 본다고 오래 걸렸네요. | 13:49 |
| ianychoi | 넵넺 | 13:49 |
| ChJR | 테스트 좀 꼭 부탁드립니다. 제 맥은 이미 GNU 툴킷이 섞여 있어서 확실하게 테스트하기 힘들어서 말입니다... | 13:50 |
| ianychoi | 아하.. 넵 | 13:52 |
| ianychoi | 네 잘 되네요.. 제 맥에서요 ㅎ | 13:58 |
| ianychoi | 몇 가지 형식 관련 부분만 제가 코멘트 할께요.. | 13:58 |
| ChJR | 옙 | 14:00 |
| ianychoi | 네 파일명만 tools/test-po.sh 로 바꿔주심 될 거 같아요 | 14:01 |
| ChJR | 리뷰 올린 건 어떻게 처리하나요? | 14:01 |
| ianychoi | git commit -a --amend 하시면 커밋을 변경할 수 있거든요 | 14:02 |
| ianychoi | 제 생각엔.. git mv test-po.bash tools/test-po.sh | 14:02 |
| ianychoi | 로 하시구요 | 14:02 |
| ChJR | 아하, ID만 같으면 리뷰 커맨드로 수정이 된단 거군요 | 14:02 |
| ianychoi | tox.ini에서 파일 레퍼런스 바꿔주시고.. | 14:02 |
| ianychoi | 넵넵 | 14:02 |
| ianychoi | 그리고 git commit -a --amend 하신담에.. git review 하시면 두 번째 패치로 올라가요 | 14:02 |
| ChJR | bash에서 sh로 확장명 바꾼 건 파일 확장명의 일관성 때문인가요? | 14:03 |
| ianychoi | 넵넵 | 14:03 |
| ChJR | 그러면 쉬뱅도 수정해야 하나요..? | 14:03 |
| ChJR | env로 실제 사용하는 배시 셸 찾아서 실행하도록 되어 있는데... | 14:04 |
| ianychoi | 아아.. | 14:04 |
| ianychoi | 다른 파일들도 /bin/bash 로 되어서 괜찮을 거 같아요 ㅎ | 14:04 |
| ChJR | 그러니까... 제 스크립트는 #!/usr/bin/env bash 로 되어 있는데 이것도 통일을 위해 #!/bin/bash로 바꿔야 하는 건가 해서요 | 14:05 |
| ChJR | 얼라리 이거 언제 바꿔뒀었지 | 14:06 |
| ChJR | 이미 그렇게 해서 올려뒀었네요. 신경 안 쓰셔도 될 것 같습니다. | 14:06 |
| ianychoi | 아아 넵넵 제가 처음 확인했을 땐 /bin/bash 였어요 ㅎ | 14:08 |
| ChJR | find 확인 경로 같은 건 어떻게 지정하는 편이 좋을까요 | 14:09 |
| ChJR | 그냥 상대 경로로 지정해도 문제가 없을까요? | 14:09 |
| ChJR | 아니면 toxinidir 을 받아서 처리할 수 있을까요? | 14:10 |
| ianychoi | 음.. {toxinidir} 를 bash argument로 하면 어떨가요? | 14:11 |
| ChJR | 상대 경로로 하니까 명령어 실행시킨 디렉토리 기준에서의 상대 경로로 시작해버리네요 | 14:11 |
| ianychoi | tox.ini에서 | 14:12 |
| ianychoi | bash -c "{toxinidir}/test-po.bash {toxinidir}" | 14:12 |
| ianychoi | bash -c "{toxinidir}/test-po.sh {toxinidir}" | 14:12 |
| ianychoi | bash -c "{toxinidir}/tools/test-po.sh {toxinidir}" | 14:13 |
| ianychoi | 그리고 bash 파일 안에서.. | 14:13 |
| ianychoi | FIND_DIR=$1 이렇게 하면 argument를 받을테고.. | 14:13 |
| ianychoi | find $FIND_DIR ... 이렇게 하면 되지 않을까 생각해요 | 14:14 |
| ChJR | 말씀하신 방법으로 테스트 중입니다 | 14:17 |
| ChJR | 동작 확인했고요, 해당 스크립트만 실행하면 실행 디렉터리 하위의 po, pot 파일들에 대해서만도 테스트 가능하게 처리해뒀습니다. | 14:21 |
| ChJR | 어이쿠 리뷰 다시 올려야 하네요 | 14:22 |
| ChJR | ianychoi 님 리뷰 다시 올렸습니다. | 14:24 |
| ianychoi | 아 넵 | 14:58 |
| ChJR | 끙 | 15:03 |
| ChJR | 배시가 기본 셸이 아닌 POSIX도 있었구나... | 15:03 |
| ChJR | ianychoi 님, 이제 호출하기도 죄송한 시각입니다만... 번역 통계에 컨트리뷰터 리스트를 Zanata API로 가져오기 가 구체적으로 뭘 의미하는 건지 이제 와서 보니 모르겠습니다... 설명 부탁합니다. | 15:07 |
| ianychoi | 네.. 그 부분은 제가 잘 정리를 나중에 할께요.. ㅜㅜ | 15:14 |
| ianychoi | 일본 분이 자세히 리뷰주셨으니 그 부분 반영해 주심 문제없을 듯 해요 | 15:14 |
| ChJR | 옙. 마침 해당 리뷰 보고 있었습니다. 해당 코드로 수정한 다음에 배시 테스트 및 sh 테스트해보고 다시 리뷰 올릴게요. | 15:16 |
| ianychoi | 네네 Zanata API쪽은 제가 잘 정리해볼께요.. 몇 가지 정리할 것들이 있는데 | 15:18 |
| ianychoi | 제 진도가 느리군요.. 여기부턴 ㅜㅜ | 15:18 |
| ChJR | 시간이 늦었는데 대화 괜찮으신가요? | 15:21 |
| ChJR | 저는 약 2시 정도까진 작업하려고 합니다 | 15:22 |
| ChJR | 완전 POSIX 호환이 목표였는데 그보단 하향 조절해서 macOS에서도 굴러만 가게, bash 의존성이 있는 것으로 마무리해야겠네요. | 16:29 |
| ChJR | IFS를 '\0' 문자로 설정하고 동작되는 걸 보고 싶었는데, 테스트에서 동작이 안 되어서... | 16:29 |
| ChJR | 리뷰 제목도 POSIX에서 macOS로 변경해서 4번째 패치를 업로드. | 16:31 |
| ChJR | 먼저 들어가겠습니다. | 16:57 |
| *** ChJR has quit IRC | 16:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!