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

undefined variables when compiling #16

Closed
huguesalary opened this issue Mar 10, 2017 · 5 comments
Closed

undefined variables when compiling #16

huguesalary opened this issue Mar 10, 2017 · 5 comments

Comments

@huguesalary
Copy link
Contributor

On current master branch @ddce54f991517838e1b42a27ebcfdce1c04afc72

go build returns:

# github.com/bcicen/ctop/cwidgets/expanded
ctop/cwidgets/expanded/info.go:19: p.Seperator undefined (type *termui.Table has no field or method Seperator)
ctop/cwidgets/expanded/mem.go:73: mbar.NumFmt undefined (type *termui.MBarChart has no field or method NumFmt)

First error is a simple spelling mistake p.Seperator should be p.Separator, pull request submitted (#15)

For the 2nd error, NumFmt doesn't exist. Not sure how to fix.

@bcicen
Copy link
Owner

bcicen commented Mar 10, 2017

Thanks @huguesalary. For NumFmt, this is part of a pending PR to termui: gizak/termui#118

@mbilker
Copy link

mbilker commented Mar 10, 2017

The fork is 17 commits behind and it does not include the spelling fix for Seperator. Could you rebase your fork so cloning the fork allows for ctop to compile without changing Separator back to Seperator?

@bcicen
Copy link
Owner

bcicen commented Mar 10, 2017

Sure, fork is current now.

@bcicen
Copy link
Owner

bcicen commented Mar 13, 2017

Vendored deps + build steps have been merged into current master; closing this for now.

@bcicen bcicen closed this as completed Mar 13, 2017
@vielmetti
Copy link
Contributor

vielmetti commented Apr 7, 2017

I'm following the build instructions and porting to arm64 (#68). I get this same error as reported above.

# github.com/bcicen/ctop/cwidgets/expanded
src/github.com/bcicen/ctop/cwidgets/expanded/mem.go:73: mbar.NumFmt undefined (type *termui.MBarChart has no field or method NumFmt)
go version go1.8 linux/arm64

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

No branches or pull requests

4 participants