Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

install: use a cpio passthrough function for root #104

Merged
merged 1 commit into from
Jul 11, 2018

Conversation

rminnich
Copy link
Member

Just doing a dumb raw dd of partitions is too dangerous.
This way we can figure out if the cpio is valid and not
copy the whole partition if the cpio is smaller (and it
is by a lot).

Signed-off-by: Ronald G. Minnich rminnich@gmail.com

@rminnich
Copy link
Member Author

untested.

@rminnich rminnich force-pushed the fixrootcopy branch 2 times, most recently from 2c63c6a to 7095923 Compare July 1, 2018 03:37
@rminnich
Copy link
Member Author

rminnich commented Jul 1, 2018

the install works, can't test, my celes is still pretty busted.

@TrevorFarrelly
Copy link
Member

TrevorFarrelly commented Jul 2, 2018

This works, but it looks like it shouldn't. After install every symlink on startup fails, but it still boots and everything seems to be there. @jbeta51 and I are unreliably reproducing it, we're not sure what the problem is.

@rminnich
Copy link
Member Author

rminnich commented Jul 9, 2018

thanks for testing, we will need to try to figure out in more detail.

Just doing a dumb raw dd of partitions is too dangerous.
This way we can figure out if the cpio is valid and not
copy the whole partition if the cpio is smaller (and it
is by a lot).

This does an install on my broken celes, I can't test it because
the celes is broken ... but it does do the full install in 18 seconds.

Signed-off-by: Ronald G. Minnich <rminnich@gmail.com>
@rminnich rminnich changed the title WIP: use a cpio passthrough function for root install: use a cpio passthrough function for root Jul 11, 2018
@rminnich
Copy link
Member Author

Turns out I had forgotten to write a trailer record
This one might do it.

Copy link
Contributor

@jbeta51 jbeta51 left a comment

Choose a reason for hiding this comment

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

We need to remember to create the pass function

@rminnich rminnich merged commit a535f42 into u-root:master Jul 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants