clamav/clamav.patch

Tue, 29 Mar 2011 20:04:34 +0200

author
Michael Schloh von Bennewitz <michael@schloh.com>
date
Tue, 29 Mar 2011 20:04:34 +0200
changeset 334
4a34d7a82eab
parent 233
f5f6e93f3dc6
child 498
c6b0e6c7f67d
permissions
-rw-r--r--

Rework package yet again, correcting and introducing new buildconf logic:
Conditionally disable bootstrap stage comparison correctly, correct
english grammar, better find system as(1) and ld(1), indotruce detailed
optimization option messages, more completely guess cpu types, allow
profiled bootstrapping without a preinstalled GCC because many other
compilers have long since implemented 64-bit arithmetic, instruct make
to build sequentially (not in sparallel) when building a profiled
bootstrap as GCC online documents recommend, and generally improve
comment blocks.

The single most important correction in this changeset relates to the
GCC changed optimization policy since at least GCC 4.5, in which -march
is always passed and not always correctly guessed. In the case of this
package, allowing GCC to guess the architecture leads to wild build
errors at various subcomponents (zlib, libgcc, libiberty...) and
bootstrap stages. It seems quite platform specific, and the safest
approach to correcting this seems to be explicitly always specifying the
-march argument when bootstrapping GCC. Because the best choice 'native'
is not available when bootstrapping using a foreign (non GCC) compiler,
a guess is made according to rpmmacros l_platform in that case.

It is questionable as to whether these recent optimization changes
on the part of GCC or this package are compatible with each other,
or if either are complete or correct at all. At least applying these
corrections allows this package to build again in most cases test.

     1 diff -Nau clamav-milter/clamav-milter.c.orig clamav-milter/clamav-milter.c
     2 Index: clamav-milter/clamav-milter.c
     3 --- clamav-milter/clamav-milter.c.orig	2009-10-28 16:34:13.000000000 +0100
     4 +++ clamav-milter/clamav-milter.c	2009-11-01 15:26:46.212861448 +0100
     5 @@ -32,7 +32,7 @@
     6  #include <syslog.h>
     7  #endif
     8  #include <time.h>
     9 -#include <libmilter/mfapi.h>
    10 +#include "milter/mfapi.h"
    12  #include "clamav.h"
    14 diff -Nau clamav-milter/clamfi.c.orig clamav-milter/clamfi.c
    15 Index: clamav-milter/clamfi.c
    16 --- clamav-milter/clamfi.c.orig	2009-10-28 16:34:13.000000000 +0100
    17 +++ clamav-milter/clamfi.c	2009-11-01 15:26:46.214251049 +0100
    18 @@ -29,7 +29,7 @@
    19  #include <unistd.h>
    20  #include <ctype.h>
    22 -#include <libmilter/mfapi.h>
    23 +#include "milter/mfapi.h"
    25  #include "shared/optparser.h"
    26  #include "shared/output.h"
    27 diff -Nau clamav-milter/clamfi.h.orig clamav-milter/clamfi.h
    28 Index: clamav-milter/clamfi.h
    29 --- clamav-milter/clamfi.h.orig	2009-07-06 15:21:58.000000000 +0200
    30 +++ clamav-milter/clamfi.h	2009-11-01 15:26:46.214882752 +0100
    31 @@ -22,7 +22,7 @@
    32  #define _CLAMFI_H
    34  #include "shared/optparser.h"
    35 -#include <libmilter/mfapi.h>
    36 +#include "milter/mfapi.h"
    38  extern uint64_t maxfilesize;
    39  extern int addxvirus;
    40 diff -Nau configure.orig configure
    41 Index: configure
    42 --- configure.orig	2009-10-28 16:34:13.000000000 +0100
    43 +++ configure	2009-11-01 15:29:06.145022137 +0100
    44 @@ -16692,9 +16692,6 @@
    46          save_LIBS="$LIBS"
    47      CLAMAV_MILTER_LIBS="$CLAMAV_MILTER_LIBS -lpthread"
    48 -    if test -d /usr/lib/libmilter ; then
    49 -	CLAMAV_MILTER_LIBS="$CLAMAV_MILTER_LIBS -L/usr/lib/libmilter"
    50 -    fi
    51      LIBS="$LIBS -lmilter $CLAMAV_MILTER_LIBS"
    52      { $as_echo "$as_me:${as_lineno-$LINENO}: checking for mi_stop in -lmilter" >&5
    53  $as_echo_n "checking for mi_stop in -lmilter... " >&6; }
    54 @@ -16842,9 +16839,9 @@
    55  fi
    57      LIBS="$save_LIBS"
    58 -    for ac_header in libmilter/mfapi.h
    59 +    for ac_header in milter/mfapi.h
    60  do :
    61 -  ac_fn_c_check_header_mongrel "$LINENO" "libmilter/mfapi.h" "ac_cv_header_libmilter_mfapi_h" "$ac_includes_default"
    62 +  ac_fn_c_check_header_mongrel "$LINENO" "milter/mfapi.h" "ac_cv_header_libmilter_mfapi_h" "$ac_includes_default"
    63  if test "x$ac_cv_header_libmilter_mfapi_h" = x""yes; then :
    64    cat >>confdefs.h <<_ACEOF
    65  #define HAVE_LIBMILTER_MFAPI_H 1
    66 diff -Nau libclamav/others.c.orig libclamav/others.c
    67 Index: libclamav/others.c
    68 --- libclamav/others.c.orig	2009-10-28 16:34:13.000000000 +0100
    69 +++ libclamav/others.c	2009-11-01 15:26:46.257883269 +0100
    70 @@ -177,8 +177,10 @@
    71      if(is_rar_initd) return;
    72      is_rar_initd = 1;
    74 +#if 0
    75      rhandle = lt_dlfind("libclamunrar_iface", "unrar");
    76      if (!rhandle)
    77 +#endif
    78  	return;
    80      if (!(cli_unrar_open = (int(*)(int, const char *, unrar_state_t *))lt_dlsym(rhandle, "libclamunrar_iface_LTX_unrar_open")) ||
    81 diff -Nau shared/output.c.orig shared/output.c
    82 Index: shared/output.c
    83 --- shared/output.c.orig	2009-10-28 16:34:13.000000000 +0100
    84 +++ shared/output.c	2009-11-01 15:26:46.259056039 +0100
    85 @@ -67,7 +67,7 @@
    86  pthread_mutex_t logg_mutex = PTHREAD_MUTEX_INITIALIZER;
    87  #endif
    89 -#ifdef  C_LINUX
    90 +#if 0
    91  #include <libintl.h>
    92  #include <locale.h>

mercurial