glib: remove perl from REQUIRED_PACKAGES; add few fixes to improve test res...

Marcel Telka

2022-08-25 c815d077d24cbd1f914fed75cc43c5821f3f4a19

glib: remove perl from REQUIRED_PACKAGES; add few fixes to improve test results

 components/library/glib/Makefile

@@ -22,7 +22,7 @@
COMPONENT_NAME=         glib
COMPONENT_MJR_VERSION=    2.62
COMPONENT_MNR_VERSION=    6
COMPONENT_REVISION=     2
COMPONENT_REVISION=     3
# Note: When updating this component make sure that it still works with our HAL.
# Version 2.66.x is known for its problems eg. when plugging/unplugging USB devices under X11.
# 2.62.6 is the last known working version (2.63.x, 2.64.x untested).
@@ -38,53 +38,48 @@
COMPONENT_PROJECT_URL=  https://www.gtk.org/
COMPONENT_FMRI=         library/glib2

PATH=$(PATH.gnu)

include $(WS_MAKE_RULES)/common.mk

PYTHON_VERSION = 3.5

LDFLAGS += -lsocket -lsecdb -lnsl
# dbus-daemon is in /usr/lib
PATH = $(PATH.gnu):/usr/lib

# GCC 6
CFLAGS += -Wno-error=format-nonliteral
# socketpair(3socket) requires -lsocket -lnsl
LDFLAGS += -lsocket -lnsl

# Support for hexadecimal strings in strtod() is available in C99 mode only,
# but we cannot use -std=c99 because float/double support is broken there
# (https://www.illumos.org/issues/14935)
CFLAGS += -std=gnu99

# We need standard conforming getpwnam_r()/getpwuid_r()
CPPFLAGS += $(CPP_XPG6MODE)

CONFIGURE_OPTIONS.32 = --sysconfdir=/etc
CONFIGURE_OPTIONS.64 = --sysconfdir=/etc/$(MACH64)
# Some warnings are treated as errors by default but we do not want that
CFLAGS += -Wno-error=format-nonliteral
CFLAGS += -Wno-error=format-security

# We do not have getxattr() so disable xattr
CONFIGURE_OPTIONS += -Dxattr=false
CONFIGURE_OPTIONS += -Dfam=true
CONFIGURE_OPTIONS += -Dman=true

CONFIGURE_ENV    += PERL=$(PERL)
CONFIGURE_ENV    += PYTHON=$(PYTHON)
CONFIGURE_ENV    += CPPFLAGS="$(CPPFLAGS)"

# Dtrace flags
DFLAGS.64 = -64
COMPONENT_BUILD_ENV += DFLAGS=$(DFLAGS.$(BITS))

COMPONENT_POST_CONFIGURE_ACTION= \
  ( cd $(@D); $(GSED) -i "s/_FILE_OFFSET_BITS=64/_FILE_OFFSET_BITS=$(BITS)/g" ./build.ninja )

# Otherwise tests fail
#ENV = /usr/bin/env -i

# We need to install libraries and set correct LD_LIBRARY_PATH before test
$(TEST_TARGET): $(INSTALL_TARGET)

COMPONENT_TEST_ENV.32 = LD_LIBRARY_PATH=$(PROTO_DIR)/usr/lib PATH=$(PATH):/usr/lib
COMPONENT_TEST_ENV.64 = LD_LIBRARY_PATH=$(PROTO_DIR)/usr/lib/$(MACH64) PATH=$(PATH)/usr/lib/$(MACH64)
COMPONENT_TEST_ENV = $(COMPONENT_TEST_ENV.$(BITS))
COMPONENT_TEST_ENV += LC_ALL=C.UTF-8

COMPONENT_TEST_MASTER = $(COMPONENT_TEST_RESULTS_DIR)/results-all.master
COMPONENT_TEST_TRANSFORMER = $(NAWK)
COMPONENT_TEST_TRANSFORMS = "'/^(Fail|Ok|Expected Fail|Unexpected Pass|Skipped|Timeout)/'"

REQUIRED_PACKAGES += runtime/perl-522
# dbus-daemon is needed for testing
REQUIRED_PACKAGES += system/library/dbus

# update-desktop-database is needed for testing
REQUIRED_PACKAGES += library/desktop/desktop-file-utils

# Auto-generated dependencies
REQUIRED_PACKAGES += library/file-monitor/gamin
REQUIRED_PACKAGES += library/libffi

 components/library/glib/glib2-charset-alias.p5m

@@ -14,6 +14,7 @@
#

set name=pkg.fmri value=pkg:/$(COMPONENT_FMRI)/charset-alias@$(IPS_COMPONENT_VERSION),$(BUILD_VERSION)
set name=pkg.human-version value=$(HUMAN_VERSION)
set name=pkg.summary value="$(COMPONENT_SUMMARY) - just the /usr/lib/.../charset.alias file"
set name=info.classification value="org.opensolaris.category.2008:Desktop (GNOME)/Libraries"
set name=info.upstream-url value=$(COMPONENT_PROJECT_URL)

 components/library/glib/glib2.p5m

