Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix naming conflict on MSVC Debug mode #919

Closed
wants to merge 2 commits into from

Conversation

SourDumplings
Copy link

There is naming conflict between qjs exe and lib. When building on MSVC Debug mode, qjs.exe will link qjs.lib and generate qjs.pdb, which is conflict with qjs lib pdb file and Error like:
D:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.42.34433\bin\HostX64\x64\link.exe /ERRORREPORT:Q UEUE /OUT:"E:\codes\SCGF\build\external\quickjs\Debug\qjs.exe" /INCREMENTAL /ILK:"qjs_exe.dir\Debug\qjs.ilk" /NOLOGO Debu g\qjs.lib kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi3 2.lib /MANIFEST /MANIFESTUAC:"level='asInvoker' uiAccess='false'" /manifest:embed /DEBUG /SUBSYSTEM:CONSOLE /TLBID:1 /DYN AMICBASE /NXCOMPAT /IMPLIB:"E:/codes/SCGF/build/external/quickjs/Debug/qjs.lib" /MACHINE:X64 /machine:x64 qjs_exe.dir\De bug\repl.obj qjs_exe.dir\Debug\standalone.obj qjs_exe.dir\Debug\qjs.obj "qjs_exe.dir\Debug\quickjs-libc.obj" 8>LINK : fatal error LNK1201: error writing to program database 'E:\codes\SCGF\build\external\quickjs\Debug\qjs.pdb'; check f or insufficient disk space, invalid path, or insufficient privilege [E:\codes\SCGF\build\external\quickjs\qjs_exe.vcxproj]

@saghul
Copy link
Contributor

saghul commented Feb 16, 2025

Is there no otehr way? I'd rather not change how users invoke the executable.

@SourDumplings
Copy link
Author

Is there no otehr way? I'd rather not change how users invoke the executable.

Or renaming lib target as qjs_lib?

@saghul
Copy link
Contributor

saghul commented Feb 16, 2025

Maybe. The question is: why doesn't the CI fail?

@chqrlie
Copy link
Collaborator

chqrlie commented Feb 16, 2025

How about naming the library libqjs.dll on Windows targets?

@SourDumplings SourDumplings closed this by deleting the head repository Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants