Bug 194681 - bashism in oppdater.sh script
Summary: bashism in oppdater.sh script
Status: RESOLVED FIXED
Alias: None
Product: i18n
Classification: Translations
Component: nn (show other bugs)
Version: unspecified
Platform: Debian testing Unspecified
: NOR normal
Target Milestone: ---
Assignee: Gaute Hvoslef Kvalnes
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-30 19:39 UTC by Raphael Geissert
Modified: 2009-08-26 21:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Fix bashisms (604 bytes, patch)
2009-05-30 19:39 UTC, Raphael Geissert 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Raphael Geissert 2009-05-30 19:39:05 UTC 
Version:           4.2.2 (using KDE 4.2.2)
Installed from:    Debian testing/unstable Packages

While hunting down bashisms in Debian packages I found the following in the oppdater.sh script:

possible bashism in ./usr/share/locale/nn/LC_SCRIPTS/kdelibs4/oppdater.sh line 37 (echo -e):
echo -e '\n\n'
possible bashism in ./usr/share/locale/nn/LC_SCRIPTS/kdelibs4/oppdater.sh line 44 (should be 'b = a'):
if [ "$tooold" == 0 ]

Attachinig patch in a moment. The Debian maintainers have also suggested to stop including those scripts in the install target.
Comment 1 Raphael Geissert 2009-05-30 19:39:52 UTC 
Created attachment 34129 [details]
Fix bashisms
Comment 2 Albert Astals Cid 2009-08-05 23:06:13 UTC 
Karl can you have a look please?
Comment 3 Karl Ove Hufthammer 2009-08-26 19:45:25 UTC 
The script was not actually used by KDE; it was only an internal script for use by the translators. I’ve now removed it from SVN. Please close this bug as ‘fixed’ (I don’t have permission to do so).
Comment 4 Albert Astals Cid 2009-08-26 21:26:32 UTC 
Fixed, i've also given you some more karma.