실 전 ! Enlightenment 오 픈 소 스 컨 트 리 뷰 션

경 험 에 기 반 한 코 드 리 뷰 와 함 께 떠 나 는

Enlightenment 오 픈 소 스 컨 트 리 뷰 션 이 야 기

삼 성 전 자 삼 성 전 자 SWSW 센 터센 터차 세 대차 세 대 Computing LabComputing Lab

서 주 영 책 임 서 주 영 책 임<juyung.seo@samsung.com><juyung.seo@samsung.com>

< 목 차 >

#1 소 개#2 Enlightenment#3 사 례 분 석#4 끝 으 로

#1 소 개

오 늘 이 야 기 할 내 용

● 내 가 Enlightenment 컨 트 리 뷰 터 다 ! 라 고생 각 하 자 .● Enlightenment 오 픈 소 스 활 동 경 험● 컨 트 리 뷰 션 사 례 분 석● 실 습 병 행● 인 터 랙 티 브 한 시 간

오 늘 이 야 기 할 내 용

● Enlightenment 프 로 젝 트 에 서 는 헌 법● 하 지 만 이 게 모 든 오 픈 소 스 프 로 젝 트 에 대 한 답

은 아 니 다 .● 참 고 만 하 자 .

저 는 요

● 삼 성 전 자 SW 센 터● 차 세 대 Computing Lab● 서 주 영 책 임● <juyung.seo@samsung.com>

● 개 발 자

저 는 요

● 네 트 워 크 보 안 익 명 통 신 전 공● 타 이 젠 UIFW EFL 개 발

● 2010.01 ~● http://tizen.org

● Enlightenment/EFL 오 픈 소 스 커 미 터● 2011.02 ~● http://www.enlightenment.org

저 는 요

● 천 재 태 지 , SeoZ● http://seoz.com

● 한 때 는 바 이 크 라 이 더● 사 랑 하 는 아 내 와 딸

저 는 요

● 오 픈 소 스 중 심 의 생 활 패 턴

저 는 요

● Ohloh● https://www.ohloh.net/accounts/seoz

여 러 분 은 요 ?

#2 Enlightenment

Enlightenment 는

● 1996 년 윈 도 매 니 저 로 시 작● 지 금 은 프 로 젝 트 이 름 으 로 도 사 용● 윈 도 매 니 저 + EFL 라 이 브 러 리 + 유 틸 리 티+ 애 플 리 케 이 션 등 등● http://enlightenment.org

EFL 은

● 툴 킷● 미 리 만 들 어 진 편 의 라 이 브 러 리 모 음● 애 플 리 케 이 션 을 개 발 할 때 사 용● Enlightenment Foundation Libraries

제 품

● 삼 성 NX300 카 메 라

제 품

● 기 어 2, 기 어 2 네 오

제 품

● 냉 장 고 i-Kitchen● http://www.youtube.com/watch?v=5ZpxHgBoi0s

제 품

● 모 바 일 ?

웹 사 이 트

● http://www.enlightenment.org

소 스 코 드 관 리

● SVN → GIT 전 향● 2013.2● svn.enlightenment.org

소 스 코 드 관 리

● Git Web● http://git.enlightenment.org/

#2-1 커 뮤 니 티

분 위 기

● 가 족 같 은 분 위 기● 비 속 어 , 욕 에 연 연 하 지 말 자● Spank! WTF! 은 욕 이 아 님

개 발 자 유 형

● Contributor● Probie● Committer● Founder● Release Manager

개 발 자 유 형

● Contributor● 프 로 젝 트 발 전 에 공 헌 하 는 자● 공 헌 에 는 여 러 가 지 방 법 이 존 재

● Probie● Committer● Founder● Release Manager

개 발 자 유 형

● Contributor● Probie

● 견 습 생 .● 제 한 적 인 커 밋 권 한 을 부 여 받 은 자 .● 직 접 master 브 랜 치 에 push 하 는 권 한 은 없 다 .

