-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
PHP code fails to compile: Unsupported tl-schema version -1
#2835
Comments
README actually claims that |
Same error if building with Ninja:
|
Also the same error when building from 586bc78 (master):
|
Do you have a big-endian CPU/OS? TDLib support only little-endian systems. |
@levlam Yes, this is on a Big-endian system. Is it possible to add support for BE systems? While I am running this on an old macOS, which the upstream may not be too motivated to support, Big-endian systems are supported in the modern versions of OpenBSD, FreeBSD and some distros of Linux. So it will benefit quite a number of people. |
This would be hard to achieve, because it requires many changes in different parts of the code. |
Could you say if it is known to be broken or just nobody tested it? (I mean, beyond initial error – I have no idea at the moment what causes it and therefore how serious it is). I am asking since FreeBSD, for example, does not seem to restrict it to LE archs: https://github.com/freebsd/freebsd-ports/blob/9c477091cee352b9e783062234ab276bbb61d16c/net-im/tdlib/Makefile (though of course it is possible that they never tried it as well). Oddly, btw, they do not declare a dependency on any P. S. Though on Linux the error has been reported earlier: #364 |
It is known to be broken. There are a few other places that are known to work only on little-endian systems, and unknown number of unidentified places that will work incorrectly on big-endian systems. Current;y, TDLib will not build on big-endian systems, which is better than failing in runtime. PHP is optional for TDLib building and the issue has nothing to do with PHP. |
@levlam Is there a ticket or discussion to track the issues on Big-endian? There may be people interested to help here, since this is OS-agnostic problem. |
You have already mentioned the issue 364. There is no complete list of features that must be ported to big-endian, but it would be a pretty long list. At least all usages of memcpy, as<..> and reinterpret_cast must be checked and some of them must be rewritten. |
@levlam You close this as completed, has it been fixed? |
It was closed as duplicate of the issue 364. |
@levlam Got it, thank you. An open duplicate is unneeded, of course. |
What could be the reason for this?
The text was updated successfully, but these errors were encountered: