GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Support :python option in compile-time config by AJReade · Pull Request #48 · livebook-dev/pythonx

Conversation

@AJReade

@AJReade AJReade commented May 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The :python option (e.g., "3.14t" for free-threaded builds) set via config :pythonx, :uv_init is read from Application.compile_env but never forwarded to Pythonx.Uv.fetch/3 or Pythonx.Uv.init/3
  • This causes uv sync to download the regular Python build instead of the requested variant

Context

PR #45 added the :python option to Pythonx.uv_init/2 (the runtime function), which works correctly. However, the compile-time config path in application.ex was not updated to forward this option. Users following the recommended Mix project setup via config :pythonx, :uv_init silently get the wrong Python variant.

Change

One line in lib/pythonx/application.ex — read :python from the config and include it in the opts passed to fetch/3 and init/3:

# Before
opts = [uv_version: uv_version]

# After
python = uv_init_env[:python]
opts = [uv_version: uv_version, python: python]

Verification

Tested on macOS aarch64. Before the fix, config :pythonx, :uv_init, python: "3.14t" downloads cpython-3.14.3-macos-aarch64-none (regular). After the fix, it downloads cpython-3.14.3+freethreaded-macos-aarch64-none and confirms GIL disabled.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@AJReade
fix: pass :python option from config to uv fetch/init
528c7c3
The :python option (e.g., "3.14t" for free-threaded builds) was read
from Application.compile_env but never forwarded to Pythonx.Uv.fetch/3
or Pythonx.Uv.init/3. This caused uv to download the regular Python
build instead of the free-threaded variant.
jonatanklosko
jonatanklosko approved these changes May 18, 2026
View reviewed changes

@jonatanklosko jonatanklosko left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call, thank you :)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@jonatanklosko jonatanklosko changed the title fix: forward :python option from compile-time config to uv fetch/init Support :python option in compile-time config May 18, 2026
@jonatanklosko
jonatanklosko merged commit c45963b into livebook-dev:main May 18, 2026
9 checks passed

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonatanklosko jonatanklosko jonatanklosko approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@AJReade @jonatanklosko
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.