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

Solaris Build Flags #1015

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

matthinrichsen-wf
Copy link

If building with js, with references to "golang.org/x/sys/unix" causes the following compile errors:

golang.org/x/sys/unix/fcntl_darwin.go:15:42: undeclared name: Flock_t
golang.org/x/sys/unix/ioctl.go:14:47: undeclared name: Winsize
golang.org/x/sys/unix/ioctl.go:25:47: undeclared name: Termios
golang.org/x/sys/unix/sockcmsg_unix.go:53:18: undeclared name: Cmsghdr
golang.org/x/sys/unix/sockcmsg_unix.go:59:9: undeclared name: Cmsghdr
golang.org/x/sys/unix/sockcmsg_unix.go:80:52: undeclared name: Cmsghdr
golang.org/x/sys/unix/syscall_bsd.go:128:63: undeclared name: Rusage
golang.org/x/sys/unix/syscall_unix.go:193:7: undeclared name: RawSockaddrInet4
golang.org/x/sys/unix/syscall_bsd.go:147:54: undeclared name: _Socklen
golang.org/x/sys/unix/syscall_unix.go:201:9: undeclared name: RawSockaddrInet6
golang.org/x/sys/unix/syscall_unix.go:201:9: too many errors

Previously opened the issue: #1014

@joshprzybyszewski-wf
Copy link

+1

@matthinrichsen-wf
Copy link
Author

@dgsb for code review

@freeformz
Copy link
Collaborator

I have little knowledge about GOOS=js. Can you provide some instructions on how to test this?

@joshprzybyszewski-wf
Copy link

@freeformz An easy way would be to import logrus into a gopherjs project. Perhaps using an example from their linked docs, then importing logrus and using it?

gopherjs: https://github.com/gopherjs/gopherjs
simple UI example: https://www.marwan.io/blog/simple-ui-gopherjs-vecty

basically you just need a main file that imports logrus, then do a gopherjs build . after doing all of the necessary install steps.

@stale stale bot added the stale label Feb 26, 2020
@markphelps markphelps removed the stale label Feb 26, 2020
Repository owner deleted a comment from stale bot Feb 26, 2020
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.

4 participants