Bug 215833 - [PATCH] sysutils/rubygem-winrm-fs: update to 0.3.2
Summary: [PATCH] sysutils/rubygem-winrm-fs: update to 0.3.2
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ruby (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-01-06 18:39 UTC by joe
Modified: 2017-01-06 21:43 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (ruby)


Attachments
rubygem-winrm-fs 0.3.2 (818 bytes, patch)
2017-01-06 18:39 UTC, joe
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description joe 2017-01-06 18:39:26 UTC
Created attachment 178578 [details]
rubygem-winrm-fs 0.3.2

Upgrade gem due to recent update of logging rubygem. Fixes recent problem with sysutils/vagrant not functioning.
Comment 1 Steve Wills freebsd_committer freebsd_triage 2017-01-06 18:58:16 UTC
The latest version of the winrm-fs gem is 1.0.1. Are you sure you don't want to update to that version? If vagrant won't work with that one, we can always create a port for the 0.3.x version of winrm-fs. And we can test if vagrant works with the winrm-fs 1.0.1 by patching it, and add that to the vagrant port if so.
Comment 2 joe 2017-01-06 19:00:33 UTC
It may not be worth the effort of creating different ports for this. To the best of my knowledge, Vagrant is the only port that uses this rubygem.

As for the version, an upgrade beyond this minor revision would introduce a number of other port upgrades that may introduce instability in Vagrant.
Comment 3 Steve Wills freebsd_committer freebsd_triage 2017-01-06 19:19:13 UTC
(In reply to joe from comment #2)
True, we'd have to update rubygem-winrm, but I notice the latest version of that (2.1.1) supports a newer rubygem-ntlm and drops the requirement for rubygem-uuidtools, while adding a requirement for rubygem-erubis, but we already have a port for that so that's not an issue. So if we patch vagrant to work with the newer winrm-fs and winrm and it works, that would seem like a good idea.
Comment 4 Steve Wills freebsd_committer freebsd_triage 2017-01-06 19:36:53 UTC
After looking a bit more, I see these issues:

https://github.com/mitchellh/vagrant/issues/7526
https://github.com/mitchellh/vagrant/pull/8102

particularly:

Due to the above, winrm-fs is orders of magnitude faster and more reliable

But it looks like until the next version is out, we have to wait. So I'll go ahead and commit this for now.
Comment 5 Steve Wills freebsd_committer freebsd_triage 2017-01-06 19:42:21 UTC
The files/patch-gemspec for sysutils/rubygem-winrm-fs can be removed here, correct?
Comment 6 joe 2017-01-06 19:52:08 UTC
Sorry, the "files/patch-gemspec" should be removed. (Turns out that it is not present in my own repository.)
Comment 7 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-01-06 21:43:10 UTC
Updated to 0.3.2 in r430749. Thanks!