Change tests for glibc version #12

已合併
SlavekB 5 年之前 將 1 次代碼提交從 feat/reverse-glibc-v2-tests 合併至 master
所有者

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 年之前
所有者

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 ```
發佈者
所有者

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 approved these changes 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
未選擇里程碑
No Assignees
2 參與者
訊息
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tqt3#12
Loading…
尚未有任何內容