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)
	Mon, Sep 7, 7:10 PM

	Unknown Object (File)
	Fri, Sep 4, 8:37 PM

	Unknown Object (File)
	Fri, Sep 4, 8:34 PM

	Unknown Object (File)
	Fri, Sep 4, 2:17 PM

	Unknown Object (File)
	Thu, Sep 3, 8:59 AM

	Unknown Object (File)
	Wed, Sep 2, 2:30 PM

	Unknown Object (File)
	Mon, Aug 31, 4:01 PM

	Unknown Object (File)
	Mon, Aug 31, 4:01 PM

View All 63 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