Skip to content

fix build issue on macOS: unistd.h and complex.h #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jiabinf wants to merge 7 commits into
base: master
Choose a base branch
from
Open

fix build issue on macOS: unistd.h and complex.h #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
bb96c91
remove unused files
Oct 19, 2018
8cf43b5
testing macOS build
jiabinf Nov 30, 2020
9eb4498
add HAVE_UNISTD_H
jiabinf Nov 30, 2020
25148fc
Revert "remove unused files"
jiabinf Nov 30, 2020
73eab37
fix complex.h
jiabinf Nov 30, 2020
950d978
remove fp.h to adapt new Xcode
jiabinf Sep 4, 2025
4168a45
include unistd.h to fix macos build
jiabinf Sep 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions deps/common.gyp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,11 @@
{
'xcode_settings': {
'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
}
}
},
'defines': [
'HAVE_UNISTD_H',
],
},
],
['OS=="win"',
{
Expand Down
16 changes: 3 additions & 13 deletions deps/libpng/pngpriv.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,18 +436,8 @@
*/
# include <float.h>

# if (defined(__MWERKS__) && defined(macintosh)) || defined(applec) || \
defined(THINK_C) || defined(__SC__) || defined(TARGET_OS_MAC)
/* We need to check that <math.h> hasn't already been included earlier
* as it seems it doesn't agree with <fp.h>, yet we should really use
* <fp.h> if possible.
*/
# if !defined(__MATH_H__) && !defined(__MATH_H) && !defined(__cmath__)
# include <fp.h>
# endif
# else
# include <math.h>
# endif
# include <math.h>

# if defined(_AMIGA) && defined(__SASC) && defined(_M68881)
/* Amiga SAS/C: We must include builtin FPU functions when compiling using
* MATH=68881
Expand Down Expand Up @@ -910,7 +900,7 @@ PNG_INTERNAL_FUNCTION(png_voidp,png_malloc_base,(png_const_structrp png_ptr,
#if defined(PNG_TEXT_SUPPORTED) || defined(PNG_sPLT_SUPPORTED) ||\
defined(PNG_STORE_UNKNOWN_CHUNKS_SUPPORTED)
/* Internal array allocator, outputs no error or warning messages on failure,
* just returns NULL.
* just returns NULL.
*/
PNG_INTERNAL_FUNCTION(png_voidp,png_malloc_array,(png_const_structrp png_ptr,
int nelements, size_t element_size),PNG_ALLOCATED);
Expand Down
2 changes: 1 addition & 1 deletion deps/pHash/ph_fft.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
#define PI 3.1415926535897932

#include <math.h>
#include </usr/include/complex.h>
#include <complex.h>
#include <stdlib.h>

int fft(double *x, int N, complex double *X);
Expand Down
2 changes: 2 additions & 0 deletions deps/zlib/gzguts.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
* For conditions of distribution and use, see copyright notice in zlib.h
*/

#include <unistd.h>

#ifdef _LARGEFILE64_SOURCE
# ifndef _LARGEFILE_SOURCE
# define _LARGEFILE_SOURCE 1
Expand Down