RESOLVED FIXED 241125
Add NetBSD support for MachineContext 
https://bugs.webkit.org/show_bug.cgi?id=241125
Summary Add NetBSD support for MachineContext
Leonardo Taccari
Reported 2022-05-31 05:17:37 PDT
This adds support to NetBSD for MachineContext. (Locally present in pkgsrc - the NetBSD package system - for several WebKitGTK releases but unfortunately never upstreamed, sorry!)
Attachments
Add NetBSD support for MachineContext (5.64 KB, patch)
2022-05-31 05:19 PDT, Leonardo Taccari 		no flags
DetailsFormatted DiffDiff
Add NetBSD support for MachineContext (5.80 KB, patch)
2022-05-31 05:30 PDT, Leonardo Taccari 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Leonardo Taccari
Comment 1 2022-05-31 05:19:11 PDT
Created attachment 459883 [details] Add NetBSD support for MachineContext
Leonardo Taccari
Comment 2 2022-05-31 05:29:25 PDT
Comment on attachment 459883 [details] Add NetBSD support for MachineContext Whitespace accidentally present... reworking on diff!
Leonardo Taccari
Comment 3 2022-05-31 05:30:04 PDT
Created attachment 459884 [details] Add NetBSD support for MachineContext
Radar WebKit Bug Importer
Comment 4 2022-06-07 05:18:15 PDT
<rdar://problem/94518456>
Yusuke Suzuki
Comment 5 2022-06-07 06:01:18 PDT
If it gets ready for review, please set r? :)
Leonardo Taccari
Comment 6 2022-06-07 06:53:40 PDT
Hello Yusuke! Whoops, definitely ready for review since 2022-05-31, I had completely missed that, sorry! Thanks a lot for pointing that out!
Yusuke Suzuki
Comment 7 2022-06-07 06:55:07 PDT
Comment on attachment 459884 [details] Add NetBSD support for MachineContext r=me
Yusuke Suzuki
Comment 8 2022-06-07 06:55:16 PDT
Nice, thank you!
Leonardo Taccari
Comment 9 2022-06-08 06:22:38 PDT
JFTR, after this patch is applied and patch in https://bugs.webkit.org/show_bug.cgi?id=241386 is applied `jsc' properly runs without any further local patches.
EWS
Comment 10 2022-06-08 08:50:26 PDT
Committed r295381 (251389@main): <https://commits.webkit.org/251389@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 459884 [details].
Note You need to log in before you can comment on or make changes to this bug.