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

Revert "Remove devel subpackage" #11

Closed

Conversation

Michal-Szczepaniak
Copy link

This reverts commit dd9305a. It's not useless if you want to make voicecall plugin. It works if voicecall-manager is started without invoker but if you don't link against voicecall and start voicecall-manager with invoker it doesn't load the plugin.

supersedes #10

This reverts commit dd9305a. It's not
useless if you want to make voicecall plugin. It works if
voicecall-manager is started without invoker but if you don't link
against voicecall and start voicecall-manager with invoker it doesn't
load the plugin.
@Michal-Szczepaniak
Copy link
Author

mal said to mention that it can't find symbol _ZTI24AbstractVoiceCallHandler

@k1gen
Copy link

k1gen commented Apr 22, 2023

it works! please merge this

@pvuorela
Copy link
Contributor

This reverts commit dd9305a. It's not useless if you want to make voicecall plugin

The devel package doesn't include any headers or such. This project doesn't support creating external voicecall plugins. Doing such now would require copying the headers to the plugin project.

Also if there are such external hacked plugins, they should keep on working if they were built with the library available and linked against it. If built without it, the build should fail, right?

Would be different thing if there were some headers installed.

A bit curious on what this external plugin would be.

@Michal-Szczepaniak
Copy link
Author

But creating external plugins work fine just fine since 4.3 or even ealier. The .so file is there, the lack of -devel package just prevents you from linking against it so you have to hack in symlink manually

@pvuorela
Copy link
Contributor

I wouldn't call it working just fine if the setup involves copying header files from the project. That's a hack.

@Michal-Szczepaniak
Copy link
Author

And whose fault is that :)

@Michal-Szczepaniak
Copy link
Author

Then the solution is not only restoring devel package but adding headers

@Michal-Szczepaniak
Copy link
Author

If you want look at my plugin here you go, i actually just added voicecall as submodule instead of copying headers from nowhere

https://github.com/Michal-Szczepaniak/Yottagram/tree/master/YottagramVoicecallPlugin

@pvuorela
Copy link
Contributor

Then the solution is not only restoring devel package but adding headers

Yes. This would make some sense. No guarantees on APIs, of course, but at least it would allow to play with the thing better than with header copies (which git submodule effectively is).

@Michal-Szczepaniak
Copy link
Author

Then please do that after merging this :)

@k1gen
Copy link

k1gen commented Apr 24, 2023

jolla, please merge, this fixes issue you created with 4.5 release for casual users who just want to be able to call in yottagram with sailfish dialer

@rubdos
Copy link

rubdos commented Apr 24, 2023

This also sounds like something we may want to support from within Whisperfish, when I get to implementing voice calls. Is there a place where we can draw inspiration for adding a headers package in here?

Of course, official API support is not something I will be asking for any time soon!

@Michal-Szczepaniak
Copy link
Author

I mean adding headers to devel package is simplest thing ever its just matter if jolla wants it or not

@pvuorela
Copy link
Contributor

Took a while, but there #12

@Michal-Szczepaniak
Copy link
Author

@pvuorela you have my thanks and love <3

@pvuorela
Copy link
Contributor

Merged #12. Closing this.

@pvuorela pvuorela closed this May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants