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

Cannot compile taiga with Visual Studio 2015 #139

Closed
AsukaLangleyfag opened this issue Jul 23, 2015 · 7 comments
Closed

Cannot compile taiga with Visual Studio 2015 #139

AsukaLangleyfag opened this issue Jul 23, 2015 · 7 comments

Comments

@AsukaLangleyfag
Copy link

Error C3688: invalid literal suffix 'L'; literal operator or literal operator template 'operator ""L' not found in taiga\src\library\anime_db.cpp line 640

@AsukaLangleyfag
Copy link
Author

Could this be connected with erengy/anitomy#2 ?

@erengy
Copy link
Owner

erengy commented Jul 23, 2015

Don't think so. It might be possible to fix the issue by adding a space before each L". In any case, Database::ClearInvalidValues function is no longer relevant, so we can just remove that and its call in Database::HandleCompatibility as well.

@AsukaLangleyfag
Copy link
Author

Sorry, but I can't help you with removing Database::ClearInvalidValues, but adding a space before each "L worked fine. But another error:
Error C2397: conversion from 'int' to 'unsigned short' requires a narrowing conversion in \taiga\src\ui\dlg\dlg_anime_list.cpp at lines 1412, 1415, 1418, 1421, 1424, 1427, 1430, 1433

@erengy erengy closed this as completed in fddda28 Aug 6, 2015
@tophf
Copy link

tophf commented May 19, 2017

FYI, to make utf8proc dependency compile in VS2015 I had to add UTF8PROC_EXPORTS in C++ preprocessor definitions. This value is specified in CMake file of utf8proc but for some reason (not required with VS2017?) isn't present in Taiga.vcxproj. The error message was "definition of dllimport data not allowed". Turns out I had to manually set the new utf8proc git url as it wasn't updated on submodule update.

@erengy
Copy link
Owner

erengy commented May 19, 2017

That shouldn't be necessary since eac4d9c. From what I can see, you don't have this commit in your fork.

@tophf
Copy link

tophf commented May 19, 2017

I don't update that fork but locally I have the latest master.

@tophf
Copy link

tophf commented May 19, 2017

Huh, apparently re-fetching the submodule failed to use the new git url https://github.com/erengy/utf8proc.git, and was still using the old one. Thanks for pointing out that commit.

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

No branches or pull requests

3 participants