Skip to content

Add support for M1 ARM#85

Merged
cezmunsta merged 8 commits into
masterfrom
m1-arm
Mar 11, 2023
Merged

Add support for M1 ARM#85
cezmunsta merged 8 commits into
masterfrom
m1-arm

Conversation

@cezmunsta
Copy link
Copy Markdown
Owner

The currently supported architectures are:

  • linux/amd64
  • darwin/amd64

Support for the M1 is required.

Resolves #84

The currently supported architectures are:

* `linux/amd64`
* `darwin/amd64`

Support for the M1 is required.

Resolves #84
@cezmunsta cezmunsta added enhancement New feature or request help wanted Extra attention is needed labels Apr 28, 2022
@cezmunsta cezmunsta requested a review from Oneiroi April 28, 2022 10:35
@cezmunsta cezmunsta self-assigned this Apr 28, 2022
@cezmunsta cezmunsta marked this pull request as draft April 28, 2022 10:35
@cezmunsta cezmunsta marked this pull request as ready for review April 28, 2022 11:17
Copy link
Copy Markdown
Collaborator

@Oneiroi Oneiroi left a comment

Choose a reason for hiding this comment

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

Makefile changes appear sane, however as per slack some of the tests are failing:

make test
ok  	github.com/cezmunsta/ssh_ms/ssh	0.408s
# github.com/shirou/gopsutil/disk
iostat_darwin.c:28:2: warning: 'IOMasterPort' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:132:1: note: 'IOMasterPort' has been explicitly marked deprecated here
# github.com/shirou/gopsutil/host
smc_darwin.c:75:41: warning: 'kIOMasterPortDefault' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:123:19: note: 'kIOMasterPortDefault' has been explicitly marked deprecated here
1.7.0
time="2022-04-28T12:23:44+01:00" level=fatal msg="Failed to lookup https://github.com/cezmunsta/ssh_ms/releases/latest"
FAIL	github.com/cezmunsta/ssh_ms/cmd	11.068s
ok  	github.com/cezmunsta/ssh_ms/vault	4.246s
ok  	github.com/cezmunsta/ssh_ms/log	0.193s
ok  	github.com/cezmunsta/ssh_ms/config	0.291s
FAIL
make: *** [test] Error 1

@Oneiroi
Copy link
Copy Markdown
Collaborator

Oneiroi commented Mar 10, 2023

Tests are passing 👍

 ssh_ms git:(m1-arm) make test
ok  	github.com/cezmunsta/ssh_ms/ssh	(cached)
ok  	github.com/cezmunsta/ssh_ms/cmd	5.595s
ok  	github.com/cezmunsta/ssh_ms/vault	(cached)
ok  	github.com/cezmunsta/ssh_ms/log	(cached)
ok  	github.com/cezmunsta/ssh_ms/config	(cached)

@cezmunsta
Copy link
Copy Markdown
Owner Author

Tests are passing +1

Cheers for checking! 🍻

@cezmunsta cezmunsta merged commit e898c65 into master Mar 11, 2023
@cezmunsta cezmunsta deleted the m1-arm branch March 11, 2023 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for M1 ARM

2 participants