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

[APIS-1007] 11.4 드라이버 호환성 테스트 - ODBC (#3) #76

Merged
merged 2 commits into from
Dec 27, 2024

Conversation

Kang-dot
Copy link
Contributor

@Kang-dot Kang-dot commented Dec 24, 2024

http://jira.cubrid.org/browse/APIS-1007

  • purpose
    ODBC드라이버와 CUBRID 11.4의 호환성 테스트

  • implementation
    호환성 테스트외에 cci와 빌드 환경 업데이트를 병행함

  • remarks
    N/A

* [APIS-1008] ODBC submodule(cci) 버전 업데이트

* [APIS-1008] ODBC 프로젝트 파일 수정

- ODBC에서 사용하는 windows sdk 버전 수정 (10.0.17763.0)
- build.bat 파일 수정

* [APIS-1008] cci 로드 스크립트 수정

- vs2017로 cci를 로드해서 빌드하던 스크립트를 기존 스크립트에 반영

* [APIS-1008] batch 파일 수정

- nsis 환경 변수를 사용하도록 수정
- release|x64를 테스트에 사용하도록 수정

* [APIS-1008] odbc 버전 변경

- 11.4.0.0062로 버전 변경
@Kang-dot Kang-dot requested a review from hwany7seo December 24, 2024 06:56
@Kang-dot Kang-dot self-assigned this Dec 24, 2024
@@ -1,5 +1,5 @@
#define MAJOR_VERSION 11
#define MINOR_VERSION 3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

드라이버는 이제 Server 버전을 따르지 않습니다.
그러므로 CCI 또는 ODBC 변경사항이 크지 않다면 PATCH_VERSION만 올려주시면 됩니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네 확인했습니다 버전을 "11.3.1.0062"로 수정하겠습니다

@@ -15,7 +14,4 @@ copy installer\installer-unicode.nsi %INSTALL_DIRS%\installer-unicode.nsi
copy installer\license.txt %INSTALL_DIRS%\license.txt
copy installer\README.txt %INSTALL_DIRS%\README.txt

c:
cd C:\NSIS

makensis %WORKSPACE%\%INSTALL_DIRS%\installer-unicode.nsi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

일반 사용자도 빌드 할 수 있기 때문에
VS2017COMNTOOLS나 NSIS가 환경 변수에 등록할수 있도록
환경 변수를 체크하는 것이 좋을 것 같습니다. (체크후 설정되어 있지 않으면 진행되지 않도록)
그리고 makensis앞에도 환경 변수가 있으면 좋겠습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네 확인했습니다 다만 nsis 환경 변수 같은 경우에는 보통 변수를 따로 잡지 않고 Path 내부에 nsis 디렉토리만 설정해서 사용하는 것 같아서 makensis명령이 실행 안됐을떄만 if 분기로 아래 예시처럼 처리하도록 수정하겠습니다

makensis

if %ERRORLEVEL% NEQ 0 (
  echo error: cannot find nsis
  goto :EOF
)

echo nsis launch success

- 드라이버 버전을 11.3.1.0062로 수정
- 빌드 스크립트에서 vs2017, nsis 환경 변수를 확인하도록 설정
@Kang-dot Kang-dot requested a review from hwany7seo December 26, 2024 06:05
@Kang-dot Kang-dot merged commit e9bbfdf into CUBRID:develop Dec 27, 2024
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

Successfully merging this pull request may close these issues.

2 participants