Thanks to visit codestin.com
Credit goes to github.com

Skip to content

version strategy: zonkio's arm arch name are different from GOOARCH #25

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 5 commits into from
May 19, 2021

Conversation

bearsh
Copy link
Contributor

@bearsh bearsh commented Apr 23, 2021

on arm (32bit), use 'uname -m' to get the excact arm version

@bearsh
Copy link
Contributor Author

bearsh commented Apr 23, 2021

regarding tests, AFAIK there's no easy way to setup a github action which runs on arm...

@fergusstrange
Copy link
Owner

First of all apologies @bearsh this totally slipped me by.

I had a look into adding a CircleCI/Travis build that will be able to run the platform-tests on ARM, which I think is a must if we're going to claim support. I'm happy to have a play around with this but it may take me a little while. In the meantime see if you can get the build up and running mimicking the same behaviour as the Github Actions platform-test build?

Thanks

@fergusstrange
Copy link
Owner

fergusstrange commented May 18, 2021

Hey @bearsh I've added some CircleCI tests for arm64 though there's no coverage for arm32. Can you rebase off this branch and see if the tests pass?

If we can get that coverage at least I'll merge in.

Thanks!

fergusstrange and others added 2 commits May 18, 2021 20:38
on arm (32bit), use 'uname -m' to get the excact arm version
@bearsh bearsh force-pushed the feature/arm-version branch from d9a54db to 65f76f6 Compare May 18, 2021 12:48
@bearsh
Copy link
Contributor Author

bearsh commented May 18, 2021

mmh it seems simply force push doesn't add the new tests to this PR

@fergusstrange
Copy link
Owner

Urgh. I'll merge as is and we'll fix this up on master after we have a build to work from.

@fergusstrange fergusstrange merged commit 06efa36 into fergusstrange:master May 19, 2021
@fergusstrange
Copy link
Owner

Looks like we got a green build first hit @bearsh https://app.circleci.com/pipelines/github/fergusstrange/embedded-postgres/8/workflows/5735b0b4-0712-47a6-bcd9-f447924decc8/jobs/12

Great work! As mentioned in #26 I may look to rework the version strategy to get some coverage in there now we have some more complex logic.

If you fancy a crack at this feel free to chip in and happy to work with you / review more PRs.

@fergusstrange
Copy link
Owner

Release published with your changes @bearsh.

@bearsh bearsh deleted the feature/arm-version branch July 14, 2021 15:38
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