Upgrade to go 1.16 and add darwin-arm64 support.

[DennoVonDiesel]

go 1.16 is required for darwin-arm64 support. I've added this to the build process.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[williantenfen]

nice

@yuting-fan

[williantenfen]

@DennoVonDiesel when I try to use the install file as: install -i /usr/local/sessionmanagerplugin -b /usr/local/bin/session-manager-plugin from path session-manager-plugin/Tools/src/darwin it displays:

install: illegal option -- i
usage: install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
               [-o owner] file1 file2
       install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
               [-o owner] file1 ... fileN directory
       install -d [-v] [-g group] [-m mode] [-o owner] directory ...

I'm missing some steps?

[DennoVonDiesel]

@williantenfen I don't see that the system command install has a -i option on either darwin or linux. I think the correct way is to unzip the bundle file that gets created after building the package and run that installer.

[williantenfen]

thanks! got it. working fine!

[andymac4182]

This is great hopefully @Yangtao-Hua or @yuting-fan can look at merging soon.

[kylebebak]

@Yangtao-Hua @yuting-fan

It would be really nice to get session-manager-plugin working for M1 machines without having to use a fork of this library!

[Yangtao-Hua]

Thank you for the changes, we noticed it is highly required feature, it is in our plan.

[andymac4182]

Do you have an ETA @Yangtao-Hua ?

[andymac4182]

@Yangtao-Hua This is flagged as a highly required feature but no movement in over 3 months.

Should any community work towards this repository move away from AWS and be managed by the community due to the lack of engagement?

[zbintliff]

I think go:build isn't added until 1.17.

Any reason not to upgrade to latest go version (which also removes the +build lines)?

[nitikaaws]

We are actively working to add darwin-arm64 support and this change is currently under review. Will keep this thread updated on the release as we make progress.

[andymac4182]

Any update @nitikaaws ?

[mbaranovski]

Nice definition of "actively" @nitikaaws
Seriously guys... what's taking so long, this is absurd.

cc. @yuting-fan @Yangtao-Hua @YujiaozhAws

[nitikaaws]

Apologies for the delay in releasing the support for darwin-arm64! Our team is looking into this and will provide an update when we are closer to the release.

[andymac4182]

@nitikaaws Any chance of a community release? You said you were reviewing this change in November.

[nitikagoyal87]

Session Manager Plugin support for darwin-arm64 has been released with version 1.2.463.0. This pull request has been incorporated in 60f4fa7 commit. Thanks for your patience and support, please let us know your feedback.