● Committer● Founder● Release Manager

개 발 자 유 형

● Contributor● Probie● Committer

● Git 에 바 로 push 가 가 능 한 자 .● 신 뢰 를 기 반 으 로 추 천 에 의 해 권 한 이 부 여 된 다 .● 반 대 가 있 는 경 우 권 한 을 받 지 못 한 다 .

● Founder● Release Manager

개 발 자 유 형

● Contributor● Probie● Committer● Founder

● 창 시 자● Enlightenment/EFL 을 최 초 에 만 든 사 람● Carsten (Raster) Haitzler

● Release Manager

개 발 자 유 형

개 발 자 유 형

개 발 자 유 형

● Contributor● Probie● Committer● Founder● Release Manager

● 릴 리 스 주 기 관 리● 릴 리 스 관 장

개 발 자 유 형

● 거기 에 ...

● b0rker!

개 발 자 분 포

● 지 도● http://www.enlightenment.org/p.php?p=contact/devsmap&l=en

개 발 자 분 포

● Active● 62 명

● Inactive● 85 명

개 발 자 분 포

● 한 국 인● Active: 12 명● Inactive: 3 명

커 밋 권 한 정 리

● [E-devel] the great commit access purge of march 2014... LOOK FOR YOUR NAME!● 2014/03/19● http://sourceforge.net/p/enlightenment/mailman/message/32119732/

한 국 모 임

● http://efl.or.kr

#2-2 도 구

통 합 관 리 도 구

● Trac → Phabricator● 더 많 은 기 능 제 공● 조 금 더 빠 른 반 응 속 도

Phabricator

● 변 태 같 은 툴● 페 이 스 북 에 서 만 들 고 사 용● https://phab.enlightenment.org

Phabricator

위 키

● https://phab.enlightenment.org/w/

메 일 링 리 스 트

● http://www.enlightenment.org/p.php?p=contact&l=en

메 일 링 리 스 트● https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

메 일 링 리 스 트● http://sourceforge.net/p/enlightenment/mailman/enlightenment-devel/

IRC

● http://www.enlightenment.org/p.php?p=contact&l=en● irc.freenode.net● 6667, 8001● #e, #edevelop, #e.kr

IRC

#2-3 컨 트 리 뷰 션

컨 트 리 뷰 션 의 유 형

● 버 그 리 포 트● 버 그 수 정● 기 능 추 가● 번 역● 홍 보● 문 서 작 성● 등 등

컨 트 리 뷰 션 방 법 변 화

● → 메 일 링 리 스 트 이 용 Phabricator● https://phab.enlightenment.org/differential/

Phabricator 이 용

● Arcanist 도 구 사 용● https://phab.enlightenment.org/w/arcanist/

● 직 접 패 치 업 로 드● 메 일 로 전 달

프 로 세 스

● Contributor● 패 치 업 로 드● 리 뷰● 수 락 (accept) 혹 은 거절 (change request)● 반 영 (push)● 끝 (close)

프 로 세 스

● 커 미 터● 작 업 (commit)● 반 영 (push)

프 로 세 스

● BUILD BREAK! 발 생 가 능

코 딩 규 칙

● https://phab.enlightenment.org/w/coding_convention/

● https://phab.enlightenment.org/w/edc_formatting_rule/

Git 사 용 규 칙

● https://phab.enlightenment.org/w/git_practices/

● 기 본 적 으 로 는 rebase

● 신 규 기 능 추 가 등 정 말 필 요 한 경 우 에 만 브 랜 치 허 용

커 밋 전 체 크 포 인 트

● https://phab.enlightenment.org/w/commit_check_point/

#3 사 례 분 석

사 례 #1

● 첫 경 험● 각 자 의 첫 경 험 공 유

사 례 #1

● “elm_theme” doxygen fix● http://sourceforge.net/p/enlightenment/mailman/message/25928625/

사 례 #1

사 례 #1

● 교 훈● 작 은 것 부 터 시 작 하 자 .● 일 단 보 내 라 .

