New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #33957 (closed defect: fixed)

Opened 14 months ago

Last modified 14 months ago

Webkit-gtk 1.2.7 error ‘struct’ tag used in naming ‘union _GMutex’

Reported by: jovo@… Owned by: myschizobuddy@…
Priority: Normal Milestone:
Component: ports Version: 2.0.4
Keywords: Cc: devans@…, vloodo@…, macports@…, elelay@…, jeremyhu@…
Port: webkit-gtk

Description

Webkit-gtk does not install due to a compilation error in GOwnPtr.h (?)

:info:build /opt/local/bin/msgmerge --update pt.po webkit.pot
:info:build ............................ done.
:info:build /opt/local/bin/gmkdir -p ./.deps/DerivedSources
:info:build   CXX    WebCore/plugins/gtk/libwebkit_1_0_la-PluginPackageGtk.lo
:info:build In file included from WebCore/plugins/gtk/PluginPackageGtk.cpp:36:
:info:build ./JavaScriptCore/wtf/gobject/GOwnPtr.h:34: error: ‘struct’ tag used in naming ‘union _GMutex’
:info:build make[1]: *** [WebCore/plugins/gtk/libwebkit_1_0_la-PluginPackageGtk.lo] Error 1
:info:build make[1]: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_www_webkit-gtk/webkit-gtk/work/webkit-1.2.7'
:info:build make: *** [all] Error 2
:info:build make: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_www_webkit-gtk/webkit-gtk/work/webkit-1.2.7'
:info:build shell command " cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_www_webkit-gtk/webkit-gtk/work/webkit-1.2.7" && /usr/bin/make -w all " returned error 2
:error:build Target org.macports.build returned: shell command failed (see log for details)
:debug:build Backtrace: shell command failed (see log for details)
    while executing
"command_exec build"
    (procedure "portbuild::build_main" line 8)
    invoked from within
"$procedure $targetname"
:info:build Warning: the following items did not execute (for webkit-gtk): org.macports.activate org.macports.build org.macports.destroot org.macports.install
:notice:build Log for webkit-gtk is at: /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_ports_www_webkit-gtk/webkit-gtk/main.log

This on Macbook Pro with OS X 1.0.6.8 and Macports 2.0.4

Attachments

main.log (1.9 MB) - added by vloodo@… 14 months ago.
The error log file generated when trying to build webkit-gtk
patch-JavaScriptCore-wtf-gobject-GOwnPtr.h.diff (1023 bytes) - added by elelay@… 14 months ago.
include glib.h directly

Change History

comment:1 Changed 14 months ago by jovo@…

This looks similar to Bugzilla defect https://bugs.webkit.org/show_bug.cgi?id=72085 but I have no idea how to fix this in MP.

comment:2 Changed 14 months ago by macsforever2000@…

  • Priority changed from High to Normal
  • Cc devans@… added; myschizobuddy@… removed
  • Owner changed from macports-tickets@… to myschizobuddy@…

comment:3 follow-up: ↓ 5 Changed 14 months ago by macsforever2000@…

Please clean and attach the main.log.

comment:4 Changed 14 months ago by vloodo@…

  • Cc vloodo@… added

Cc Me!

comment:5 in reply to: ↑ 3 ; follow-up: ↓ 6 Changed 14 months ago by vloodo@…

I am having the same issue (I think). What exactly do you mean by "cleaning" main.log?

Replying to macsforever2000@…:

Please clean and attach the main.log.

comment:6 in reply to: ↑ 5 Changed 14 months ago by vloodo@…

Never mind, I just realized you meant clean the build files first. Silly me.

Replying to vloodo@…:

I am having the same issue (I think). What exactly do you mean by "cleaning" main.log?

Replying to macsforever2000@…:

Please clean and attach the main.log.

Changed 14 months ago by vloodo@…

The error log file generated when trying to build webkit-gtk

comment:7 Changed 14 months ago by elelay@…

comment:8 Changed 14 months ago by vloodo@…

I can confirm that changing line 34 of GOwnPtr.h this way:

-typedef struct _GMutex GMutex;
+typedef union  _GMutex GMutex;

allows me to compile.

Changed 14 months ago by elelay@…

include glib.h directly

comment:9 Changed 14 months ago by elelay@…

or, to be compatible with older versions of glib2, #include <glib.h> and remove the custom definitions (see attached patch).

comment:10 Changed 14 months ago by macports@…

  • Cc macports@… added

Cc Me!

comment:11 Changed 14 months ago by elelay@…

  • Cc elelay@… added

Cc Me!

comment:12 Changed 14 months ago by jeremyhu@…

  • Cc jeremyhu@… added

Cc Me!

comment:13 Changed 14 months ago by jeremyhu@…

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.