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

Missing skia library #130

Open
peardox opened this issue Jul 3, 2022 · 5 comments
Open

Missing skia library #130

peardox opened this issue Jul 3, 2022 · 5 comments

Comments

@peardox
Copy link

peardox commented Jul 3, 2022

Observed Behavoiur
If a compiled application is distributed without skia.dll (so/dylib? Can't check) the behaviour of the end application is to silently terminate without complaint - i.e. there is no indication that the application was started in the first place - hello world without the 'Hello World'.

Desired Behaviour
Warn used skia.dll could not be found

As it is rather easy to miss packaging a dll with an executable when testing (even scarier - could hit distribution) a missing skia.dll status should be notified to the user so the developer can kick himself (or herself)...

@peardox
Copy link
Author

peardox commented Jul 3, 2022

Added easy 'how to break skia' as part of a repo for illustration
https://github.com/peardox/Compositing

@viniciusfbb
Copy link
Member

viniciusfbb commented Jul 4, 2022

Hello @peardox!

Well, there is no warning because of a Delphi limitation. Our code throws an exception with the proper message but the exceptions in the initialization section are not displayed. Docwiki itself warns of this: https://docwiki.embarcadero.com/RADStudio/Sydney/en/Exceptions_(Delphi)#Raising_and_Handling_Exceptions

Note: Raising an exception in the initialization section of a unit may not produce the intended result.

The good news is that in our next version there will be no such problem as we will have static linking on Windows, that is, there will be no dll as a dependency. It is likely that by the end of the month the launch will take place

@valber-custodio
Copy link

Is there any solution for this case?

@Lotharingie88
Copy link

Lotharingie88 commented Apr 19, 2023 via email

@kazuser
Copy link

kazuser commented Feb 23, 2024

The good news is that in our next version there will be no such problem as we will have static linking on Windows, that is, there will be no dll as a dependency. It is likely that by the end of the month the launch will take place

So... ~1,5 years later... will it be at all, Vinícius? 🥲

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants