-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added updated darwin support for Apple Silicon Macs and new Software Updates #2
Conversation
@Ishaanahuja7 Thank you! |
Hi @Ishaanahuja7 and @usiegl00, sorry for the lack of movement on this so far! I'm starting to work on these this week, so I'm hoping we can at least get these sigar changes pulled into mettle. Currently working on setting this up and testing at the moment! |
Any updates? @space-r7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Hey @Ishaanahuja7, I'm on research at the moment. I'll be able to return to this the first week of January. Sorry for the delay on this! |
I can probably take a look at this in the meantime if you don't mind @space-r7, just got an M1 for this kind of thing. |
Not at all! Thanks, Brent! |
|
@busterb Looks good to me. Can anybody merge this PR? |
This should be merged now, but I'm not sure why github didn't automatically mark this pull request as merged. |
@timwr Can you make a new tag and bump version to 1.6.5 so I can update the tar.gz in mettle? |
This was landed to the wrong branch, I will rework commits and tag shortly. |
@timwr sorry for the confusion here, we are using Merges have been sorted and new tag for 1.6.5 exists. |
Updates
darwin_sigar.c
for latest macOS version support.