Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-133307: Add new Pdb config location $XDG_CONFIG_HOME/pdb/.pdbrc#133733

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
makukha wants to merge4 commits intopython:main
base:main
Choose a base branch
Loading
frommakukha:133307-pdb-xdg-config-home

Conversation

makukha
Copy link

@makukhamakukha commentedMay 9, 2025
edited by github-actionsbot
Loading

@bedevere-app
Copy link

Most changes to Pythonrequire a NEWS entry. Add one using theblurb_it web app or theblurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply theskip news label instead.

@makukhamakukha changed the titlegh-133307: Added new .pdbrc location, "$XDG_CONFIG_HOME/pdb/.pdbrc"gh-133307: Add new Pdb config location $XDG_CONFIG_HOME/pdb/.pdbrcMay 9, 2025
@gaogaotiantian
Copy link
Member

I'm a -1 on this one. We don't have anything in CPython that's using$XDG_CONFIG_HOME for now. We do have a few cases where we read config files from home dir like IDLE or netrc. This increases the maintenance effort, introduces some inconsistency, without much benefits. I understand that this method stores the user data/config per app instead of per user, but if we really need to do that, we should do it for the whole Python, not just pdb - then it would probably be a PEP-level change.

I think you need to at least get some supports from other people - users from d.p.o. and/or some core devs. That would show that people are interested in this feataure. At this moment I'm not fully convinced that we should have this.

makukha reacted with heart emoji

@makukha
Copy link
Author

@gaogaotiantian thank you for the detailed response! Could you please keep this PR open for a week or two? I'll drop a post in Discourse and get back here.

@gaogaotiantian
Copy link
Member

I won't close this PR. Feel free to gather some opinions and promote this again in the future.

makukha reacted with heart emoji

sharktide added a commit to sharktide/cpython that referenced this pull requestMay 28, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@gaogaotiantiangaogaotiantianAwaiting requested review from gaogaotiantiangaogaotiantian is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@makukha@gaogaotiantian

[8]ページ先頭

©2009-2025 Movatter.jp