Opened 4 years ago

Closed 2 years ago

#62842 closed defect (fixed)

webkit2-gtk @2.28.2_2 +debug+minibrowser+quartz: error: use of undeclared identifier 'remoteProcessID'

Reported by: cooljeanius (Eric Gallager) Owned by: cooljeanius (Eric Gallager)
Priority: Normal Milestone:
Component: ports Version: 2.6.4
Keywords: haspatch Cc: dbevans (David B. Evans)
Port: webkit2-gtk

Description

I fixed this before but forgot to save my patch, so I've regenerated it and am making sure to submit it this time. The error I'm fixing is this:

/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_www_webkit2-gtk/webkit2-gtk/work/webkitgtk-2.28.2/Source/WebKit/Platform/IPC/Connection.cpp:672:223: error: use of undeclared identifier 'remoteProcessID'
    RELEASE_LOG_ERROR(IPC, "Connection::waitForSyncReply: Timed-out while waiting for reply for %{public}s::%{public}s from process %d, id = %" PRIu64, messageReceiverName.toString().data(), messageName.toString().data(), remoteProcessID(), syncRequestID);
                                                                                                                                                                                                                              ^
1 error generated.

Will attach patch next.

Attachments (1)

patch-Source_WebKit_Platform_IPC_Connection.cpp.diff (1.4 KB) - added by cooljeanius (Eric Gallager) 4 years ago.
patch I had to apply to webkit2-gtk sources to get it to build for me

Download all attachments as: .zip

Change History (12)

Changed 4 years ago by cooljeanius (Eric Gallager)

patch I had to apply to webkit2-gtk sources to get it to build for me

comment:1 Changed 4 years ago by kencu (Ken)

I haven't seen this issue so far. Looks like an undeclared function error? What system are you on?

FWIW, it looks like this is just a logging thing, but getpid returns the local id, and remoteProcessID was supposed to give the calling function's ID...

comment:3 in reply to:  1 Changed 4 years ago by cooljeanius (Eric Gallager)

Replying to kencu:

I haven't seen this issue so far. Looks like an undeclared function error? What system are you on?

I'm on Big Sur.

FWIW, it looks like this is just a logging thing, but getpid returns the local id, and remoteProcessID was supposed to give the calling function's ID...

I think it's ok to be wrong there if that's what it takes to get it to build; I'm not sure how to trigger that codepath anyways...

comment:4 Changed 3 years ago by kencu (Ken)

Cc: kencu removed

comment:5 Changed 3 years ago by albcn

Same issue in here with macOS Monterey 12.3.1 on Apple M1 and 2.7.2 on arm64 builds. The proposed patch proposed by Eric works nicely (thanks).

comment:6 Changed 2 years ago by cooljeanius (Eric Gallager)

Still occurs with webkit2-gtk @2.28.2_3

comment:7 Changed 2 years ago by kencu (Ken)

this must only show up in the quartz build, on an M1 Mac.

here's where that function lives:

https://github.com/WebKit/WebKit/search?q=remoteProcessID

Ideally someone (with a great deal of spare time :>) would upgrade this port to something a few years newer and see if that fixes it. For now, doing what Eric suggested gets it to build.

Or perhaps just delete that line of code for an M1 quartz build.

comment:8 in reply to:  7 Changed 2 years ago by cooljeanius (Eric Gallager)

Replying to kencu:

this must only show up in the quartz build, on an M1 Mac.

I'm on x86_64 though; my MBP is from 2019

comment:9 Changed 2 years ago by kencu (Ken)

oh, ok… quartz build on newer systems then, it seems.

comment:10 Changed 2 years ago by cooljeanius (Eric Gallager)

comment:11 Changed 2 years ago by cooljeanius (Eric Gallager)

Owner: set to cooljeanius
Resolution: fixed
Status: newclosed

In 3549bacfa635797227cb3c1161429d7d829a5115/macports-ports (master):

webkit2-gtk: fix quartz build on Big Sur

possibly closes: #62842 (untested)

Note: See TracTickets for help on using tickets.