[MPlayer-cvslog] r22797 - trunk/libdha/Makefile

[Ivan Kalvachev]

2007/3/24, diego <subversion at mplayerhq.hu>:
> Author: diego
> Date: Sat Mar 24 14:25:51 2007
> New Revision: 22797
>
> Modified:
>    trunk/libdha/Makefile
>
> Log:
> Remove OpenBSD linking hack.  If this is (still) necessary, it needs to be
> done properly from configure.
>
>
> Modified: trunk/libdha/Makefile
> ==============================================================================
> --- trunk/libdha/Makefile       (original)
> +++ trunk/libdha/Makefile       Sat Mar 24 14:25:51 2007
> @@ -16,12 +16,6 @@ SRCS_MPLAYER = libdha.c \
>
>  CFLAGS = -fPIC
>
> -ifeq ($(TARGET_OS),OpenBSD)
> -ifeq ($(TARGET_ARCH_X86),yes)
> -EXTRALIBS += -li386
> -endif
> -endif


It seems like libi386 is required for  i386_iopl() in pci_openbsd.c ,
#include "machine/sysarch.h".

In openbsd configure includes libi386 when win32loader is enabled. It
is always included  for netbsd (32bit arch).

Fix should be trivial.

Did you made a test compilation on your OpenBSD box before committing?
Because your commit message implies you have no idea if this change is
even supposed to work.