Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: check free disk space and allow to specify a different download/extraction directory#23

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
MatteoPologruto wants to merge10 commits intomain
base:main
Choose a base branch
Loading
fromset-different-directory

Conversation

@MatteoPologruto
Copy link
Contributor

@MatteoPologrutoMatteoPologruto commentedNov 3, 2025
edited
Loading

Motivation

The default download/extraction directory might be on a disk with insufficient space.

Change description

Two changes have been introduced:

  • allow the user to set a different temporary directory in which the Debian image is downloaded and/or extracted, using the--temp-dir flag;
  • check that the temporary directory is in a disk with enough free space before starting a download/extraction.

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged inmain.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@MatteoPologrutoMatteoPologruto self-assigned thisNov 3, 2025
@MatteoPologrutoMatteoPologruto added the bugSomething isn't working labelNov 3, 2025
per1234
per1234 previously requested changesNov 3, 2025
@MatteoPologrutoMatteoPologruto marked this pull request as ready for reviewNovember 3, 2025 16:22
@MatteoPologrutoMatteoPologruto requested a review froma teamNovember 3, 2025 16:22
Co-authored-by: Per Tillisch <accounts@perglass.com>
@per1234per1234 dismissed theirstale reviewNovember 3, 2025 16:26

Requested changes have been made. Thanks!

MatteoPologrutoand others added4 commitsNovember 6, 2025 14:30
Co-authored-by: Luca Rinaldi <lucarin@protonmail.com>
Co-authored-by: Luca Rinaldi <lucarin@protonmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@lucarin91lucarin91lucarin91 left review comments

@per1234per1234per1234 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@MatteoPologrutoMatteoPologruto

Labels

bugSomething isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Check available disk space before downloading/extracting images

3 participants

@MatteoPologruto@lucarin91@per1234

[8]ページ先頭

©2009-2025 Movatter.jp