search

naver / arcus-zookeeper

ZooKeeper fork for ARCUS
Apache License 2.0
7 stars 12 forks source link

Fix build.xml: support for JDK >= 1.9 #6

Closed jh-jeong closed 7 years ago

jh-jeong commented 7 years ago

이슈: openjdk 최신버전 (>= 1.9) 에서 arcus가 build 되지 않는 현상 빌드 환경: a docker container from ubuntu:lastest image (Docker 17.03.1-ce, build c6d412e)

# uname -a
Linux 227f9156bfa5 4.9.27-moby #1 SMP Thu May 11 04:01:18 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

openjdk-1.9-jdk 가 설치된 환경에서 arcus를 build 할 시:

#./build.sh
ARCUS BUILD PROCESS: START
--------------------------
Working directory is /opt/arcus.
Detailed build log is recorded to scripts/build.log.
--------------------------
[git submodule init] .. SUCCEED
[git submodule update] .. SUCCEED
[server/config/autorun.sh] .. SUCCEED
[clients/c/config/autorun.sh] .. SUCCEED
[zookeeper/ant clean compile_jute bin-package] .. START
Error has occured. /opt/arcus/scripts/etc/autorun.sh has failed.
Error has occured. ./build.sh has failed.
Check /opt/arcus/scripts/build.log

build.log 확인 시:

Working directory is /opt/arcus.
/opt/arcus /opt/arcus/scripts
Submodule 'clients/c' (https://github.com/naver/arcus-c-client.git) registered for path 'clients/c'
Submodule 'clients/java' (https://github.com/naver/arcus-java-client.git) registered for path 'clients/java'
Submodule 'server' (https://github.com/naver/arcus-memcached.git) registered for path 'server'
Submodule 'zookeeper' (https://github.com/naver/arcus-zookeeper.git) registered for path 'zookeeper'

...

jute:
    [javac] Compiling 39 source files to /opt/arcus/zookeeper/build/classes
    [javac] warning: [options] bootstrap class path not set in conjunction with -source 1.5
    [javac] error: Source option 1.5 is no longer supported. Use 1.6 or later.
    [javac] error: Target option 1.5 is no longer supported. Use 1.6 or later.

BUILD FAILED
/opt/arcus/zookeeper/build.xml:240: Compile failed; see the compiler error output for details.

Total time: 1 second

이는 jdk >= 1.9 이후로 1.5 버전으로의 컴파일 옵션이 더 이상 지원되지 않기 때문에 발생한 에러로 보여집니다: 관련 링크.

arcus-zookeeper/build.xml 에서 컴파일 옵션을 수정해주면 문제 없이 빌드되는 것을 확인했습니다.

Compatibility issue를 최소화하기 위해, 1.5 -> 1.6 으로 컴파일 버전을 한 단계만 올렸습니다.

jhpark816 commented 7 years ago

@jh-jeong

먼저, PR 감사합니다.

arcus-zookeeper는

현재 PR은 ZooKeeper 3.4.5 자체에 대한 수정 PR이며, 향후 ZooKeeper 3.5 버전 기반으로 arcus-zookeeper를 다시 만든다면, 자연스럽게 해결되는 부분임을 참고 바랍니다.

따라서, 현재 PR을 반영하고자 한다면, 현재 RP의 commit 메세지에 ZooKeeper 어떤 버전부터 현재 PR의 사항이 반영된 상태 인지를 명시해 주면 좋겠습니다. 향후, ZooKeeper 3.5 버전으로 갈아타는 경우 현재 PR의 적용 여부를 확인하기 위함입니다.

jh-jeong commented 7 years ago

@jhpark816 커밋 메세지를 수정해서 다시 커밋했는데, 이렇게 하면 될까요?

jhpark816 commented 7 years ago

@jh-jeong 현재 PR을 close하시고 single commit으로 해서 다시 PR 보내주세요. ZOOKEEPER-2646이 들어간 commit message로 commit해 주세요.

jh-jeong commented 7 years ago

네, 알겠습니다. 감사합니다.