Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 172459 - sys-devel/prelink-20060712 cron script spits harmless warning daily if KDE not installed
Summary: sys-devel/prelink-20060712 cron script spits harmless warning daily if KDE no...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo Toolchain Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-27 20:14 UTC by Boney McCracker
Modified: 2007-03-28 06:19 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Boney McCracker 2007-03-27 20:14:39 UTC
On any system that does not have a directory matching /etc/env.d/*kde*, running the cron script as written generates the following error:

sed: can't read /etc/env.d/*kde*: No such file or directory

Thanks for the nice script.  But it has two statements that attempt to access this file (one in the case of "prelinking=no" and one in the case of "prelinking=yes").  I simply commented out these lines in the script, but not all users will know to do that.  Both of these statements should be conditional, only being executed if the system has kde.

Yes, it's easy to comment these two lines out, but this script is installed automatically with the prelink ebuild, so it ought to work.  

I think an easy solution that keeps it a no-brainer for users would be either:

a) changing those two statements to check if /etc/conf.d/*kde* exists first; or 

b) a single "kde=yes" flag in the conf.d file that triggers conditional execution of these two lines in the script.

Thanks.

Reproducible: Always

Steps to Reproduce:
1. Set /etc/conf.d/prelink variable "prelinking=yes"
2. Run /etc/cron.daily/prelink
3.

Actual Results:  
sed: can't read /etc/env.d/*kde*: No such file or directory


Expected Results:  
Above error should not be generated and prelinking should proceed normally.

Commenting the two offending lines of the script is an easy fix.  But having one person stick an "if" statement on them is a lot easier than having a thousand people comment these two lines.
Comment 1 SpanKY gentoo-dev 2007-03-28 06:19:31 UTC
fixed in cvs

workaround: install KDE ;)