majn / telegram-purple

Adds support for Telegram to Pidgin, Adium, Finch and other Libpurple based messengers.
GNU General Public License v2.0
735 stars 81 forks source link

Support libpurple 3 (unreleased) #366

Open je-vv opened 7 years ago

je-vv commented 7 years ago

When building against libpurple/pidgin 3, 1st I needed to patch "configure" so that pkg-config asked for "purple-3" rather than purple, but that's the working part, :-).

Then, if building the master git version, compiling fails at:

+++ telegram-base.c: In function ‘tgp_error_if_false’: telegram-base.c:240:5: warning: implicit declaration of function ‘purple_connection_error_reason’ [-Wimplicit-function-declaration] purple_connection_error_reason (conn->gc, PURPLE_CONNECTION_ERROR_AUTHENTICATION_FAILED, msg); ^~~~~~~~~~ telegram-base.c:241:5: error: too few arguments to function ‘purple_notify_message’ purple_notify_message (_telegram_protocol, PURPLE_NOTIFY_MSG_ERROR, cause, msg, NULL, NULL, NULL); +++

And if building on the development (1.4.0) branch, compiling fails at:

+++ In file included from telegram-purple.h:54:0, from tgp-net.c:43: tgp-chat.h:49:65: error: unknown type name ‘PurpleTypingState’ unsigned int tgprpl_send_chat_typing (PurpleConversation *conv, PurpleTypingState typing, gpointer ignored); ^~~~~ tgp-net.c: In function ‘net_on_connected_assert_success’: tgp-net.c:281:5: warning: implicit declaration of function ‘purple_connection_error_reason’ [-Wimplicit-function-declaration] purple_connection_error_reason (tls_get_conn (TLS), PURPLE_CONNECTION_ERROR_NETWORKERROR, ("Cannot connect to main server")); +++

It seems like telegram-purple still doesn't support libpurple/pidgin 3. Understood it's not yet released, but some (like me) need version 3. If there's any branch able to support libpurple/pidgin 3, please let me know...

Thanks,

je-vv commented 7 years ago

Realized not that urgent... Sorry...

BenWiederhake commented 7 years ago

True, not urgent. But we will want purple-3 support eventually.

EndlessEden commented 6 years ago

What would it take to make this plugin libpurple-3 ready?

BenWiederhake commented 6 years ago

Good question. Presumably updating the structures that interface tgp and libpurple.

If you do any modifications, please keep it intact for libpurple-2. You're welcome to extend the tests to ensure this.