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

cmake-generated nmake file is missing jpeg.lib when linking plClient #17

Closed
philippelatulippe opened this issue Apr 20, 2011 · 3 comments
Labels

Comments

@philippelatulippe
Copy link
Contributor

I currently have the latest from this repository. After running cmake -G "NMake Makefiles" . in the root of the project, I end up with a broken build.make for plClient. It is missing the path to jpeg.lib in the section "Linking CXX executable plClient.exe". Adding the path to build.make manually works.

I have checked CMakeLists.txt, and it seems to be fine: target_link_libraries(plClient ${JPEG_LIBRARY})

In my CMakeCache.txt, JPEG_LIBRARY is properly defined.

The broken file in question: Plasma/Sources/Plasma/Apps/plClient/CMakeFiles/plClient.dir/build.make

The CmakeLists.txt which is used to generate it: Plasma/Sources/Plasma/Apps/plClient/CMakeLists.txt

I am using cmake 2.8.4 for windows.

@philippelatulippe
Copy link
Contributor Author

It happens when the build type is set to "Live" but not when it is set to "Dev".

@branan
Copy link
Member

branan commented Apr 26, 2011

This is (I think) a CMake bug. I've gotten it to link just by adjusting CMakeLists.txt so the link libraries are in a different order and re-generating. It affects both nmake and the visual studio GUI. If I can figure out a way to reliable get it to build, I'll adjust the CMakeLists.txt file.

@branan
Copy link
Member

branan commented May 26, 2011

I haven't seen this in a while. I don't know if there was an updated to MSVC that fixed it, or if I've just been lucky. I'm just going to close it for now. If this re-emerges I'll happily reopen it.

@branan branan closed this as completed May 26, 2011
zrax pushed a commit to zrax/Plasma that referenced this issue Apr 24, 2020
Fix essentials for localization support.

Closes H-uru#9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants