delete check for sys/cdefs.h from configure

RESOLVED FIXED in Firefox 43

Status

RESOLVED FIXED
3 years ago
6 months ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla43

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
We never consult HAVE_SYS_CDEFS_H anywhere.  All the places we do
include <sys/cdefs.h> do it unconditionally.  So let's get rid of this
useless check.
(Assignee)

Comment 1

3 years ago
Created attachment 8644941 [details] [diff] [review]
delete check for sys/cdefs.h from configure
Attachment #8644941 - Flags: review?(mshal)
Comment on attachment 8644941 [details] [diff] [review]
delete check for sys/cdefs.h from configure

Wow, the sys/cdefs.h check has been there since the "Free the lizard" commit as:

echo ${ECHO_FLAG} "Do we have either <cdefs.h> or <sys/cdefs.h>....${ENDER}"
if test ! -f /usr/include/cdefs.h -a ! -f /usr/include/sys/cdefs.h
then
       MACROS="${MACROS} -DNO_CDEFS_H"
       echo " no"
else
       echo " yes"
fi
Attachment #8644941 - Flags: review?(mshal) → review+
https://hg.mozilla.org/mozilla-central/rev/1955a6afd353
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43

Updated

6 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.