Skip to content

Conversation

@appleboy
Copy link
Member

@appleboy appleboy commented Dec 3, 2016

critical issue #752

Please help to review.

cc @javierprovecho @tboerger

Signed-off-by: Bo-Yi Wu [email protected]

@tboerger
Copy link
Contributor

tboerger commented Dec 3, 2016

And why do you also ignore bsd?

@javierprovecho
Copy link
Member

@javierprovecho javierprovecho merged commit 556287f into master Dec 3, 2016
@vermotr
Copy link

vermotr commented Dec 3, 2016

I also use Google App Engine and I have the same issue #752 than @kujenga:

go-app-builder: Failed parsing input: parser: bad import "syscall" in golang.org/x/crypto/ssh/terminal/terminal.go from GOPATH

I have temporarily fixed it by checking out the bb159f9 commit, but can you try to fix this problem?

javierprovecho added a commit that referenced this pull request Dec 3, 2016
This reverts commit 556287f, reversing
changes made to 32cab50.
@javierprovecho
Copy link
Member

@vermotr sorry, master from github and gopkg mirror is now stable at v1.1.3, with both this and #744 reverted on master. We'll keep working on this feature in develop

@javierprovecho javierprovecho added the type/bug Found something you weren't expecting? Report it here! label Dec 3, 2016
@vermotr
Copy link

vermotr commented Dec 3, 2016

@javierprovecho No worries, thanks a lot for the fix!

@appleboy
Copy link
Member Author

appleboy commented Dec 4, 2016

@vermotr I make new PR to ignore Google App Engine os type. Please help to test again #754.

appleboy pushed a commit that referenced this pull request Feb 1, 2017
* Revert "Merge pull request #753 from gin-gonic/bug"

This reverts commit 556287f, reversing
changes made to 32cab50.

* Revert "Merge pull request #744 from aviddiviner/logger-fix"

This reverts commit c3bfd69, reversing
changes made to 9177f01.

* Update gin version
appleboy pushed a commit that referenced this pull request May 29, 2017
* Revert "Merge pull request #753 from gin-gonic/bug"

This reverts commit 556287f, reversing
changes made to 32cab50.

* Revert "Merge pull request #744 from aviddiviner/logger-fix"

This reverts commit c3bfd69, reversing
changes made to 9177f01.

* add custom Delims support

* add some test for Delims

* remove the empty line for import native package

* remove unuseful comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/bug Found something you weren't expecting? Report it here!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants