Bug 209906

Summary: Unportable test(1) construct in cleanup_digikamdb
Product: [Applications] digikam Reporter: Thomas Klausner <tk>
Component: Portability-RuntimeAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 1.0.0   
Target Milestone: ---   
Platform: NetBSD pkgsrc   
OS: NetBSD   
Latest Commit: Version Fixed In: 1.0.0
Attachments: Patch fixing the problem.

Description Thomas Klausner 2009-10-08 20:55:05 UTC
Version:           1.0.0-beta5 (using KDE 4.3.1)
Compiler:          n/a n/a
OS:                NetBSD
Installed from:    NetBSD pkgsrc

The script utilities/scripts/cleanup_digikamdb uses the unportable bash "==" comparison operator for test(1). It is not supported by most other shells or even test(1) from GNU coreutils. Please use "=" instead.
Comment 1 Thomas Klausner 2009-10-08 20:55:40 UTC
Created attachment 37454 [details]
Patch fixing the problem.
Comment 2 Thomas Klausner 2009-10-08 20:58:22 UTC
Note: I know that it has "#!/bin/bash" at the beginning, but the script works fine otherwise and could be changed to just use /bin/sh instead.
Comment 3 Andi Clemens 2009-10-08 23:33:29 UTC
SVN commit 1032924 by aclemens:

Use more portable comparison operator. Thanks to Thomas Klausner.

BUG:209906

 M  +3 -3      cleanup_digikamdb  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1032924