Skip to content

Open-archiver: approve pnpm build scripts and run build:oss without subshell - #14711

Merged
michelroegl-brunner merged 1 commit into
mainfrom
fix/open-archiver-pnpm-build
May 26, 2026
Merged

Open-archiver: approve pnpm build scripts and run build:oss without subshell#14711
michelroegl-brunner merged 1 commit into
mainfrom
fix/open-archiver-pnpm-build

Conversation

@MickLesk

Copy link
Copy Markdown
Member

✍️ Description

🔗 Related Issue

Fixes #14694

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to script metadata (PocketBase/website data).
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@MickLesk
MickLesk requested a review from a team as a code owner May 25, 2026 19:13
@github-actions github-actions Bot added bugfix update script A change that updates a script labels May 25, 2026
@MickLesk

Copy link
Copy Markdown
Member Author

Investigate an STD issue, therefore its missing here atm

@MickLesk
MickLesk force-pushed the fix/open-archiver-pnpm-build branch from 6e91eb9 to dce2984 Compare May 25, 2026 19:16
@MickLesk

Copy link
Copy Markdown
Member Author

okay work with STD now

@michelroegl-brunner
michelroegl-brunner merged commit 37649c1 into main May 26, 2026
1 check passed
@derpuma

derpuma commented May 27, 2026

Copy link
Copy Markdown

Hi, I get an error during install at the "pnpm approve-builds --yes" command.

Looks like it is a problem of
CLEAN_INSTALL=1 fetch_and_deploy_gh_release "openarchiver" "LogicLabs-OU/OpenArchiver" "tarball"

Details from installation log:

✔️ Deployed: openarchiver (0.5.0)
✖️ in line 49: exit code 1 (General error / Operation not permitted): while executing command pnpm approve-builds --yes

--- Last 20 lines of log ---

  • prettier-plugin-tailwindcss 0.6.14
  • typescript 5.8.3
  • vitepress 1.6.4
  • vitepress-openapi 0.1.18

╭ Warning ─────────────────────────────────────────────────────────────────────╮
│ │
│ Ignored build scripts: @tailwindcss/oxide, msgpackr-extract, sqlite3, │
│ vue-demi. │
│ Run "pnpm approve-builds" to pick which dependencies should be allowed │
│ to run scripts. │
│ │
╰──────────────────────────────────────────────────────────────────────────────╯

packages/frontend prepare$ svelte-kit sync || echo ''
packages/frontend prepare: Done
Done in 12.8s using pnpm v10.13.1
 ERROR  Unknown option: 'yes'
For help, run: pnpm help approve-builds
[2026-05-27 08:45:33] [ERROR] in line 49: exit code 1 (General error / Operation not permitted): while executing command pnpm approve-builds --yes

✖️ Installation failed in container 1011 (exit code: 1)
📋 Installation log: /tmp/open-archiver-1011-8be70e21.log

Error: General error / Operation not permitted

What would you like to do?

  1. Remove container and exit
  2. Keep container for debugging
  3. Retry with verbose mode (full rebuild)
  4. Retry with DNS override in LXC (8.8.8.8 / 1.1.1.1)

Select option [1-4] (default: 1, auto-remove in 60s):
✔️ Container 1011 removed

Any idea how to get it installed?

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 31, 2026
@github-actions
github-actions Bot deleted the fix/open-archiver-pnpm-build branch June 3, 2026 09:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Open-Archiver]  ELIFECYCLE  Command failed

4 participants