Overview

Request 759908 superseded

- Update to version 3.0.0
* https://github.com/darktable-org/darktable/releases/tag/release-3.0.0
- Removed 0001-Fix-build-with-exiv2-0.27.patch
- Removed basecurve.readme.patch

Loading...

Stefan Brüns's avatar

You are submitting this to graphics and thus Factory, so you should be able to answer questions regarding packaging. If you don't know why openmp and opencl are omitted, please go ahead and either ask darix or determine it yourself, and add a comment to the spec file then.

Runtime requires should never be added as build time requires, this causes unnecessary rebuilds and extends dependency chains. Also, the Factory auto-bots do install checks on submission time, so removal of a dependency won't go unnoticed. Adding a build dependency will stop a package from building, but older builds will remain in the package repositories, so this does not work as a safeguard. If you want to make sure a program works, add an openQA test.

Anyway, you haven't told why the roboto fonts are required at all. There are no configure checks for it. There is no mention in the upstream changelog. So why is it needed? Iff it really needed, add a comment in the spec file, preferably with an upstream bug report reference.


Andreas Schneider's avatar

I'm not an cpu architecture guru, but here are my guesses:

  • The OpenMP code is SIMD optimized for x86_64 and not for ppc or arm
  • The OpenCL Kernels don't compile on ppc or arm

but see yourself

https://build.opensuse.org/package/show/home:gladiac:branches:graphics/darktable

The roboto font is required by the elegant css themes, see the RELEASE_NOTES

Request History
Andreas Schneider's avatar

gladiac created request

- Update to version 3.0.0
* https://github.com/darktable-org/darktable/releases/tag/release-3.0.0
- Removed 0001-Fix-build-with-exiv2-0.27.patch
- Removed basecurve.readme.patch


openSUSE Build Service is sponsored by