Skip to content

compiler/natives/syscall: Don't import bytes package. #453

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

Merged
merged 1 commit into from
May 8, 2016

Conversation

dmitshur
Copy link
Member

@dmitshur dmitshur commented May 8, 2016

The real syscall package doesn't import bytes. This keeps the order in which stdlib packages must be built in for GopherJS consistent with the real standard library.

Source: https://godoc.org/syscall?import-graph

Helps #388.

@dmitshur dmitshur force-pushed the fix-syscall-imports branch from 8a047cf to 402d303 Compare May 8, 2016 21:42
The real syscall package doesn't import bytes. This keeps the order in
which stdlib packages must be built in for GopherJS consistent with the
real standard library.

Helps #388.
@neelance
Copy link
Member

neelance commented May 8, 2016

LGTM

@dmitshur dmitshur merged commit ce68d34 into master May 8, 2016
@dmitshur dmitshur deleted the fix-syscall-imports branch May 8, 2016 23:16
dmitshur added a commit that referenced this pull request Jun 26, 2016
The real time package doesn't import strings package. This keeps the
import graph of augmented time package consistent with the real
standard library.

Source: https://godoc.org/time?import-graph

This is a change analogous to #453.
dmitshur added a commit that referenced this pull request Jun 26, 2016
The real time package doesn't import strings package. This keeps the
import graph of augmented time package consistent with the real
standard library.

Source: https://godoc.org/time?import-graph

This is a change analogous to #453.
dmitshur added a commit that referenced this pull request Jun 26, 2016
The real time package doesn't import strings package. This keeps the
import graph of augmented time package consistent with the real
standard library.

Source: https://godoc.org/time?import-graph

This is a change analogous to #453.
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.

2 participants