Skip to content

Add release for android OS #905

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
Jan 19, 2020
Merged

Add release for android OS #905

merged 1 commit into from
Jan 19, 2020

Conversation

uudashr
Copy link
Contributor

@uudashr uudashr commented Jan 3, 2020

Add release for android. Some of use Termux on Android as alternative development machine.

.golangci.yml Outdated
@@ -44,7 +44,6 @@ linters-settings:
- dupImport # https://github.com/go-critic/go-critic/issues/845
- ifElseChain
- octalLiteral
- whyNoLint
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, this is needed for latest v1.22.x releases due to go-critic update. The GitHub check should pass once go-critic/go-critic#845 is merged and deployed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any suggestion what should I do now?

  1. Return the whyNoLint config
  2. ???

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just undo the whyNoLint removal commit and it should be good to merge.

Copy link
Member

@ernado ernado Jan 8, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GolangCI failure is expected, yes.

To undo this commit you can do the following:

$ git reset --hard 6fd38300fbdc897e6786e899717e712cbecde0b8
$ git push -f uudashr 

Where 6fd3830 is your first commit, and uudashr is your fork remote name.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tpounds already undo the whyNoLint, should good now

@tpounds tpounds added enhancement New feature or improvement installation labels Jan 4, 2020
@jirfag jirfag merged commit 8b29179 into golangci:master Jan 19, 2020
@jirfag
Copy link
Contributor

jirfag commented Jan 19, 2020

thank you!

@jirfag
Copy link
Contributor

jirfag commented Jan 19, 2020

I've needed to revert this change because a new release can't be built on android. I got errors like these:

   ⨯ release failed after 35.30s error=failed to build for android_amd64: # github.com/golangci/golangci-lint/cmd/golangci-lint
/usr/local/go/pkg/tool/darwin_amd64/link: running clang failed: exit status 1
ld: warning: option -s is obsolete and being ignored
ld: unknown option: -z
clang: error: linker command failed with exit code 1 (use -v to see invocation)

   ⨯ release failed after 621.89s error=failed to build for android_arm64: # github.com/golangci/golangci-lint/cmd/golangci-lint
/usr/local/go/pkg/tool/darwin_amd64/link: running clang failed: exit status 1
clang: error: invalid linker name in argument '-fuse-ld=gold'

@tpounds
Copy link
Contributor

tpounds commented Jan 19, 2020

@jirfag AFAICT, we need to have cgo and the Android NDK to build this release target.

@jirfag
Copy link
Contributor

jirfag commented Jan 20, 2020

thank you,
@tpounds @uudashr will you try to make a new pull request with adding needed libraries? If yes - please check locally that make deploy works (set flags to just build but not deploy).

@uudashr
Copy link
Contributor Author

uudashr commented Feb 10, 2020

Somehow go get also failed on android

$ GO111MODULE=on go get github.com/golangci/golangci-lint/cmd/golangci-lint@master
go: finding github.com/golangci/golangci-lint master
go: finding github.com/golangci master
go: finding github.com/golangci/golangci-lint/cmd/golangci-lint master
go: finding github.com/golangci/golangci-lint/cmd master
go: finding github.com master
go: downloading github.com/golangci/golangci-lint v1.23.6
go: extracting github.com/golangci/golangci-lint v1.23.6
# github.com/golangci/golangci-lint/cmd/golangci-lint
/data/data/com.termux/files/usr/lib/go/pkg/tool/android_arm64/link: running aarch64-linux-android-clang failed: exit status 1
clang-9: error: invalid linker name in argument '-fuse-ld=gold'

@ldez ldez added this to the v1.23 milestone Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants