Change tests for glibc version #12

병합
SlavekB feat/reverse-glibc-v2-tests 에서 master 로 1 commits 를 머지했습니다 5 년 전
SlavekB 코멘트됨, 5 년 전
소유자

It resolves building with libc libraries other than glibc – for example musl libc.

It resolves building with libc libraries other than glibc – for example [musl libc](https://www.musl-libc.org/).
SlavekB R14.0.7 release 5 년 전 마일스톤을 추가하였습니다.
SlavekB added the PR/rfc label 5 년 전
MicheleC 코멘트됨, 5 년 전
소유자

PR looks good, nevertheless I wonder if it still makes sense to check for glibc < 2. Based on release history, glibc 2 was release on 1997-01-26, so we can safely assume that all the systems we are building TDE with glibc are using a version >=2. IMO we could simplify the code into

#define QT_SOCKLEN_T		socklen_t
PR looks good, nevertheless I wonder if it still makes sense to check for glibc < 2. Based on release history, glibc 2 was release on 1997-01-26, so we can safely assume that all the systems we are building TDE with glibc are using a version >=2. IMO we could simplify the code into ``` #define QT_SOCKLEN_T socklen_t ```
SlavekB 코멘트됨, 5 년 전
포스터
소유자

You're right, we can now assume that using glibc will not be older than version 2.

Patch updated.

You're right, we can now assume that using glibc will not be older than version 2. Patch updated.
Ghost closed this pull request 5 년 전
Ghost reopened this pull request 5 년 전
MicheleC 이 변경사항을 승인하였습니다. 5 년 전
MicheleC left a comment
소유자

LGTM.

LGTM.
SlavekB closed this pull request 5 년 전
SlavekB 삭제된 브랜치 feat/reverse-glibc-v2-tests 5 년 전
SlavekB removed the PR/rfc label 5 년 전
The pull request has been merged as 1aa221a2bc.
로그인하여 이 대화에 참여
No reviewers
마일스톤 없음
담당자 없음
참여자 2명
알림
마감일

마감일이 설정되지 않았습니다.

의존성

No dependencies set.

Reference: TDE/tqt3#12
불러오는 중...
아직 콘텐츠가 없습니다.