WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
253812
[Win] Remove DLLLauncherMain.cpp
https://bugs.webkit.org/show_bug.cgi?id=253812
Summary
[Win] Remove DLLLauncherMain.cpp
Fujii Hironori
Reported
2023-03-13 01:06:26 PDT
[Win] Remove DLLLauncherMain.cpp AppleWin has gone. We no longer need DLLLauncherMain.cpp that make CMake complicated. However, we have to set the path to WebKitLibraries/win/bin64, or copy all DLLs.
> path %WEBKIT_LIBRARIES%\bin64;%path%
I prefer the former. WinCairo tester buildbots also have to set the path.
Attachments
Add attachment
proposed patch, testcase, etc.
Fujii Hironori
Comment 1
2023-03-13 01:10:41 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/11435
Stephan Szabo
Comment 2
2023-03-13 10:06:19 PDT
It's unclear to me by what you mean by setting the path? Do you mean that the user of the binary needs to set the path? If so, I'm not convinced that's a good tradeoff, especially not without a lot more discussion involved.
Radar WebKit Bug Importer
Comment 3
2023-03-20 01:07:15 PDT
<
rdar://problem/106932734
>
EWS
Comment 4
2023-07-06 13:00:20 PDT
Committed
265819@main
(21785f58f0ef): <
https://commits.webkit.org/265819@main
> Reviewed commits have been landed. Closing PR #11435 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug