humenda / rustl4re

This repository contains patches for the Rust to L4Re OS port. It's a place to gather patches before they find their way in the upstream projects.
15 stars 6 forks source link

Moved to l4re-snapshot 21.7 #2

Closed sevenautumns closed 3 years ago

sevenautumns commented 3 years ago

I tried bringing the master branch to snapshot version 21.7.
In the process I somewhat disconnected the master branch.

What I did:

I only tested the result for amd64 with make qemu. Both the hello-shared and Rust based two_task_ipc seem to work perfectly.

humenda commented 3 years ago

Wow, this is pretty awesome! I have been wanting this for a long time and didn't make any progress due to lack of time. Please give me a few days to review.

sevenautumns commented 3 years ago

Wow, this is pretty awesome! I have been wanting this for a long time and didn't make any progress due to lack of time. Please give me a few days to review.

Glad to hear that. Ill take from the files in this folder:
https://github.com/humenda/rustl4re/tree/master/src/l4/mk/arch
that this project is not expected to succeed for other architectures than amd64, am I correct with that assumption? Would further contributions regarding support for arm be welcome?

humenda commented 3 years ago

Is there a chance you could rebase your work such that upstream contains the current snapshot, i.e. a second commit to the first snapshot and rebase master onto it? Ideally, I'd like to have a workflow similar to the one of git-buildpackage, but I'm not sure how to do a merge from upstream to master that doesn't delete the files added on master.

sevenautumns commented 3 years ago

I dont quite understand.
How should the commit history of master be structured afterwards

Right now your master branch is structured like this:
Initial commit of l4re-snapshot 18.7 -- strip l4linux -- implementation of l4rust
The first two commits are taken from upstream

My master branch is structured like this: Initial commit of l4re-snapshot 18.7 -- strip l4linux -- Update to l4re-snapshot 21.7` -- rebased implementation of l4rust -- fixes for 21.7

I understand that upstream should either look like this: Initial commit of l4re-snapshot 18.7 -- strip l4linux -- Update to l4re-snapshot 21.7 or: Initial commit of l4re-snapshot 18.7 -- Update to l4re-snapshot 21.7 -- strip l4linux

But what should my pull request master look like to be compliant ?

humenda commented 3 years ago

It looks strange with the 0 commits merged, but it got merged :).