Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

hosiet
Copy link
Contributor

@hosiet hosiet commented Sep 14, 2021

This Pull Request closes: #359 .

NOTE: This Pull Request comes with inevitable side effects! Merge with caution.

CI-built ksnip package now additionally conflicts with libkimageannotator-common.

CI-built deb package provides kimageannotator translation files, which are supposed to be provided by libkimageannotator-common in Debian official packages. The additional conflicts relationship prevents file conflicts from happening.

This means users will not longer able to install CI-built ksnip deb package and distribution-provided KDE spectacle (KDE's default screenshot tool) together, because kde-spectacle depends on distribution-provided libkimageannotator libraries and translations.

Let CI-built ksnip conflicts with libkimageannotator-common.

CI-built deb package provides kimageannotator translation files,
which are supposed to be provided by libkimageannotator-common in
Debian official packages. The additional conflicts relationship
prevents file conflicts from happening.

NOTE: this means you will not longer able to install CI-built ksnip
and distribution-provided KDE spectacle together, because
kde-spectacle depends on distribution-provided libkimageannotator
libraries and translations.
@DamirPorobic
Copy link
Member

Sounds problematic, basically you cannot install the ci deb package on KDE Plasma as spectacle is always installed there, or?

Do we have any workaround? Maybe installing the translation files in the same directory as ksnip translations?

@hosiet
Copy link
Contributor Author

hosiet commented Sep 17, 2021

As long as translation files are provided separately, file conflict is inevitable. Even if files are installed elsewhere, you need to make (the embedded) libkimageannotator smart enough to pick up translation elsewhere.

@DamirPorobic DamirPorobic merged commit 09598b1 into ksnip:master Sep 18, 2021
@DamirPorobic
Copy link
Member

Thanks for providing this PR :)

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.

kImageAnnotator not tanslated with deb package
2 participants