FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add command-line functionality to uuid module · Issue #88597 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

Add command-line functionality to uuid module #88597

Closed
Closed
Add command-line functionality to uuid module#88597
Assignees
Labels
3.12only security fixestype-featureA feature request or enhancement

Description

ephenix
mannequin
opened on Jun 16, 2021
BPO 44431
Nosy @serhiy-storchaka, @erlend-aasland, @ephenix
PRs
  • bpo-44431: Added command line functionality to UUID module. #26751
    		•

    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

    Activity

    1. ephenix commented on Jun 16, 2021

      ephenixmannequin
      on Jun 16, 2021
      MannequinAuthor

      Suggested functionality:

      python -m uuid
      b9aa5a06-e2cd-4c26-b26b-1590f01ea996

    2. ephenix
      added
      3.11only security fixes
      type-featureA feature request or enhancement
      on Jun 16, 2021

    3. serhiy-storchaka commented on Jun 16, 2021

      serhiy-storchaka
      on Jun 16, 2021
      Member

      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.

    4. ezio-melotti
      transferred this issue fromon Apr 10, 2022

    5. achhina commented on Nov 7, 2022

      achhina
      on Nov 7, 2022
      Contributor

      @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.

    6. ephenix commented on Nov 10, 2022

      ephenix
      on Nov 10, 2022
      Author

      @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.

    7. achhina
      added 8 commits that reference this issue on Nov 14, 2022

      pythonGH-88597: Added command line interface to UUID module.

      b99c70a

      pythonGH-88597: Added command line interface to UUID module.

      5a9999c

      pythonGH-88597: Added command line interface to UUID module.

      81c6b06

      pythonGH-88597: Added command line interface to UUID module.

      99df7ef

      pythonGH-88597: Added command line interface to UUID module.

      31edacd

      pythonGH-88597: Added command line interface to UUID module.

      cc423c8

      pythonGH-88597: Added command line interface to UUID module.

      18f698b

      pythonGH-88597: Added command line interface to UUID module.

      a0416f1

    8. 5 remaining items

    9. gpshead
      self-assigned this
      on Jan 22, 2023
    10. gpshead
      added a commit that references this issue on Jan 22, 2023

      GH-88597: Added command line interface to UUID module. (#99463)

      95f5b05

    11. gpshead commented on Jan 22, 2023

      gpshead
      on Jan 22, 2023
      Member

      thanks for the contribution.

    12. gpshead
      closed this as completedon Jan 22, 2023

    13. serhiy-storchaka commented on Jan 22, 2023

      serhiy-storchaka
      on Jan 22, 2023
      Member

      Thank you for your contribution @achhina. But I have few nitpicks:

      1. 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).
      2. 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.
      3. 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.

    14. achhina commented on Jan 22, 2023

      achhina
      on Jan 22, 2023
      Contributor

      @serhiy-storchaka that makes sense, I can create a follow-up patch in regards to the proposed changes if everyone agrees - cc @gpshead.

    15. gpshead commented on Jan 22, 2023

      gpshead
      on Jan 22, 2023
      Member

      Good thinking Serhiy, those changes makes sense to me.

    16. gpshead
      reopened this on Jan 22, 2023

    17. gpshead commented on Jan 22, 2023

      gpshead
      on Jan 22, 2023
      Member

      Just @ me on the PR when you've created one @achhina.

    18. achhina
      added a commit that references this issue on Jan 22, 2023

      pythonGH-88597: Rename uuid cli args and parameters to be in line wit…

      bad954e
    19. gpshead
      added a commit that references this issue on Jan 25, 2023

      GH-88597: Rename uuid's new CLI args to be in line with uuidgen. (#10…

      952a1d9

    20. gpshead commented on Jan 25, 2023

      gpshead
      on Jan 25, 2023
      Member

      thanks for the contribution!

    21. gpshead
      closed this as completedon Jan 25, 2023
    22. iritkatriel
      added a commit that references this issue on Jan 25, 2023

      pythonGH-88597: Rename uuid's new CLI args to be in line with uuidgen. (

      97081c9
    23. mdboom
      added a commit that references this issue on Jan 31, 2023

      pythonGH-88597: Rename uuid's new CLI args to be in line with uuidgen. (

      bf22554
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Metadata

    Metadata

    Assignees

    Labels

    3.12only security fixestype-featureA feature request or enhancement

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      Back | FazBrowse Home | New Git URL