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
Unknown Object (File)
Mon, Dec 9, 7:06 AM
Unknown Object (File)
Sat, Nov 30, 9:01 AM
Unknown Object (File)
Nov 4 2024, 10:35 AM
Unknown Object (File)
Oct 29 2024, 9:57 PM
Unknown Object (File)
Oct 21 2024, 11:59 AM
Unknown Object (File)
Oct 19 2024, 6:14 PM
Unknown Object (File)
Oct 16 2024, 1:46 PM
Unknown Object (File)
Oct 2 2024, 11:54 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...