GeezerVison is showing github.com — Add command-line functionality to uuid module · Issue #88597 · python/cpython
Add command-line functionality to uuid module #88597Description
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 = None
closed_at = None
created_at = <Date 2021-06-16.04:14:46.410>
labels = ['type-feature', '3.11']
title = 'Add command-line functionality to uuid module'
updated_at = <Date 2021-06-16.22:06:09.458>
user = 'https://github.com/ephenix' bugs.python.org fields:
activity = <Date 2021-06-16.22:06:09.458>
actor = 'erlendaasland'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = []
creation = <Date 2021-06-16.04:14:46.410>
creator = 'ephenix'
dependencies = []
files = []
hgrepos = []
issue_num = 44431
keywords = ['patch']
message_count = 2.0
messages = ['395906', '395967']
nosy_count = 3.0
nosy_names = ['serhiy.storchaka', 'erlendaasland', 'ephenix']
pr_nums = ['26751']
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue44431'
versions = ['Python 3.11'] Linked PRs
Activityephenix commented on Jun 16, 2021 MannequinAuthor More actions Suggested functionality:
python -m uuid
b9aa5a06-e2cd-4c26-b26b-1590f01ea996
serhiy-storchaka commented on Jun 16, 2021 I think it would be a useful feature.
But would not be better to make interface similar to interface of standard Linux command uuidgen? In any case it should support the -h or --help options. achhina commented on Nov 7, 2022 @ephenix would you still like to work on this as I see you've closed your PR? If not, I'd like to work on this; I'll wait a couple of days to give you a chance to respond if you'd still like to continue. ephenix commented on Nov 10, 2022
@ephenix would you still like to work on this as I see you've closed your PR? If not, I'd like to work on this; I'll wait a couple of days to give you a chance to respond if you'd still like to continue.
I lost track of this -- feel free to take my pull request and do anything necessary to get it merged. IIRC the changes I made were working. gpshead commented on Jan 22, 2023 thanks for the contribution. serhiy-storchaka commented on Jan 22, 2023 Thank you for your contribution @achhina. But I have few nitpicks:
- Most CLI tools on Unix and Linux use minus followed by a single letter as short options, and double minus followed by a sequence of letters as long option. An alternative style used in minority of commands (e.g.
find and java) use single minus for long options. But your proposition mixes the styles: there are short options like -u, long options like --name, and also long option with a single minus -ns. argparse of course supports both styles, but I was surprised that it supports a mix, and I am not sure that it will not be broken in future (-ns can be interpreted as two separate options -n and -s).
- Standard Linux utility
uuidgen has similar interface, but conflicting short options. -n is a short alias of --namespace, and -N is a short alias of --name. As you see option -n has different semantic.
uuidgen accepts namespace argument in form: @dns @url @oid @x500. In your proposition it should be in form: NAMESPACE_DNS NAMESPACE_URL NAMESPACE_OID NAMESPACE_X500. And it is not documented.
I propose to rename -n to -N and -ns to -n, and accept simpler compatible namespace syntax. Reacted by Éric and Erlend E. Aasland achhina commented on Jan 22, 2023 @serhiy-storchaka that makes sense, I can create a follow-up patch in regards to the proposed changes if everyone agrees - cc @gpshead. Reacted by Erlend E. Aasland gpshead commented on Jan 22, 2023 Good thinking Serhiy, those changes makes sense to me. gpshead commented on Jan 22, 2023 Just @ me on the PR when you've created one @achhina. gpshead commented on Jan 25, 2023 thanks for the contribution!
MetadataMetadataDevelopmentNo branches or pull requestsIssue actions
|