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

patch --commit on github dependency SEGV #17982

Open
psethwick opened this issue Mar 7, 2025 · 0 comments
Open

patch --commit on github dependency SEGV #17982

psethwick opened this issue Mar 7, 2025 · 0 comments
Labels
bun install Something that relates to the npm-compatible client bun patch confirmed bug We can reproduce this issue crash An issue that could cause a crash

Comments

@psethwick
Copy link

psethwick commented Mar 7, 2025

How can we reproduce the crash?

bun init
bun install git+https://github.com/expressjs/express # any package works as long as it's a git+http dep
bun patch express
touch node_modules/express/bun.patch
bun patch --commit 'node_modules/express'

Relevant log output

❯ bun patch --commit 'node_modules/express'
bun patch v1.2.4 (fd9a5ea6)
============================================================
Bun v1.2.4 (fd9a5ea6) Linux x64
Linux Kernel v6.13.5 | glibc v2.41
CPU: sse42 popcnt avx avx2 avx512
Args: "bun" "patch" "--commit" "node_modules/express"
Features: text_lockfile
Elapsed: 2ms | User: 0ms | Sys: 3ms
RSS: 1.07GB | Peak: 17.56MB | Commit: 1.07GB | Faults: 0

panic(main thread): Segmentation fault at address 0x756
oh no: Bun has crashed. This indicates a bug in Bun, not your code.

Stack Trace (bun.report)

Bun v1.2.4 (fd9a5ea) on linux x86_64 [PatchCommand]

Segmentation fault at address 0x00007BBD

Features: text_lockfile

Sentry Issue: BUN-DWG

@psethwick psethwick added the crash An issue that could cause a crash label Mar 7, 2025
@github-actions github-actions bot added the linux An issue that occurs on Linux label Mar 7, 2025
@psethwick psethwick changed the title patch on github dependency SEGV patch --commit on github dependency SEGV Mar 7, 2025
@RiskyMH RiskyMH added bun install Something that relates to the npm-compatible client confirmed bug We can reproduce this issue and removed linux An issue that occurs on Linux labels Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bun install Something that relates to the npm-compatible client bun patch confirmed bug We can reproduce this issue crash An issue that could cause a crash
Projects
None yet
Development

No branches or pull requests

3 participants