Motion - Bug Report 2006x 07x 08x 070417

BUG: configure defines a redundant freebsd for motion on FreeBSD


On FreeBSD the compiler defines by default __FreeBSD__.
Defining __freebsd__ separately in configure is redundant and bad style.
If you need to know the OS version, include <sys/param.h> to find it
defined like this...

#define __FreeBSD_version 601101

That is major == 6, minor == 1, and some patch stuff which should never
be a problem to applications, because the API and ABI remain the same.

Change the #ifdef and #if defined in the code to refer __FreeBSD__ and
forget about the redundant definition in the configure phase.

CFLAGS:  -D__freebsd__ 

Environment

Motion version: 3.2.6
ffmpeg version:  
Shared libraries: postgresql
Server OS: FreeBSD 6.1-STABLE

-- JukkaUkkonen - 08 Jul 2006

Follow up

Thanks , i'll try to test your advice and see if it works .

-- AngelCarpintero - 09 Jul 2006

Fix record

Fixed in svn 112.

-- AngelCarpintero - 08 Oct 2006

In 3.2.7

-- KennethLavrsen - 20 Oct 2006
Topic revision: r4 - 20 Oct 2006, KennethLavrsen
Copyright © 1999-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Please do not email Kenneth for support questions (read why). Use the Support Requests page or join the Mailing List.
This website only use harmless session cookies. See Cookie Policy for details. By using this website you accept the use of these cookies.