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

Support building shared library of ucd #1

Merged
merged 8 commits into from
Nov 27, 2023

Conversation

csukuangfj
Copy link

Also, support installing ucd when invoking make install.

@@ -44,7 +44,7 @@
#define iswspace ucd_isspace
#define iswupper ucd_isupper
#define iswxdigit ucd_isxdigit
#define tolower ucd_tolower
#define toupper udc_toupper
// #define tolower ucd_tolower
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I don't uncomment them, it causes errors in my code since I am using std::tolower and std::toupper in my code.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This symlink causes errors on Windows when invoking make install since windows does not support symlinks.

@@ -20,7 +20,7 @@
#define ENDIAN_H_COMPAT_SHIM
#pragma once

#if __has_include_next(<endian.h>)
#if __has_include_next(<endian.h>) && !defined(__APPLE__)
Copy link
Author

@csukuangfj csukuangfj Nov 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without !defined(__APPLE__) it causes errors for iOS.

@synesthesiam synesthesiam merged commit 8593723 into rhasspy:master Nov 27, 2023
@synesthesiam
Copy link

Thanks!

@mudler
Copy link

mudler commented Nov 28, 2023

Hi 👋 this actually broke building rhaspy - see: mudler/LocalAI#1356 and rhasspy/piper#286

@GiviMAD
Copy link

GiviMAD commented Dec 3, 2023

@csukuangfj @mudler after these PR the piper windows build with cmake is broken, reverting the PR I can build again.
rhasspy/piper#293
BR

@csukuangfj
Copy link
Author

@csukuangfj @mudler after these PR the piper windows build with cmake is broken, reverting the PR I can build again.
rhasspy/piper#293
BR

could you try rhasspy/piper-phonemize#21

@GiviMAD
Copy link

GiviMAD commented Dec 4, 2023

Same error:

LINK : fatal error LNK1181: cannot open input file '..\ucd-tools\Release\ucd.lib'

@csukuangfj
Copy link
Author

Same error:

LINK : fatal error LNK1181: cannot open input file '..\ucd-tools\Release\ucd.lib'

Could you tell us what changes you have made to use that PR from Piper-phonemize?

@GiviMAD
Copy link

GiviMAD commented Dec 5, 2023

Sorry,
I used these url https://github.com/csukuangfj/piper-phonemize/archive/refs/heads/fix-cmake.zip in the Addexternalproject declaration. I can give it another try tomorrow, but I think I did it correctly.

@synesthesiam
Copy link

synesthesiam commented Dec 5, 2023

I've set commit 0f65aa301e0d6bae5e172cc74197d32a6182200f of this repo (espeak-ng) to be used by piper-phonemize until I have more time to look into the problem.

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.

4 participants