사 례 #2

사 례 #2

● 교 훈● IRC 를 적 극 활 용 할 것 .● 안 드 로 이 드 의 경 우 AndChat, AndroIRC 추 천 .● 대 화 가 활 발 하 게 일 어 나 는 여 러 개 채 널 을 한 달 간 켜

두 었 을 때 3MB 데 이 터 전 송 발 생 .● 단 , 배 터 리 소 모 심 함 .

사 례 #3

● http://git.enlightenment.org/tools/elm-theme-viewer.git/

● https://github.com/seoz/elm-theme-viewer

사 례 #3

● 교 훈● 커 밋 권 한 이 나 브 랜 치 생 성 권 한 이 없 으 면 github

와 같 은 사 이 트 를 활 용 할 것 .

사 례 #4

● Enventor● http://git.enlightenment.org/tools/enventor.git/● https://phab.enlightenment.org/w/projects/enventor/

● EDC 에 디 터● 여 가 시 간 을 활 용 해 제 작● Hermet 님

사 례 #4

사 례 #4

● Elm Theme Viewer● http://git.enlightenment.org/tools/elm-theme-viewer.git/

● Elementary theme previewer● 여 가 시 간 을 활 용 해 제 작● SeoZ 님

사 례 #4

사 례 #4

사 례 #4

● 교 훈● 개 인 시 간 할 애 는 어 느 정 도 필 요

사 례 #5

● https://phab.enlightenment.org/D619#13

사 례 #5

사 례 #5

● 교 훈● Coding convention 은 반 드 시 지 키 자 .

사 례 #6

● https://phab.enlightenment.org/D532

● https://phab.enlightenment.org/D619

● https://phab.enlightenment.org/D356

사 례 #6

사 례 #6

사 례 #6

사 례 #6

● 교 훈● 리 뷰 가 오 래 걸 릴 수 있 다 .● 살 짝 재 촉 하 자 .● 친 분 활 용 .

사 례 #7

● https://phab.enlightenment.org/D356

사 례 #7

사 례 #7

● 교 훈● 패 치 를 작 게 만 들 자 .● 패 치 가 크 면 리 뷰 어 가 부 담 을 느 낀 다 .● 제 대 로 된 리 뷰 를 받 기 어 렵 다 .

사 례 #8

● https://phab.enlightenment.org/D373#7● http://sourceforge.net/p/enlightenment/mailman/message/31577806/

사 례 #8

사 례 #8

사 례 #8

● 교 훈● 한 패 치 는 한 기 능 만 하 는 걸 로 ...● Atomic Patch● http://en.wikipedia.org/wiki/Atomic_commit● 자 연 스 럽 게 커 밋 갯 수 도 올 라 감

사 례 #9

● https://phab.enlightenment.org/audit/view/author/huchi

사 례 #9

사 례 #9

● https://phab.enlightenment.org/maniphest/query/open/

사 례 #9

● 교 훈● 버 그 리 포 팅 시 스 템 을 활 용 하 자 .● 일 거리 가 잘 정 리 되 어 있 다 .

사 례 #10

● http://sourceforge.net/p/enlightenment/mailman/message/32018153/

사 례 #10

사 례 #10

사 례 #10

● 교 훈● 커 밋 메 시 지 도 신 경 써 서 작 성 하 자 .● 규 칙 을 확 인 하 자 .● https://phab.enlightenment.org/w/git_practices/#commit-message

사 례 #11

● 커 밋 권 한 추 천 (Nomination)● http://sourceforge.net/p/enlightenment/mailman/message/32236932/

사 례 #11

사 례 #11

● 교 훈● 때 로 는 특 정 분 야 에 집 중 해 서 컨 트 리 뷰 션 하 자 .● 자 신 만 의 전 문 분 야 를 만 들 자 .

사 례 #12

● 커 밋 권 한 추 천 ... 하 지 만 ...

사 례 #12

사 례 #12

