Overview

Request 759888 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


Andreas Schneider's avatar

Yes, sdl seems to be not needed at all. Why openmp and opencl is disabled you have to ask darix. This is a merge from graphics:darktable:master spec file which builds current master.

You normally add requries also to the buildrequires to be notified about changes. For example if a package gets deprecated. Especially if you build for differnt distros you want to be sure that what you require really exists. The OBS doesn't test install-ability of packages!

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