310773 – net-misc/chrony-1.24 Ebuild "unintelligently" adds the -s argument when it shouldn't.

Bug 310773 - net-misc/chrony-1.24 Ebuild "unintelligently" adds the -s argument when it shouldn't.

Summary: net-misc/chrony-1.24 Ebuild "unintelligently" adds the -s argument when it sh...

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-03-22 18:28 UTC by Walther
Modified:	2010-03-23 09:16 UTC (History)
CC List:	1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Walther 2010-03-22 18:28:02 UTC

The ebuild auto adds the -s flag to the server startup if the "rtcfile" string is found in the chrony config.

But using grep -q 'rtcfile' "${CFGFILE}" && ARGS="${ARGS} -s"  is the wrong way to do this. What if the rtcfile line happens to be commented?

Try grep -q '^rtcfile' "${CFGFILE}" && ARGS="${ARGS} -s" next time, okay? The same applies to dumponexit.

Reproducible: Always

Comment 1 Torsten Veller (RETIRED) gentoo-dev

2010-03-23 09:12:41 UTC

The anchors were lost while fixing bug 279477.
<http://sources.gentoo.org/viewcvs.py/gentoo-x86/net-misc/chrony/files/chronyd.conf?r1=1.5&r2=1.6>

Fixed. Thanks

Comment 2 Torsten Veller (RETIRED) gentoo-dev

2010-03-23 09:15:15 UTC

Reassign it first

Comment 3 Torsten Veller (RETIRED) gentoo-dev

2010-03-23 09:15:59 UTC