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-110964: clinic: pass clinic argument to bad_argument()#110984

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

Closed
vstinner wants to merge1 commit intopython:mainfromvstinner:clinic_bad_arg

Conversation

vstinner
Copy link
Member

@vstinnervstinner commentedOct 17, 2023
edited by bedevere-appbot
Loading

Don't rely on the global 'clinic' argument: pass explicitly a 'clinic' argument.

@vstinner
Copy link
MemberAuthor

The only purpose of this change is to avoid usingclinic global variable indef bad_argument() method. Just for that, I had to modify many lines :-(

@vstinner
Copy link
MemberAuthor

cc@AlexWaygood

@AlexWaygood
Copy link
Member

AlexWaygood commentedOct 17, 2023
edited
Loading

Please give me some time to look at this! :-)

@vstinnervstinner requested review froma team andgpshead ascode ownersOctober 17, 2023 22:11
@vstinner
Copy link
MemberAuthor

"Check if generated files are up to date (pull_request)" failure looks unrelated:

./configure --config-cache --with-pydebug --enable-shared    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.11.6/x64/lib/pkgconfigconfigure: loading cache config.cacheconfigure: error: `PKG_CONFIG_PATH' has changed since the previous run:configure:   former value:  `/opt/hostedtoolcache/Python/3.11.5/x64/lib/pkgconfig'configure:   current value: `/opt/hostedtoolcache/Python/3.11.6/x64/lib/pkgconfig'

Don't rely on the global 'clinic' argument: pass explicitly a'clinic' argument.
@vstinner
Copy link
MemberAuthor

I rebased my PR on the main branch to try to fix the "Check if generated files are up to date" job failure.

@vstinner
Copy link
MemberAuthor

Please give me some time to look at this! :-)

Mmmmh, so do you need more time to review?

@vstinner
Copy link
MemberAuthor

Well, since@AlexWaygood is not available for review, i think that i will just merge my change next days.

@AlexWaygood
Copy link
Member

AlexWaygood commentedNov 3, 2023
edited
Loading

Thanks for waiting, sorry for the slow response from me! I wanted to take the time to think about this properly, but haven't had a chance recently due to being busy at work, and it's unlikely that I'll have the time in the next week either.

My instinct is that there should be a simpler solution here that involves fewer changes, but if you feel like there's a rush to get this merged for whatever reason, then please go ahead. I agree I've kept you waiting for a while :-)

@vstinner
Copy link
MemberAuthor

me:

The only purpose of this change is to avoid using clinic global variable in def bad_argument() method. Just for that, I had to modify many lines :-(

@AlexWaygood:

My instinct is that there should be q simpler solution here that involves fewer changes,

Yeah, there should be a way to redesign the code to have to pass less arguments. I dislike the current API :-( Moreover, passing limited_capi and clinic is kind of redundant, since usingclinic.limited_capi is the same as passinglimited_capi.

I close my PR for now. We can revisit this code later when we will try to give rid of the globalclinic variable.

AlexWaygood and erlend-aasland reacted with thumbs up emoji

@vstinnervstinner deleted the clinic_bad_arg branchNovember 3, 2023 11:22
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@erlend-aaslanderlend-aaslandAwaiting requested review from erlend-aaslanderlend-aasland is a code owner

@AlexWaygoodAlexWaygoodAwaiting requested review from AlexWaygood

@gpsheadgpsheadAwaiting requested review from gpsheadgpshead 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
@vstinner@AlexWaygood

[8]ページ先頭

©2009-2025 Movatter.jp