Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

master build error fix, require go v1.11.4 or higher #126

Merged
merged 5 commits into from Jan 5, 2019

Conversation

itswisdomagain
Copy link
Contributor

Building current source on master fails with the following transitive dependency fetch error:

go: rsc.io/qr@v0.2.0: unrecognized import path "rsc.io/qr" (https fetch: Get https://rsc.io/qr?go-get=1: dial tcp 216.239.36.21:443: i/o timeout)

The dependency (github.com/mdp/qrterminal) that introduced that failing sub-dependency is removed in this PR. It was introduced recently in #54 to solve the problem of "huge" qrcode display on terminal.

github.com/skip2/go-qrcode was reintroduced to handle small qrcode printing on terminal. The current output with this PR is perhaps not as small as was achieved in #54 but certainly smaller than before #54:

screenshot 2019-01-04 at 11 07 50 pm

Also, this PR branch code was built with go v1.11.4 and the updated dependency checksums in go.sum will fail validation in previous go versions. The readme has been updated to reflect that only go v1.11.4 and higher can now be used to build this repo's code successfully.

@barisere barisere merged commit b8bb928 into raedahgroup:master Jan 5, 2019
itswisdomagain pushed a commit that referenced this pull request Jan 9, 2019
master build error fix, require go v1.11.4 or higher

Former-commit-id: b8bb928
@itswisdomagain itswisdomagain deleted the qrcode-fix branch January 10, 2019 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants