Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify command option before parsing config file #55270

Closed
sdouche mannequin opened this issue Jan 29, 2011 · 4 comments
Closed

Verify command option before parsing config file #55270

sdouche mannequin opened this issue Jan 29, 2011 · 4 comments
Assignees
Labels
performance Performance or resource usage

Comments

@sdouche
Copy link
Mannequin

sdouche mannequin commented Jan 29, 2011

BPO 11061
Nosy @tarekziade, @merwok

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = 'https://github.com/tarekziade'
closed_at = <Date 2011-02-04.17:26:48.582>
created_at = <Date 2011-01-29.15:20:54.382>
labels = ['performance']
title = 'Verify command option before parsing config file'
updated_at = <Date 2011-02-04.17:28:57.052>
user = 'https://bugs.python.org/sdouche'

bugs.python.org fields:

activity = <Date 2011-02-04.17:28:57.052>
actor = 'eric.araujo'
assignee = 'tarek'
closed = True
closed_date = <Date 2011-02-04.17:26:48.582>
closer = 'eric.araujo'
components = ['Distutils2']
creation = <Date 2011-01-29.15:20:54.382>
creator = 'sdouche'
dependencies = []
files = []
hgrepos = []
issue_num = 11061
keywords = []
message_count = 4.0
messages = ['127432', '127456', '127584', '127617']
nosy_count = 3.0
nosy_names = ['tarek', 'eric.araujo', 'sdouche']
pr_nums = []
priority = 'normal'
resolution = 'rejected'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'performance'
url = 'https://bugs.python.org/issue11061'
versions = ['3rd party']

@sdouche
Copy link
Mannequin Author

sdouche mannequin commented Jan 29, 2011

Distutils2 verify command (like sdist) when he use it. A small optimization is add the verification step at launch.

@sdouche sdouche mannequin added the type-bug An unexpected behavior, bug, or error label Jan 29, 2011
@sdouche sdouche mannequin assigned tarekziade Jan 29, 2011
@merwok
Copy link
Member

merwok commented Jan 29, 2011

I’m afraid I don’t understand your report.

@sdouche
Copy link
Mannequin Author

sdouche mannequin commented Jan 31, 2011

Oups sorry Eric. I created the report very quickly after a discussion with Tarek. If you type a wrong command like "python -m distutils2:run wrong", disutils2 still parses the setup.cfg. How resolve this? Don't know, maybe a migration to optparse.

Anyway, close the bug if you think it's irrevelant.

@merwok
Copy link
Member

merwok commented Jan 31, 2011

Well, the setup.cfg could register a command named “wrong”, so d2 really has to parse all config files before rejecting a command.

@merwok merwok closed this as completed Feb 4, 2011
@merwok merwok added performance Performance or resource usage and removed type-bug An unexpected behavior, bug, or error labels Feb 4, 2011
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance or resource usage
Projects
None yet
Development

No branches or pull requests

1 participant