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

feat: import @hargonix hardware diff #11

Open
wants to merge 65 commits into
base: master
Choose a base branch
from

Conversation

hargoniX
Copy link

This imports most of my hardware API related diff in the repo. I did not succeed in properly rebasing my branch onto humenda/master after trying for half a day sadly. This patch alone is not quite capable of building as some things that I can't seem to figure out are still failing but I'll leave it here for now regardless.

@humenda
Copy link
Owner

humenda commented Nov 14, 2023 via email

@hargoniX
Copy link
Author

hargoniX commented Nov 14, 2023

It's mostly the amount of changes, I think it is doable with enough attention to detail, I'm going to try again later.

@hargoniX
Copy link
Author

This is properly rebased now but it doesn't build yet and also contains all of my thesis.

I'm gonna look into fixing the build.

Do you want verix to be here as well?

@hargoniX hargoniX force-pushed the rustl4rehumenda branch 10 times, most recently from cfdb29a to a048e73 Compare November 14, 2023 15:46
@hargoniX
Copy link
Author

With the additional fix commit everything is compiling now. I'll check if it still does the correct thing on irl hardware on Wednesday.

@hargoniX hargoniX marked this pull request as ready for review November 15, 2023 09:42
@hargoniX
Copy link
Author

hargoniX commented Nov 15, 2023

Tested, everything is still working.

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.

None yet

2 participants