Building apache2handler with APR 1.7.5 fails if compiler doesn't support __has_attribute

Description

Not quite sure where to report this yet, so I'm starting here.

A commit exhibiting the problem has made it into latest APR 1.7.5. The assumption that __has_attribute would be predefined is valid in theory, but not in practise. E.g. ap_config.h defines __has_attribute to 0 in case it is not defined. php-src does the same at least in TSRM.h, zend_portability.h and zend_cpuinfo.h. In my opinion, this is bad practise and should be fixed; see e.g. xxhash.h for a proper solution. Of course, the existing definitions might be relied upon by downstream consumers of our headers, so changing this would be an API break.

Anyhow, coming back to the issue at hand, I wonder whether apache2handler is even supposed to include ap_config.h explicitly. If not, we can neither blame httpd nor APR, since latest httpd can apparently be successfully built against latest APR with a compiler not supporting __has_attribute.

PHP Version

any

Operating System

any

Bắt đầu bằng cách theo dõi quá trình build apache2handler và việc nó include ap_config.h, sau đó so sánh cách xử lý __has_attribute liên quan với APR 1.7.5 và các định nghĩa trong TSRM.h, zend_portability.h và zend_cpuinfo.h. Tái hiện lỗi bằng một compiler không có __has_attribute và xác định liệu nguyên nhân nằm ở việc include hay ở hành vi tương thích; được xem là hoàn tất khi build bị ảnh hưởng thành công mà không phá vỡ khả năng tương thích header đã nêu.