@@ -16,6 +16,7 @@
#

set name=pkg.fmri value=pkg:/$(COMPONENT_FMRI)@$(IPS_COMPONENT_VERSION),$(BUILD_VERSION)
set name=pkg.human-version value=$(HUMAN_VERSION)
set name=pkg.summary value="$(COMPONENT_SUMMARY)"
set name=info.classification value="$(COMPONENT_CLASSIFICATION)"
set name=info.upstream-url value=$(COMPONENT_PROJECT_URL)

 components/library/glib/manifests/sample-manifest.p5m

@@ -14,6 +14,7 @@
#

set name=pkg.fmri value=pkg:/$(COMPONENT_FMRI)@$(IPS_COMPONENT_VERSION),$(BUILD_VERSION)
set name=pkg.human-version value=$(HUMAN_VERSION)
set name=pkg.summary value="$(COMPONENT_SUMMARY)"
set name=info.classification value="$(COMPONENT_CLASSIFICATION)"
set name=info.upstream-url value=$(COMPONENT_PROJECT_URL)

 components/library/glib/patches/21-test-one-half-convert.patch

New file
@@ -0,0 +1,33 @@
--- glib-2.62.6/glib/tests/convert.c.orig
+++ glib-2.62.6/glib/tests/convert.c
@@ -82,9 +82,14 @@ test_one_half (void)
            &bytes_read, &bytes_written,
            &error);
 
+  /* Characters that do not exist in target codeset are converted to
+   * implementation-defined character by underlying iconv().  On illumos it is
+   * '?'.  This is detected by g_convert() so error is set and NULL is
+   * returned, but both bytes_read and bytes_written are left updated.  More
+   * details: https://gitlab.gnome.org/GNOME/glib/-/issues/1303 */
   g_assert_error (error, G_CONVERT_ERROR, G_CONVERT_ERROR_ILLEGAL_SEQUENCE);
-  g_assert_cmpint (bytes_read, ==, 0);
-  g_assert_cmpint (bytes_written, ==, 0);
+  g_assert_cmpint (bytes_read, ==, 2);
+  g_assert_cmpint (bytes_written, ==, 1);
   g_assert_cmpstr (out, ==, NULL);
   g_clear_error (&error);
   g_free (out);
@@ -95,10 +100,12 @@ test_one_half (void)
                  &bytes_read, &bytes_written,
                  &error);
 
+  /* This is similar case as above so the "vulgar fraction one half" got
+   * converted to '?' and the fallback was not used. */
   g_assert_no_error (error);
   g_assert_cmpint (bytes_read, ==, 2);
   g_assert_cmpint (bytes_written, ==, 1);
-  g_assert_cmpstr (out, ==, "a");
+  g_assert_cmpstr (out, ==, "?");
   g_free (out);
 }
 

 components/library/glib/patches/22-zoneinfo.patch

New file
@@ -0,0 +1,11 @@
--- glib-2.62.6/glib/gtimezone.c.orig
+++ glib-2.62.6/glib/gtimezone.c
@@ -416,7 +416,7 @@
 
   tzdir = getenv ("TZDIR");
   if (tzdir == NULL)
-    tzdir = "/usr/share/zoneinfo";
+    tzdir = "/usr/share/lib/zoneinfo";
 
   /* identifier can be a relative or absolute path name;
      if relative, it is interpreted starting from /usr/share/zoneinfo

 components/library/glib/patches/23-vasprintf.patch

New file
@@ -0,0 +1,18 @@
illumos asprintf()/vasprintf() does not work as expected by glib.
See https://www.illumos.org/issues/14933

--- glib-2.62.6/glib/gprintf.c.orig
+++ glib-2.62.6/glib/gprintf.c
@@ -325,12 +325,6 @@
   if (len < 0)
     *string = NULL;
 
-#elif defined (HAVE_VASPRINTF)
-
-  len = vasprintf (string, format, args);
-  if (len < 0)
-    *string = NULL;
-
 #else
 
   {

 components/library/glib/patches/24-timer-test-double.patch

New file
@@ -0,0 +1,30 @@
Without this the following asserts fires.
The root cause is unclear.

--- glib-2.62.6/glib/tests/timer.c.orig
+++ glib-2.62.6/glib/tests/timer.c
@@ -25,6 +25,7 @@
 #define GLIB_DISABLE_DEPRECATION_WARNINGS 1
 
 #include "glib.h"
+#include <stdio.h>
 
 static void
 test_timer_basic (void)
@@ -38,6 +39,7 @@
   elapsed = g_timer_elapsed (timer, &micros);
 
   g_assert_cmpfloat (elapsed, <, 1.0);
+  printf("elapsed * 1e6: %f\n", elapsed * 1e6);
   g_assert_cmpuint (micros, ==, ((guint64)(elapsed * 1e6)) % 1000000);
 
   g_timer_destroy (timer);
@@ -57,6 +59,8 @@
   g_usleep (100);
   elapsed2 = g_timer_elapsed (timer, NULL);
 
+  printf("elapsed: %f\n", elapsed);
+  printf("elapsed2: %f\n", elapsed2);
   g_assert_cmpfloat (elapsed, ==, elapsed2);
 
   g_timer_destroy (timer);

 components/library/glib/patches/25-slow-tests.patch

New file
@@ -0,0 +1,40 @@
--- glib-2.62.6/glib/tests/meson.build.orig
+++ glib-2.62.6/glib/tests/meson.build
@@ -30,6 +30,7 @@
   'gwakeup' : {
     'source' : ['gwakeuptest.c', '../gwakeup.c'],
     'install' : false,
+    'suite' : ['slow'],
   },
   'hash' : {},
   'hmac' : {},
@@ -39,7 +40,9 @@
   'list' : {},
   'logging' : {},
   'macros' : {},
-  'mainloop' : {},
+  'mainloop' : {
+    'suite' : ['slow'],
+  },
   'mappedfile' : {},
   'markup' : {},
   'markup-parse' : {},
@@ -101,7 +104,9 @@
   'utils' : {},
   'unicode' : {},
   'uri' : {},
-  '1bit-mutex' : {},
+  '1bit-mutex' : {
+    'suite' : ['slow'],
+  },
   '1bit-emufutex' : {
     'source' : '1bit-mutex.c',
     'c_args' : ['-DTEST_EMULATED_FUTEX'],
@@ -128,6 +133,7 @@
       'source' : ['gwakeuptest.c', '../gwakeup.c'],
       'c_args' : ['-DTEST_EVENTFD_FALLBACK'],
       'install' : false,
+      'suite' : ['slow'],
     },
   }
 endif

 components/library/glib/patches/26-iconv-U+FFFF.patch

New file
@@ -0,0 +1,26 @@
See https://www.illumos.org/issues/14934

--- glib-2.62.6/tests/utf8.txt.orig
+++ glib-2.62.6/tests/utf8.txt
@@ -61,10 +61,6 @@
 VALID
 000007ff
 
-￿
-VALID
-0000ffff
-
 ÷¿¿¿
 NOTUNICODE
 001fffff
@@ -95,10 +91,6 @@
 VALID
 0010fffd
 
-􏿿
-VALID
-0010ffff
-
 ô€€
 NOTUNICODE
 00110000

 components/library/glib/patches/27-signal-handler-reset.patch

New file
@@ -0,0 +1,13 @@
--- glib-2.62.6/tests/mapping-test.c.orig
+++ glib-2.62.6/tests/mapping-test.c
@@ -40,6 +40,10 @@
 static void
 handle_usr1 (int signum)
 {
+  /* On illumos the signal disposition is reset to SIG_DFL so we need to set
+   * signal handler again ASAP. */
+  signal (SIGUSR1, handle_usr1);
+
   stop = TRUE;
 }
 

 components/library/glib/patches/28-iconv-no-latin1.patch

New file
@@ -0,0 +1,22 @@
illumos does not support latin1.  Use iso-8859-1 instead.

--- glib-2.62.6/gio/tests/converter-stream.c.orig
+++ glib-2.62.6/gio/tests/converter-stream.c
@@ -1169,7 +1169,7 @@
   gchar *to;
   gchar *from;
 
-  converter = (GConverter *)g_charset_converter_new ("utf-8", "latin1", &error);
+  converter = (GConverter *)g_charset_converter_new ("utf-8", "iso-8859-1", &error);
   g_assert_no_error (error);
   g_object_get (converter,
                 "to-charset", &to,
@@ -1177,7 +1177,7 @@
                 NULL);
 
   g_assert_cmpstr (to, ==, "utf-8");
-  g_assert_cmpstr (from, ==, "latin1");
+  g_assert_cmpstr (from, ==, "iso-8859-1");
 
   g_free (to);
   g_free (from);

 components/library/glib/patches/29-prevent-iconv_close-segfault.patch

New file
@@ -0,0 +1,12 @@
--- glib-2.62.6/gio/gcharsetconverter.c.orig
+++ glib-2.62.6/gio/gcharsetconverter.c
@@ -451,6 +451,9 @@
 
   if (conv->iconv == (GIConv)-1)
     {
+      /* Clear conv->iconv so iconv_close(-1) is not called - see
+       * https://www.illumos.org/issues/14937 */
+      conv->iconv = NULL;
       if (errsv == EINVAL)
     g_set_error (error, G_IO_ERROR, G_IO_ERROR_NOT_SUPPORTED,
              _("Conversion from character set “%s” to “%s” is not supported"),

 components/library/glib/pkg5

@@ -3,11 +3,11 @@
        "SUNWcs",
        "developer/build/meson",
        "developer/build/ninja",
        "library/desktop/desktop-file-utils",
        "library/file-monitor/gamin",
        "library/libffi",
        "library/pcre",
        "library/zlib",
        "runtime/perl-522",
        "runtime/python-35",
        "shell/ksh93",
        "system/library",

 components/library/glib/test/results-all.master

@@ -1,6 +1,6 @@
Ok:                 226 
Ok:                 257 
Expected Fail:      0   
Fail:               13  
Fail:               5   
Unexpected Pass:    0   
Skipped:            0   
Timeout:            0