Page MenuHomeFreeBSD
Differential D7572

Fix missing substitution of @SBINDIR@ in resolvconf script
ClosedPublic
Actions

Authored by badger on Aug 18 2016, 10:10 PM.
Tags
None
Referenced Files
F110666784: D7572.id19562.diff
Fri, Feb 21, 4:25 PM
F110664695: D7572.id19470.diff
Fri, Feb 21, 3:43 PM
Unknown Object (File)
Sun, Feb 16, 5:40 AM
Unknown Object (File)
Mon, Feb 10, 12:21 PM
Unknown Object (File)
Tue, Feb 4, 4:21 PM
Unknown Object (File)
Jan 15 2025, 11:00 PM
Unknown Object (File)
Jan 3 2025, 4:31 AM
Unknown Object (File)
Dec 22 2024, 6:00 PM
View All Files
Subscribers
imp

Details

Reviewers
pfg
gjb
Commits
rS304652: Fix missing substitution of @SBINDIR@ in resolvconf scripts
Summary

Fix missing substitution of @SBINDIR@ in resolvconf script

Certain features, such as resolv_conf_passthrough=NULL, do not work
correctly due to this missing substitution.

Also remove the @PREFIX@ substitution, which is no longer needed.

The @SBINDIR@ strings look to have appeared in r282431 with openresolv
3.7.0.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

badger updated this revision to Diff 19470.Aug 18 2016, 10:10 PM
badger retitled this revision from to Fix missing substitution of @SBINDIR@ in resolvconf script.
badger updated this object.
badger edited the test plan for this revision. (Show Details)
Herald added a subscriber: imp. · View Herald TranscriptAug 18 2016, 10:10 PM
badger updated this revision to Diff 19474.Aug 18 2016, 11:35 PM

Remove @PREFIX@.

badger updated this object.Aug 18 2016, 11:36 PM
badger updated this object.
badger added a reviewer: gjb.Aug 18 2016, 11:47 PM
badger added a reviewer: pfg.Aug 19 2016, 7:41 PM
pfg accepted this revision.Aug 21 2016, 8:17 PM
pfg edited edge metadata.

Looks good to me.

sbin/resolvconf/Makefile
32 ↗(On Diff #19474)

Is this a TAB, and if so, why?

This revision is now accepted and ready to land.Aug 21 2016, 8:17 PM
pfg added a comment.Aug 21 2016, 8:20 PM

Forgot to mention .. please MFC11. IOt's difficult to keep up with changes if things differ.

badger added inline comments.Aug 22 2016, 1:10 AM
sbin/resolvconf/Makefile
32 ↗(On Diff #19474)

Yeah; I left the SYSCONFDIR line's indent as tabs when I appended it to the sed line. All the '-e' lines line up nicely in an editor with 8 space tabs, though perhaps not in phabricator.

Closed by commit rS304652: Fix missing substitution of @SBINDIR@ in resolvconf scripts (authored by badger). · Explain WhyAug 23 2016, 2:06 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
resolvconf/
5 lines

Diff 19562

View Options

head/sbin/resolvconf/Makefile

Loading...