[MacPorts] #37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly --------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Keywords: | Port: webkit-gtk --------------------------+------------------------ I have webkit-gtk @1.10.2+universal installed and am working on a [ticket:25261 portfile for gphpedit @0.9.98] and the build fails with: {{{ /opt/local/include/webkitgtk-1.0/webkit/WebKitDOMNavigator.h:81:50: error: expected ';' after top level declarator * webkit_dom_navigator_register_protocol_handler: ^ ; }}} Looking in /opt/local/include/webkitgtk-1.0/webkit/WebKitDOMNavigator.h I see that the muniversal portgroup has mangled this header; these are lines 68 thru 90 of that header: {{{ /** #ifndef __LP64__ * webkit_dom_navigator_webkit_get_gamepads: * @self: A #WebKitDOMNavigator * * Returns: (transfer none): * **/ WEBKIT_API WebKitDOMGamepadList* webkit_dom_navigator_webkit_get_gamepads(WebKitDOMNavigator* self); /** #endif /* ! __LP64__ */ * webkit_dom_navigator_register_protocol_handler: * @self: A #WebKitDOMNavigator * @scheme: A #gchar * @url: A #gchar * @title: A #gchar * @error: #GError * * Returns: * **/ }}} The "`/* ! __LP64__ */`" comment innocently inserted by the muniversal portgroup was unfortunately inserted inside an existing block comment, thus ending the block comment prematurely. I think this is the only header that got mangled; it's the only file installed by the port that contains "`__LP64__`". The solution should be to tell muniversal to not diff that file but to employ an alternate strategy, by setting "`merger_dont_diff`" to the list of files not to diff. I'll test and then attach a patch to do this. -- Ticket URL: <https://trac.macports.org/ticket/37774> MacPorts <http://www.macports.org/> Ports system for Mac OS
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Changes (by ryandesign@…): * keywords: => haspatch -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Changes (by ryandesign@…): * status: new => closed * resolution: => fixed Comment: r104837 -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by chief1983@…): While running upgrade outdated, I suddenly hit this immediately after the previous port in the list was cleaned: {{{ Error: webkit-gtk: Error executing quartz: invalid command name "quartz- include-widgetbackingstorecairo.patch" Error: Unable to open port: Error evaluating variants }}} -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by jeremyhu@…): Replying to [comment:5 chief1983@…]:
{{{ Error: webkit-gtk: Error executing quartz: invalid command name "quartz- include-widgetbackingstorecairo.patch" Error: Unable to open port: Error evaluating variants }}}
r104903 -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by chief1983@…): I am still seeing problems caused by the comment in that header file. While building GIMP2: {{{ :info:build In file included from dialog.c:34: :info:build In file included from /opt/local/include/webkitgtk-1.0/webkit/webkit.h:27: :info:build In file included from /opt/local/include/webkitgtk-1.0/webkit/webkitdom.h:135: :info:build /opt/local/include/webkitgtk-1.0/webkit/WebKitDOMNavigator.h:80:8: warning: '/*' within block comment [-Wcomment] :info:build #endif /* __LP64__ */ :info:build ^ :info:build /opt/local/include/webkitgtk-1.0/webkit/WebKitDOMNavigator.h:81:4: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] :info:build * webkit_dom_navigator_register_protocol_handler: :info:build ~ ^ :info:build /opt/local/include/webkitgtk-1.0/webkit/WebKitDOMNavigator.h:81:50: error: expected ';' after top level declarator :info:build * webkit_dom_navigator_register_protocol_handler: }}} And so on to more failure. I don't understand why that #ifdef was put in that header like that in the first place, it seems like it's shifted down one line from where it should be, so that it would be fully surrounding just one logical block of code. Is it not possible to fix it or have a patch that makes it end up like so? {{{ #ifndef __LP64__ /** * webkit_dom_navigator_webkit_get_gamepads: * @self: A #WebKitDOMNavigator * * Returns: (transfer none): * **/ WEBKIT_API WebKitDOMGamepadList* webkit_dom_navigator_webkit_get_gamepads(WebKitDOMNavigator* self); #endif /* ! __LP64__ */ /** * webkit_dom_navigator_register_protocol_handler: * @self: A #WebKitDOMNavigator * @scheme: A #gchar * @url: A #gchar * @title: A #gchar * @error: #GError * * Returns: * **/ }}} If so, there wouldn't be any issues with the comment blocks being terminated incorrectly. I really don't know much about macports though and how this would or wouldn't work. -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by jeremyhu@…): Replying to [comment:7 chief1983@…]:
I am still seeing problems caused by the comment in that header file.
Make sure you actually have the fix installed... {{{ $ cat /opt/local/include/webkitgtk-1.0/webkitdom/WebKitDOMNavigator.h #ifndef __LP64__ #include "i386-WebKitDOMNavigator.h" #else /* __LP64__ */ #include "x86_64-WebKitDOMNavigator.h" #endif /* __LP64__ */ ]}} -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by chief1983@…): Sorry, I ran port selfupdate and then port upgrade outdated, which only left libgnome outdated because it is having an unrelated issue I believe. Then I tried to reinstall Gimp2. I haven't actually seen webkit-gtk show up in the outdated list after running selfupdate, so am I missing a step, or do I need manually apply something until the port commit is pushed to everyone or something? -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by ryandesign@…): Replying to [comment:9 chief1983@…]:
I haven't actually seen webkit-gtk show up in the outdated list after running selfupdate,
Same; the reason is that the epoch was decreased from 2 to 1 in r104700. The epoch should be set back to 2. -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:10> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by chief1983@…): That's making sense now. Yeah I'm still on webkit- gtk@1.10.2_2+quartz+universal+video. It hasn't attempted to upgrade it to 2.0.0 yet. Copying the new portfile to a local port repo and making that change, and then running portindex seems to then want to upgrade it to 2.0.0. -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:11> MacPorts <http://www.macports.org/> Ports system for OS X
#37774: webkit-gtk @1.10.2+universal: WebKitDOMNavigator.h header merged incorrectly ---------------------------+------------------------ Reporter: ryandesign@… | Owner: jeremyhu@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch Port: webkit-gtk | ---------------------------+------------------------ Comment (by ryandesign@…): MacPorts will not present webkit-gtk 2.0.0 as an available update until the port's epoch is returned to its correct value. The epoch may never be decreased or removed from a port. -- Ticket URL: <https://trac.macports.org/ticket/37774#comment:12> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts