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

Clarify README section on differences with xz #115

Closed
bearrito opened this issue Mar 22, 2024 · 5 comments
Closed

Clarify README section on differences with xz #115

bearrito opened this issue Mar 22, 2024 · 5 comments

Comments

@bearrito
Copy link

I'm not critiquing the decision, I'm trying to understand if there is a technical hurdle lurking in the background.

Right now we use xz as part of our A/B partition update for firmware like systems. As part of how the system works, we need to stream to stdout. The performance of pixz is impressive so we'd like to use it...

@vasi
Copy link
Owner

vasi commented Mar 23, 2024

Stdout is totally supported! I can do cat SOMEFILE | pixz | wc -c and I get totally reasonable results. Is there some situation where stdout is not working for you?

@bearrito
Copy link
Author

@vasi Maybe I'm misinterpreting this statement, my fault if so.

pixz does not support the command line option -c or --stdout

Namely we have something like this

xz -d -c $IMAGE_PATH | sudo dd skip=$SKIP_BLOCKS of=$PARTITION 

@vasi
Copy link
Owner

vasi commented Mar 23, 2024

Ah, so your problem is we don't have the exact same arguments as xz. But you can still do the same operation, just worded a bit different:

pixz -d < $IMAGE_PATH | sudo dd skip=$SKIP_BLOCKS of=$PARTITION

Probably we should edit the README.md to be clearer!

@bearrito
Copy link
Author

Thanks for the fast response. Get some sleep (or not) !
I'll try that tomorrow!

@vasi vasi changed the title [Question] Why was stdout out not supported? Clarify README section on differences with xz Mar 23, 2024
@vasi
Copy link
Owner

vasi commented May 17, 2024

Updated the readme!

@vasi vasi closed this as completed May 17, 2024
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

No branches or pull requests

2 participants