ashish21590 / redis

Automatically exported from code.google.com/p/redis
BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

NetBSD builds #578

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Redis 2.2.8
NetBSD 5.1

A. Specify a requirements page, ie. stipulate that GNU make is required to 
build, BSD systems make doesn't work without modification; stipulate tclsh et 
al require to run tests.

B. Several build warnings and errors. Solution:

Modify fmacros.h...

1. #define _NETBSD_SOURCE resolves warnings about missing strcasecmp() 
declarations and errors WRT redis.c line 1661 undefined SA_ macros.

2. #define _XOPEN_SOURCE 700 for NetBSD for completeness; the _NETBSD_SOURCE 
should be sufficient to resolve the missing SA_ macros.

Patch...

*** src/fmacros.h.orig  Tue Jun  7 10:45:24 2011
--- src/fmacros.h       Tue Jun  7 10:49:35 2011
***************
*** 2,9 ****
  #define _REDIS_FMACRO_H

  #define _BSD_SOURCE

! #ifdef __linux__
  #define _XOPEN_SOURCE 700
  #else
  #define _XOPEN_SOURCE
--- 2,10 ----
  #define _REDIS_FMACRO_H

  #define _BSD_SOURCE
+ #define _NETBSD_SOURCE

! #if defined(__linux__) || defined(__NetBSD__)
  #define _XOPEN_SOURCE 700
  #else
  #define _XOPEN_SOURCE

Original issue reported on code.google.com by ach...@gmail.com on 7 Jun 2011 at 9:00

GoogleCodeExporter commented 8 years ago
Got done with part of A. See this: https://github.com/antirez/redis/pull/77

Original comment by snpan...@gmail.com on 14 Sep 2011 at 8:53