flathub / org.kde.okular

https://flathub.org/apps/details/org.kde.okular
4 stars 12 forks source link

Bring back exiv2 anitya #309

Closed tsdgeos closed 12 months ago

flathubbot commented 1 year ago

Started test build 61816

flathubbot commented 1 year ago

Build 61816 failed

Justinzobel commented 1 year ago
-- Could NOT find inih (missing: inih_LIBRARY inih_INCLUDE_DIR) 
CMake Error at cmake/Findinih.cmake:28 (message):
  inih library not found

Not found in exiv build.

travier commented 1 year ago

We can disable it like we did for another app I don't remember.

travier commented 1 year ago

https://github.com/flathub/org.gimp.GIMP/blob/6acd6e001967f0934dade222ddadc1f26585a0be/org.gimp.GIMP.json#L319

flathubbot commented 1 year ago

Started test build 63769

flathubbot commented 1 year ago

Started test build 63770

flathubbot commented 1 year ago

Build 63770 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/46407/org.kde.okular.flatpakref
flathubbot commented 1 year ago

Build 63769 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/46406/org.kde.okular.flatpakref
travier commented 1 year ago

Let's remove the inih dependency instead? It's not used for the library, only for a command line tool that we don't use.

tsdgeos commented 1 year ago

Let's remove the inih dependency instead? It's not used for the library, only for a command line tool that we don't use.

Grepping in the code shows it's used in readExiv2Config which is used in places like CanonMakerNote::printCsLensType, which then i have a bit of trouble tracking where it's used but doesn't seem obvious is for the command line tool only

Is there some docu or something that says inih is only for the command line tool?

travier commented 1 year ago

https://github.com/Exiv2/exiv2/pull/2465

travier commented 1 year ago

https://exiv2.org/manpage.html#config_file

travier commented 1 year ago

https://github.com/flathub/org.gimp.GIMP/commit/4335a0912ccfd53e173703926aabbdafbcf3345f

tsdgeos commented 12 months ago

I'm honestly not convinced :D

Does it hurt us to actually provide the dependencies the library wants?

Erick555 commented 12 months ago

It's yet another point of possible failure. Especially when relying on github-generated tarballs.

tsdgeos commented 12 months ago

I don't remember a single problem with github tarballs in all the hundred flathub apps we have, can you point to a place such a problem happened?

flathubbot commented 12 months ago

Started test build 64864

flathubbot commented 12 months ago

Build 64864 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/47501/org.kde.okular.flatpakref
Erick555 commented 12 months ago

I don't remember a single problem with github tarballs in all the hundred flathub apps we have, can you point to a place such a problem happened?

Perhaps you weren't paying attention to the worldwide news. Note the difference between github auto-generated tarballs and upstream generated tarballs on github. The problem is with the former.