Skip to content

Fix linux beefbuild indefinite hang on some environments.#2471

Open
mdsitton wants to merge 1 commit into
beefytech:masterfrom
mdsitton:linux-hang
Open

Fix linux beefbuild indefinite hang on some environments.#2471
mdsitton wants to merge 1 commit into
beefytech:masterfrom
mdsitton:linux-hang

Conversation

@mdsitton

Copy link
Copy Markdown
Contributor

I noticed that beefbuild when installed into a system location that is inaccessible to writes runs into a permission issue with builds since the previous behavior was to create the BeefManaged folder relative to the bin/ location instead of the location the ide currently does which is in ~/.config/beeflang/BeefManaged

So with that said this PR makes the following changes:
Initialize the packaged Linux user data directory the same way for BeefIDE and BeefBuild so that they both store managed libraries under the user's config directory instead of the installation directory.

I've also made some additional changes to surface errors like this quicker in the future instead of having silently failing build command that seems to never exit.

As such now if there is any similar permissions issues in the future it will output something like this and exit instead of the hang:

$ beefbuild
ERROR: Failed to create managed library directory '/opt/BeefLang/bin/BeefManaged'
ERROR: Failed to initialize managed library support for project 'SDL3-Beef.git'

Initialize the packaged Linux user data directory for both BeefIDE and the CLI so BeefBuild stores managed libraries under the user's config directory instead of the installation directory.

Report managed cache directory creation failures and propagate package manager initialization errors back through project loading. This prevents projects from being marked deferred when no package work was queued, which previously left BeefBuild polling indefinitely.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant