hide-dlsym-error.patch

   1 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=830912
   2
   3 Package: fakeroot
   4 Version: 1.21-1
   5 Followup-For: Bug #830912
   6 User: ubuntu-devel@lists.ubuntu.com
   7 Usertags: origin-ubuntu yakkety ubuntu-patch
   8
   9 In Ubuntu, the attached patch was applied to achieve the following:
  10
  11 Hide the warnings, at least for now. This should bring us back to glibc 2.23
  12 state, AFAICT.
  13
  14   * Add hide-dlsym-error.patch to hide the errors from dlsym() unless
  15     debugging was enabled. This makes builds less noisy and fixes failures
  16     in APT test suite with glibc 2.24, which started reporting errors for
  17     dlsym()
  18
  19
  20 Thanks for considering the patch.
  21
  22 Description: Hide error from dlsym()
  23  dlsym(), starting in glibc 2.24 actually reports errors. In our case,
  24  we try to get ACL functions which are not in the glibc. This causes
  25  failures in test suites, so hide those messages for non-debugging
  26  purposes for now. It also makes the build logs annoying to read.
  27 Author: Julian Andres Klode &lt;juliank@ubuntu.com&gt;
  28 Origin: vendor
  29 Bug-Debian: <a href="https://bugs.debian.org/830912">https://bugs.debian.org/830912</a>
  30 Forwarded: no
  31 Last-Update: 2016-08-12
  32
  33 --- a/libfakeroot.c
  34 +++ b/libfakeroot.c
  35 @@ -256,10 +256,16 @@ void load_library_symbols(void){
  36   /* clear dlerror() just in case dlsym() legitimately returns NULL */
  37      msg = dlerror();
  38      *(next_wrap[i].doit)=dlsym(get_libc(), next_wrap[i].name);
  39 +
  40      if ( (msg = dlerror()) != NULL){
  41 -      fprintf (stderr, "dlsym(%s): %s\n", next_wrap[i].name, msg);
  42 -/*    abort ();*/
  43 +#ifdef LIBFAKEROOT_DEBUGGING
  44 +      if (fakeroot_debug) {
  45 +        fprintf (stderr, "dlsym(%s): %s\n", next_wrap[i].name, msg);
  46 +/*      abort ();*/
  47 +      }
  48 +#endif
  49      }
  50 +
  51    }
  52  }
  53