Skip to content

Conversation

@NMertsch
Copy link
Contributor

@NMertsch NMertsch commented Oct 18, 2025

briefcase dev: Only set PYTHONUNBUFFERED=1, PYTHONDEVMODE=1, and PYTHONUTF8=1 if these variables are not already set.

#2497 is only about PYTHONDEVMODE, but I think this generic implementation is less surprising to users. I might lack context, though.

(I fixed two very minor typos along the way, but don't think this is worth a towncrier fragment.)

Fixes #2497

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

Thanks - this looks great. The only thing it's missing is documentation of the new feature. A new section on the command reference for dev would make sense to me.

@NMertsch
Copy link
Contributor Author

NMertsch commented Oct 20, 2025

Thanks - this looks great. The only thing it's missing is documentation of the new feature. A new section on the command reference for dev would make sense to me.

Thank you. I added the section in commit c97b096, deployed by CI here (once flathub decides to work again). Please let me know if you want any changes (or add them yourself if that feels easier).

Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

Thanks for that - this looks great. Thanks for the PR!

@freakboy3742 freakboy3742 merged commit 3baf43c into beeware:main Oct 21, 2025
57 checks passed
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.

Add ability to disable PYTHONDEVMODE in dev command.

2 participants