[PR #5841] [MERGED] fix(cli): don't open too many files #10993

Closed
opened 2026-02-05 14:31:39 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/5841
Author: @etnoy
Created: 12/19/2023
Status: Merged
Merged: 12/19/2023
Merged by: @alextran1502

Base: mainHead: fix/cli-open-files


📝 Commits (10+)

📊 Changes

3 files changed (+5 additions, -5 deletions)

View changed files

📝 cli/package-lock.json (+3 -4)
📝 cli/package.json (+1 -0)
📝 cli/src/cores/models/asset.ts (+1 -1)

📄 Description

This PR should fix #5271 by using graceful-fs which appears to be the only way to avoid EMFILE in nodejs.

Will await some testing to see if this fixes the issue

This PR depends on #5101


🔄 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/immich-app/immich/pull/5841 **Author:** [@etnoy](https://github.com/etnoy) **Created:** 12/19/2023 **Status:** ✅ Merged **Merged:** 12/19/2023 **Merged by:** [@alextran1502](https://github.com/alextran1502) **Base:** `main` ← **Head:** `fix/cli-open-files` --- ### 📝 Commits (10+) - [`1d385b6`](https://github.com/immich-app/immich/commit/1d385b6cbef4839eae1e5e62521ed87f57b41f26) Allow building and installing cli - [`4439bc2`](https://github.com/immich-app/immich/commit/4439bc23559d5750ca45e690aadfc1df0cb05e96) Merge branch 'main' of https://github.com/immich-app/immich into feat/cli-albums - [`1b7c8a2`](https://github.com/immich-app/immich/commit/1b7c8a29cbf8cda45ed8114d4e895d061f43e3be) feat: add format fix - [`a983edd`](https://github.com/immich-app/immich/commit/a983eddb55ff8d1beb9f94fca3575c8520aeff61) docs: remove cli folder - [`b2bbdf4`](https://github.com/immich-app/immich/commit/b2bbdf423ab0db6faeb718328e619598e17259ee) Merge branch 'main' of https://github.com/immich-app/immich into feat/cli-albums - [`55615bc`](https://github.com/immich-app/immich/commit/55615bcd15a2a4ecdc9b6adc9feecc5cdc87ff3f) feat: use immich scoped package - [`9d25a60`](https://github.com/immich-app/immich/commit/9d25a60455ab7486cc9ad3a25d9c15dfa93defa9) feat: rewrite cli readme - [`06d78d2`](https://github.com/immich-app/immich/commit/06d78d2dacea40ddbdfe441ec6e1b0aa21aeac78) docs: add info on running without building - [`db28694`](https://github.com/immich-app/immich/commit/db28694f6889fb86ffc7da022bd79dc831e85b23) cleanup - [`f588416`](https://github.com/immich-app/immich/commit/f5884166594f122b97af60e04f788236aa07b21d) chore: remove import functionality from cli ### 📊 Changes **3 files changed** (+5 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `cli/package-lock.json` (+3 -4) 📝 `cli/package.json` (+1 -0) 📝 `cli/src/cores/models/asset.ts` (+1 -1) </details> ### 📄 Description This PR should fix #5271 by using graceful-fs which appears to be the only way to avoid EMFILE in nodejs. Will await some testing to see if this fixes the issue This PR depends on #5101 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 14:31:39 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#10993