Page MenuHomeFreeBSD
Differential D3208

www/varnish4 fix startup script order
ClosedPublic
Actions

Authored by feld on Jul 26 2015, 4:23 PM.
Tags
None
Referenced Files
F175263318: D3208.diff
Fri, Oct 9, 1:30 PM
Unknown Object (File)
Wed, Oct 7, 2:25 AM
Unknown Object (File)
Tue, Oct 6, 4:10 AM
Unknown Object (File)
Tue, Oct 6, 12:57 AM
Unknown Object (File)
Mon, Oct 5, 11:04 PM
Unknown Object (File)
Mon, Oct 5, 3:07 AM
Unknown Object (File)
Mon, Oct 5, 2:46 AM
Unknown Object (File)
Mon, Oct 5, 1:37 AM
View All Files
Subscribers
None

Details

Reviewers
mat
Commits
rP392997: Fix startup script order:
Summary

mat@ reported this in PR#201586

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

feld updated this revision to Diff 7345.Jul 26 2015, 4:23 PM
feld retitled this revision from to www/varnish4 fix startup script order.
feld updated this object.
feld edited the test plan for this revision. (Show Details)
feld added a reviewer: mat.
mat edited edge metadata.Jul 27 2015, 4:14 PM

Looks ok to me. Maybe the same hack needs to be added to varnishlog.in, no ?

feld updated this revision to Diff 7390.Jul 27 2015, 4:38 PM
feld edited edge metadata.

Remove inline brace to appease rclint
Add same precmd() to varnishlog which also seems to need _.vsm

Closed by commit rP392997: Fix startup script order: (authored by feld). · Explain WhyJul 27 2015, 5:53 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
www/
varnish4/
2 lines
files/
2 lines
30 lines

Diff 7345

View Options

www/varnish4/Makefile

Loading...
View Options

www/varnish4/files/varnishlog.in

Loading...
View Options

www/varnish4/files/varnishncsa.in

Loading...