rc.d/nscd: Avoid using setvar
Needs ReviewPublic
Actions

Authored by temcbun_gmail.com on Feb 3 2026, 8:38 PM.

Tags

Referenced Files

	Unknown Object (File)
	Thu, Aug 13, 12:52 PM

	Unknown Object (File)
	Wed, Aug 12, 3:28 PM

	Unknown Object (File)
	Tue, Aug 11, 6:07 PM

	Unknown Object (File)
	Sun, Aug 9, 3:21 PM

	Unknown Object (File)
	Sun, Aug 9, 1:42 AM

	Unknown Object (File)
	Sat, Aug 8, 12:46 PM

	Unknown Object (File)
	Sat, Aug 8, 3:45 AM

	Unknown Object (File)
	Wed, Aug 5, 3:30 PM

View All 49 Files

Subscribers

emaste

imp

Details

Reviewers

0mp
markj
cy

Group Reviewers

Contributor Reviews (src)

Summary

setvar is a non-standard feature of sh(1) and thus is not portable.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

temcbun_gmail.com created this revision.Feb 3 2026, 8:38 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 3 2026, 8:38 PM

temcbun_gmail.com requested review of this revision.Feb 3 2026, 8:38 PM

temcbun_gmail.com added a reviewer: cy.

I don't think there's any expectation that the rc scripts will be portable, but when there's no material difference (e.g. in performance, clarity, maintainability, etc.) I agree we should indeed prefer the portable way.

In D55091#1259161, @emaste wrote:

I don't think there's any expectation that the rc scripts will be portable, but when there's no material difference (e.g. in performance, clarity, maintainability, etc.) I agree we should indeed prefer the portable way.

I also think so. Besides, the rest of the rc scripts are using this construction instead of setvar. It lingers around only in two places: here and in rc.subr (https://reviews.freebsd.org/D55086).

Ping.

Revision Contents
Changeset List

Path

Size

libexec/

rc/

rc.d/

nscd

4 lines

Diff 171108

View Options

rc.d/nscd: Avoid using setvarNeeds ReviewPublicActions