Differential D9232

sysutils/vagrant: fix ruby_dep version
ClosedPublic
Actions

Authored by lifanov on Jan 18 2017, 5:15 PM.

Details

Reviewers

matthew

Commits

rP431819: fix sysutils/vagrant by relaxing ruby_dep requirement

Summary

``
fix sysutils/vagrant by relaxing ruby_dep requirement

PR: 215905
Submitted by: matthew
Reviewed by: matthew
Approved by: matthew (mentor), joe@thrallingpenguin.com (maintainer)
Differential Revision: https://reviews.freebsd.org/DXXXX

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lifanov updated this revision to Diff 24162.Jan 18 2017, 5:15 PM

lifanov retitled this revision from to sysutils/vagrant: fix ruby_dep version.

lifanov updated this object.

lifanov edited the test plan for this revision. (Show Details)

lifanov added a reviewer: matthew.

lifanov added a subscriber: koobs.

Herald added a subscriber: mat. · View Herald TranscriptJan 18 2017, 5:15 PM

I would like to commit this soon-ish. Vagrant is broken for me also :(

Yeah. At the moment we're waiting on the maintainer to approve the change. This was reported on 2017-01-09 and as it is two weeks for Maintainer timeout, the earliest we could commit would be 2017-01-24 unless the maintainer gets back to us sooner,

Apart from that, I think this update is fine to commit, but then I would, wouldn't I...

This revision is now accepted and ready to land.Jan 18 2017, 5:33 PM

... and it seems the maintainer has now approved, so please go ahead and commit.

Closed by commit rP431819: fix sysutils/vagrant by relaxing ruby_dep requirement (authored by lifanov). · Explain WhyJan 18 2017, 5:49 PM

This revision was automatically updated to reflect the committed changes.

Yep, thanks! I submitted this for review after the maintainer approved.