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, Jul 20, 12:15 PM

	Unknown Object (File)
	Sat, Jul 18, 11:55 PM

	Unknown Object (File)
	Sat, Jul 11, 7:37 PM

	Unknown Object (File)
	Sat, Jul 4, 7:09 AM

	Unknown Object (File)
	Wed, Jun 24, 5:40 PM

	Unknown Object (File)
	Jun 20 2026, 7:43 AM

	Unknown Object (File)
	Jun 8 2026, 10:26 PM

	Unknown Object (File)
	Jun 8 2026, 10:33 AM

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