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

Add support for install on armv7 and aarch64 #413

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

crodjer
Copy link
Contributor

@crodjer crodjer commented May 28, 2021

This allows the script to work with Raspbian/Arch/Manjaro running on a Raspberry Pi

@crodjer crodjer changed the title Add support for install on armv7l Add support for install on armv7 and aarch64 Jun 11, 2021
@crodjer
Copy link
Contributor Author

crodjer commented Jun 11, 2021

Hi @lotabout / maintainers, requesting a review for these changes.
I have been able to use the test release on my fork for this on my Raspberry Pis.

@LoricAndre
Copy link
Contributor

A refactor of the CI is in the works, this will be part of it.

install Outdated Show resolved Hide resolved
@crodjer crodjer force-pushed the install-armv7 branch 4 times, most recently from 34bccbd to 673530b Compare November 22, 2024 15:11
@LoricAndre
Copy link
Contributor

This looks good ! I'll test the different architectures to make sure everything works, and we'll be good to merge this quickly

@crodjer
Copy link
Contributor Author

crodjer commented Nov 22, 2024

@LoricAndre One thing that I discovered, the --version flag is no more.
Was that intended? Or perhaps was miss.

@LoricAndre
Copy link
Contributor

It was a miss, I just noticed it while testing. A hotfix should be available as v0.11.11 in a few minutes

@LoricAndre
Copy link
Contributor

Should be good

install Outdated Show resolved Hide resolved
@LoricAndre
Copy link
Contributor

While we're at it, do you know what the output of uname -sm is on Mac M-Series processors ? It would be nice to add it

@LoricAndre
Copy link
Contributor

I'd like to merge this quickly, as it made me realize that the install script needs a lot of work to make it more usable, and I'll start working on it as soon as we're done with this change

@crodjer
Copy link
Contributor Author

crodjer commented Nov 22, 2024

uname -sm
For M series, that should be Darwin arm64, based on what I use in my project.

@crodjer
Copy link
Contributor Author

crodjer commented Nov 22, 2024

@LoricAndre Added Darwin arm64 and a fallback message for the unknown platforms.

Also,

- Use the Github API to download the latest `version`
- Use the new release artefact format
- Fix grep pattern to work across Mac / Linux
- Fix version check to strip out `sk` prefix
- Exit when existing version already matches

This allows the install script to work with Raspbian running on a
Raspberry Pi
@crodjer
Copy link
Contributor Author

crodjer commented Nov 22, 2024

@LoricAndre I realized that the version check also wasn't working right (as there is sk prefix present) and on Mac the -P isn't available on grep.

@LoricAndre
Copy link
Contributor

Is this working on MacOS now ?

@LoricAndre
Copy link
Contributor

If it is, then everything LGTM

@crodjer
Copy link
Contributor Author

crodjer commented Nov 22, 2024

Yes, I think should be good.

@LoricAndre LoricAndre merged commit 08a8a74 into skim-rs:master Nov 22, 2024
khafatech pushed a commit to khafatech/skim that referenced this pull request Nov 23, 2024
…im-rs#413)

Also,

- Use the Github API to download the latest `version`
- Use the new release artefact format
- Fix grep pattern to work across Mac / Linux
- Fix version check to strip out `sk` prefix
- Exit when existing version already matches

This allows the install script to work with Raspbian running on a
Raspberry Pi
LoricAndre added a commit that referenced this pull request Nov 26, 2024
* Fix example link in README.md

* Fix typo and improve punctuation

* change example url to the correct repo

* Fix some links and wording

* Add support for install on Linux armv7l, aarch64 and Darwin arm64 (#413)

Also,

- Use the Github API to download the latest `version`
- Use the new release artefact format
- Fix grep pattern to work across Mac / Linux
- Fix version check to strip out `sk` prefix
- Exit when existing version already matches

This allows the install script to work with Raspbian running on a
Raspberry Pi

* chore: remove some platform-specific quirkinesses from e2e (#602)

* chore: remove some platform-specific quirkinesses from e2e

* debug

* Revert "debug"

This reverts commit ed065c3.

---------

Co-authored-by: LoricAndre <[email protected]>

* Improve grammar and wording in the readme

---------

Co-authored-by: LoricAndre <[email protected]>
Co-authored-by: Rohan Jain <[email protected]>
Co-authored-by: LoricAndre <[email protected]>
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.

3 participants