Skip to content

Conversation

@ilgooz
Copy link
Member

@ilgooz ilgooz commented Jun 25, 2021

and upgrade protoc & add linux arm support.

  • refactor data dirs for protoc and nodetime pkgs.

related to #1244.

and upgrade protoc & add linux arm support.

* refactor data dirs for protoc and nodetime pkgs.
@ilgooz ilgooz requested review from Pantani, fadeev and lumtis as code owners June 25, 2021 09:51
@ilgooz
Copy link
Member Author

ilgooz commented Jun 25, 2021

@Pantani I believe you're using an M1 machine, can you please test this PR for us to see if it works there okay?

Pantani
Pantani previously approved these changes Jun 26, 2021
lumtis
lumtis previously approved these changes Jun 28, 2021
Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
@ilgooz ilgooz dismissed stale reviews from lumtis and Pantani via a0b3214 June 29, 2021 06:13
@ilgooz ilgooz merged commit 1fd2cac into develop Jun 29, 2021
@ilgooz ilgooz deleted the refactor/nodetime-protoc branch June 29, 2021 06:14
@faddat
Copy link
Contributor

faddat commented Jul 1, 2021

works on M1.

Does not seem to work on ARM64 linux.

investigating it now

@faddat
Copy link
Contributor

faddat commented Jul 1, 2021

@faddat
Copy link
Contributor

faddat commented Jul 1, 2021

Any chance that we can merge feat/linux-arm64?

Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* feat(cmd/tools): add 'protoc' command and

and upgrade protoc & add linux arm support.

* refactor data dirs for protoc and nodetime pkgs.

* fix

* docs

* Apply suggestions from code review

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
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.

5 participants