Attempt to make WinUSB install notes clearer - #377
Merged
Kuuuube merged 3 commits intoAug 19, 2026
Conversation
Kuuuube
force-pushed
the
attempt-to-make-winusb-clearer
branch
from
August 19, 2026 17:25
30ab10c to
c3c555c
Compare
gonX
approved these changes
Aug 19, 2026
Member
There was a problem hiding this comment.
LGTM.
Re erb dependency addition: it indeed seems necessary as I apparently have that package implicitly installed as a dependency for the Ruby default gems, so I never ran into this library not being available.
While it is in the standard gems list, I don't think we can expect this to be present everywhere? Maybe some Ruby expert can chime in on that part of the change actually being necessary, but regardless this change as it is is an improvement.
Ruby docs does say that default gems are not uninstallable but this doesn't necessarily translate to meaning that the default gems are always available.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I finally waded through the dependency jungle and succeeded in getting this thing running to test my changes...
bundlerdoes not come installed by default. Theerbgem was removed at some point or something like that so we need to add it explicitly. At least this is what I needed to get it working on Arch.(Usually not required)to the step nameIf all of this doesnt make people turn their brains on for a second, I think it's a lost cause.