You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains 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
export GOPATH=/home/iguazio/gocode/
+ cd /home/iguazio/gocode/src/github.com/v3io/http_blaster
+ go get
# github.com/sasile/termui
../../sasile/termui/render.go:107:15: undefined: stack.ParseDump
../../sasile/termui/render.go:118:22: line.SrcLine undefined (type stack.Call has no field or method SrcLine)
../../sasile/termui/render.go:121:27: line.Func.PkgName undefined (type stack.Func has no field or method PkgName)
../../sasile/termui/render.go:133:24: line.Func.PkgName undefined (type stack.Func has no field or method PkgName)
../../sasile/termui/render.go:133:48: line.SrcLine undefined (type stack.Call has no field or method SrcLine)
../../sasile/termui/render.go:134:21: cannot call non-function line.Func.Name (type string)
Build step 'Execute shell' marked build as failure
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@meirleopold
FYI, changed this for now because using github.com/sasile/termui blaster build fails: https://jenkins-prod.iguazeng.com/job/http_blaster/94/console
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are there major difference from https://github.com/gizak/termui?
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MichaelSkralivetsky I think we should move using the original repo instead, WDYT?
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@meirleopold
i made following change in my fork in order to fix build failure: MichaelSkralivetsky/termui@f72beaf
yeah there are differences Sasi made from some reason in his fork
sure, ideally yes, but it won't compile when using it as is due to the same differences in Sasi's fork
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you try it? because it looks like Sasi had one commit and it looks like you removed all his code in yours
b57424dThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@meirleopold
yes, the difference is that Sasi branch is long behind original repo
feel free to try yourself