[MacPorts] #63358: glib2 @2.65.0: extra bit added on to universal.patch

MacPorts noreply at macports.org
Wed Aug 11 20:27:50 UTC 2021


#63358: glib2 @2.65.0: extra bit added on to universal.patch
--------------------+--------------------------------
 Reporter:  kencu   |      Owner:  (none)
     Type:  defect  |     Status:  new
 Priority:  Normal  |  Milestone:
Component:  ports   |    Version:
 Keywords:          |       Port:  glib2, glib2-devel
--------------------+--------------------------------
 While looking over the recent glib2 update, and considering the glib 2.67
 update to come, I noticed this bit added onto {{{universal.patch}}}:

 {{{
 --- meson.build.orig    2019-11-17 14:26:49.000000000 -0300
 +++ meson.build 2019-11-17 14:27:32.000000000 -0300
 @@ -1584,17 +1584,9 @@
    glibconfig_conf.set(d[1], val)
  endforeach

 -# We need a more robust approach here...
  host_cpu_family = host_machine.cpu_family()
 -if host_cpu_family == 'x86' or host_cpu_family == 'x86_64' or
 host_cpu_family == 's390' or host_cpu_family == 's390x' or
 host_cpu_family.startswith('arm') or host_cpu_family == 'aarch64' or
 host_cpu_family.startswith('crisv32') or
 host_cpu_family.startswith('etrax')
 -  glib_memory_barrier_needed = false
 -elif host_cpu_family.startswith('sparc') or
 host_cpu_family.startswith('alpha') or
 host_cpu_family.startswith('powerpc') or host_cpu_family == 'ia64'
 -  glib_memory_barrier_needed = true
 -else
 -  warning('Unknown host cpu: ' + host_cpu_family)
 -  glib_memory_barrier_needed = true
 -endif
 -glibconfig_conf.set('G_ATOMIC_OP_MEMORY_BARRIER_NEEDED',
 glib_memory_barrier_needed)
 +glib_is_powerpc = '#if (defined(__PPC__) || defined(__PPC64__) ||
 defined(__powerpc__) || defined(__powerpc64__) || defined(_ARCH_PPC)  ||
 defined(_ARCH_PPC64))\n#ifdef G_ATOMIC_OP_MEMORY_BARRIER_NEEDED\n#undef
 G_ATOMIC_OP_MEMORY_BARRIER_NEEDED\n#endif\n#define
 G_ATOMIC_OP_MEMORY_BARRIER_NEEDED 1\n#else\n#ifdef
 G_ATOMIC_OP_MEMORY_BARRIER_NEEDED\n#undef
 G_ATOMIC_OP_MEMORY_BARRIER_NEEDED\n#endif\n#endif'
 +glibconfig_conf.set('glib_is_powerpc', glib_is_powerpc)

  # We need to decide at configure time if GLib will use real atomic
  # operations ("lock free") or emulated ones with a mutex.  This is
  }}}

 This was all done a few years ago, I note, as the update to glib2 has been
 in the works for a while.

 This patch may well be correct -- it probably as, as all the tests work
 out well on 10.4 PPC to BigSur --  but even if so, it should be probably
 moved out of {{{universal.patch}}} and put into it's own separate
 patchfile so we don't lose track of it, and what it is supposed to be
 doing.

-- 
Ticket URL: <https://trac.macports.org/ticket/63358>
MacPorts <https://www.macports.org/>
Ports system for macOS


More information about the macports-tickets mailing list