[PR #189] [MERGED] Propose RAM default to 2G instead of 8G #726

New Issue

2025-12-29T01:24:51+01:00

adam commented

2025-12-29 01:24:51 +01:00

📋 Pull Request Information

Original PR: https://github.com/sickcodes/Docker-OSX/pull/189
Author: @sickcodes
Created: 3/14/2021
Status: ✅ Merged
Merged: 3/14/2021
Merged by: @sickcodes

Base: master ← Head: low-ram-bug

📝 Commits (1)

5024575 Propose RAM default to 2G instead of 8G

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 Dockerfile (+1 -1)

📄 Description

Fixes https://github.com/sickcodes/Docker-OSX/issues/188#issue-831137065

I'm not sure if this is a new bug but it's the first time I've seen it, after reading @C9-Dev's comment:

https://github.com/sickcodes/Docker-OSX/issues/131#issuecomment-798802122

This PR changes default RAM to 2000 which is the minimum.

Perhaps in addition, I could add

-e RAM=auto \

This would only be for blind machines... as most users tend to know how much RAM they have, or their server has already.

RAM=${RAM:-$(("$(head -n1 /proc/meminfo | tr -dc '[:digit:]') / 900000"))}000 \
exec qemu-system-x86_64 -m ${RAM} \

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/sickcodes/Docker-OSX/pull/189 **Author:** [@sickcodes](https://github.com/sickcodes) **Created:** 3/14/2021 **Status:** ✅ Merged **Merged:** 3/14/2021 **Merged by:** [@sickcodes](https://github.com/sickcodes) **Base:** `master` ← **Head:** `low-ram-bug` --- ### 📝 Commits (1) - [`5024575`](https://github.com/sickcodes/Docker-OSX/commit/502457528a91bbe1f872eb6de5ab59c53d9d9183) Propose RAM default to 2G instead of 8G ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+1 -1) </details> ### 📄 Description Fixes https://github.com/sickcodes/Docker-OSX/issues/188#issue-831137065 I'm not sure if this is a new bug but it's the first time I've seen it, after reading @C9-Dev's comment: https://github.com/sickcodes/Docker-OSX/issues/131#issuecomment-798802122 This PR changes default RAM to 2000 which is the minimum. Perhaps in addition, I could add ```dockerfile -e RAM=auto \ ``` This would only be for blind machines... as most users tend to know how much RAM they have, or their server has already. ```bash RAM=${RAM:-$(("$(head -n1 /proc/meminfo | tr -dc '[:digit:]') / 900000"))}000 \ exec qemu-system-x86_64 -m ${RAM} \ ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-29 01:24:51 +01:00

adam closed this issue

2025-12-29 01:24:51 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/Docker-OSX#726