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

mcookie: Add support for human-readable sizes with the -m option #257

Closed
alxndrv opened this issue Mar 18, 2025 · 3 comments · Fixed by #268
Closed

mcookie: Add support for human-readable sizes with the -m option #257

alxndrv opened this issue Mar 18, 2025 · 3 comments · Fixed by #268
Labels
good first issue Good for newcomers mcookie

Comments

@alxndrv
Copy link
Contributor

alxndrv commented Mar 18, 2025

The uutils implementation of mcookie currently doesn't support parsing human-readable sizes provided via the -m option.
We should support units like KiB, MiB etc. like the GNU implementation does.

Relevant code: https://github.com/uutils/util-linux/blob/main/src/uu/mcookie/src/mcookie.rs#L34-L37

@alxndrv
Copy link
Contributor Author

alxndrv commented Mar 18, 2025

@sylvestre Opened this as you requested, but don't have permissions to set the label. Mind adding the label for me? 🙏

@sylvestre sylvestre added the good first issue Good for newcomers label Mar 21, 2025
@sylvestre
Copy link
Contributor

sure, done :)

@quangcito
Copy link
Contributor

I'm taking this up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers mcookie
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants