Page MenuHomeFreeBSD

sysutils/vagrant: update to 1.7.4
ClosedPublic

Authored by mmoll on Jul 27 2015, 3:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 12:44 AM
Unknown Object (File)
Sat, Nov 23, 4:42 PM
Unknown Object (File)
Sat, Nov 23, 3:30 AM
Unknown Object (File)
Fri, Nov 22, 12:23 PM
Unknown Object (File)
Thu, Nov 21, 9:10 AM
Unknown Object (File)
Tue, Nov 19, 10:02 AM
Unknown Object (File)
Tue, Nov 19, 4:07 AM
Unknown Object (File)
Mon, Nov 18, 1:29 PM
Subscribers
None

Details

Summary

As the maintainer didn't react to my mail about D3083, I'd commit this right away, once I got clearance here.

sysutils/vagrant: update to 1.7.4

Approved by: swills (mentor), mat (mentor), Joe Benden (maintainer, timeout)
Test Plan

poudriere: http://space.kvedulv.de/FreeBSD/vagrant-1.7.4.log
portlint:

mmoll@marduk:/svn/ports/sysutils/vagrant$ portlint -ACNc
WARN: /svn/ports/sysutils/vagrant/pkg-plist: [2]: enumerating info files in the plist is deprecated in favor of adding info files into the Makefile using the INFO macro.
WARN: Makefile: [0]: possible direct use of command "install" found. use ${INSTALL_foobaa} instead.
0 fatal errors and 2 warnings found.

Please note that the .info file is not an info page and the occurrence of "install" is not an install command. :)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mmoll retitled this revision from to sysutils/vagrant: update to 1.7.4.
mmoll updated this object.
mmoll edited the test plan for this revision. (Show Details)
mmoll added reviewers: swills, mat.
mat edited edge metadata.

Except the patch regeneration that is not needed, this is good, you can commit it without updating the review.

As a side not, I'm going to release you from mentorship, so you won't even need to ask us any more :-)

sysutils/vagrant/files/patch-bin_vagrant
1โ€“2 โ†—(On Diff #7387)

I know portlint is painly saying that you MUST regenerate patches with make makepatch but it seems that this patch does not change, so, I feel the renaming and the change is not needed at all :-)

This revision is now accepted and ready to land.Jul 27 2015, 4:34 PM
This revision was automatically updated to reflect the committed changes.