Closed Bug 1192233 Opened 4 years ago Closed 4 years ago

delete check for sys/cdefs.h from configure

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox43 fixed)

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

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.
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
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.