● 교 훈● 평 소 에 자 신 의 존 재 를 알 리 자 .● IRC, 메 일 링 리 스 트 를 기 회 의 장 으 로 .

사 례 #13

● 활 동 을 많 이 하 고 존 재 를 알 렸 더 니 ...

사 례 #13

사 례 #13

사 례 #13

● 교 훈● 꾸 준 히 패 치 를 보 내 고 IRC, 메 일 링 리 스 트 에 노 출

을 했 더 니● 한 국 인 최 초 Enlightenment 프 로 젝 트 커 미 터 가

됨 (SeoZ, Hermet)● 특 정 분 야 의 메 인 테 이 너 로 인 정 받 음

사 례 #14● http://sourceforge.net/p/enlightenment/mailman/message/32043837/

● https://phab.enlightenment.org/w/elementary_focus/

● http://www.mail-archive.com/enlightenment-devel%40lists.sourceforge.net/msg61337.html

사 례 #14

사 례 #14

● 교 훈● 때 로 는 전 략 적 인 움 직 임 이 필 요 .● 메 일 링 리 스 트 , 위 키 페 이 지 등● 역 시 자 기 자 신 을 많 이 노 출 하 라 .

사 례 #15

● http://sourceforge.net/p/enlightenment/mailman/message/32254729/

사 례 #15

사 례 #15

● 교 훈● 참 여 를 하 다 보 면 영 어 도 배 우 게 된 다 .● 다 른 사 람 의 메 일 재 활 용 가 능

사 례 #16

● 리 누 스 의 법 칙 도 뒷 통 수 를 ...● 보 는 눈 만 많 다 면 , 어 떤 버 그 도 쉽 게 잡 을 수있 다 . - 리 누 스 토 발 즈

사 례 #16

사 례 #16

● 교 훈● 공 개 된 코 드 라 고 다 완 벽 한 것 은 아 니 다 .● 여 러 명 이 리 뷰 한 경 우 에 도 실 수 발 생 가 능 .

사 례 #17● http://git.enlightenment.org/core/elementary.git/commit/?id=e195428df0270cbeee43a682fd52dbe7d9babd78

사 례 #17

사 례 #17

● 교 훈● 빌 드 워 닝 수 정 은 다 른 사 람 이 흘 린 콩 고 물 .● 수 시 로 빌 드 해 보 고 워 닝 이 나 에 러 를 찾 아 야 함 .

사 례 #18

● http://git.enlightenment.org/core/elementary.git/commit/?id=99a23ad02b9f665aade5a2555c30cb7192e7e445

사 례 #18

사 례 #18

● https://scan.coverity.com/dashboard

사 례 #18● https://scan6.coverity.com:8443/reports.htm#v30506/p10305/fileInstanceId=12798434&defectInstanceId=4479868&mergedDefectId=652613

사 례 #18

● 교 훈● Coverity 같 은 정 적 분 석 툴 의 테 스 트 결 과 를 주목 하 자 .

사 례 #19● http://sourceforge.net/p/enlightenment/mailman/message/29735142/

사 례 #19

사 례 #19

● 교 훈● 의 사 결 정 의 순 간 에 자 신 의 아 이 디 어 를 제 시 하 라 .

사 례 #20

사 례 #20

사 례 #20

● 교 훈● 신 규 기 능 을 추 가 하 는 경 우 작 업 브 랜 치 를 만 들 어라 .● Merge 커 밋 은 의 미 있 는 메 시 지 를 작 성 한 다 .

#4 끝 으 로

나 만 의 것 을 찾 아 라

● 자 신 이 특 별 히 흥 미 를 갖 는 무 엇 인 가 를 찾 기 위 해 노 력 하 라 는 것 입 니 다 . 또 한 자 신 이 수 정 하 고

유 지 하 는 , 담 당 자 가 될 수 있 는 무 엇 인 가 를 찾 아 보 라 는 것 입 니 다 .

● http://www.youtube.com/watch?v=vuireCI4pvY

감 사 합 니 다