Bug 382953 - Default differences between bootstrap and CMakeLists.txt
Summary: Default differences between bootstrap and CMakeLists.txt
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Portability-Cmake (show other bugs)
Version: 5.7.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-31 10:39 UTC by Simon
Modified: 2017-08-06 23:33 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.7.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Simon 2017-07-31 10:39:57 UTC
In the bootstrap scripts the default for e.g. opencv is 3 and mediaplayer is on, while in the CMakeLists.txt these default to 2 and off. Is there a reason for this or simply an oversight?
Comment 1 caulier.gilles 2017-07-31 15:59:11 UTC
bootstrap script are mostly dedicated in priority for the bundle.

As it can be customized i don't see any problem, excepted the fact to be homogenous as you said...

I vote to put the same in bootstrap as cmakelists.txt.

For next 6.0.0, we will certainly drop openCV 2 support.

For QtAV, as it can depend of ffmpeg codecs which can be problematic in special contry, it disabled by default.

Gilles
Comment 2 Simon 2017-07-31 16:16:07 UTC
Ok so these are all those that are different currently with the current settings and my suggestion to make them the same:

 | bootstrap | cmake | suggestion
opencv3 | on | off | on
mediaplayer | on | off | off
dbus | linux:on rest:off | on | on
mysqlsupport | on | off | on
internalmysql | on | off | on
Comment 3 Simon 2017-07-31 19:23:03 UTC
Git commit 79bf081c5ee77dd518afeb3dd0b7ceae660d1449 by Simon Frei.
Committed on 31/07/2017 at 19:21.
Pushed by sfrei into branch 'master'.

Synchronize build flags to bootstrap (enable opencv3 & mysql by default)
FIXED-IN: 5.7.0

M  +3    -3    CMakeLists.txt

https://commits.kde.org/digikam/79bf081c5ee77dd518afeb3dd0b7ceae660d1449
Comment 4 Ahmed Fathi 2017-08-06 23:33:12 UTC
Git commit 17ca2f5a323bf2fad45026bb212f8beea1541f14 by Ahmed Fathi Shaban, on behalf of Simon Frei.
Committed on 06/08/2017 at 23:14.
Pushed by ahmedfathy into branch 'gsoc17-dlna'.

Synchronize build flags to bootstrap (enable opencv3 & mysql by default)
FIXED-IN: 5.7.0

M  +3    -3    CMakeLists.txt

https://commits.kde.org/digikam/17ca2f5a323bf2fad45026bb212f8beea1541f14