gh-148740: Fix uuid CLI with custom UUIDs for UUIDv3/v5 namespaces

[sumpygump]

• This applies to generating version 3 and 5 UUIDs
• Will correctly bail when invalid UUID is supplied as namespace

Example usage with correction:

./python -m uuid --uuid uuid5 --namespace '0d6a16cc-34a7-47d8-b660-214d0ae184d2' --name some.user
a32f4562-2058-5b20-8055-5f38b28285b6

Example error if invalid UUID is supplied as --namespace argument:

./python -m uuid --uuid uuid5 --namespace 'invalid uuid' --name some.user
usage: python -m uuid [-h] [-u {uuid1,uuid3,uuid4,uuid5,uuid6,uuid7,uuid8}] [-n {any UUID,@dns,@url,@oid,@x500}] [-N NAME] [-C NUM]
python -m uuid: error: badly formed hexadecimal UUID string: 'invalid uuid'.

My solution was to update the argparse options to use metavar instead of choices for the namespace
argument. This allows to supply suggestions of options that can be supplied for the --namespace arguments instead of requiring ONLY the ones in the list.

My intent was to keep the help message consistent with how it looked before. Note that it still displays the possible values in the help text for the namespace: -n, --namespace {any UUID,@dns,@url,@oid,@x500}:

./python -m uuid --help
usage: python -m uuid [-h] [-u {uuid1,uuid3,uuid4,uuid5,uuid6,uuid7,uuid8}] [-n {any UUID,@dns,@url,@oid,@x500}] [-N NAME] [-C NUM]

Generate a UUID using the selected UUID function.

options:
  -h, --help            show this help message and exit
  -u, --uuid {uuid1,uuid3,uuid4,uuid5,uuid6,uuid7,uuid8}
                        function to generate the UUID (default: uuid4)
  -n, --namespace {any UUID,@dns,@url,@oid,@x500}
                        uuid3/uuid5 only: a UUID, or a well-known predefined UUID addressed by namespace name (default: None)
  -N, --name NAME       uuid3/uuid5 only: name used as part of generating the UUID (default: None)
  -C, --count NUM       generate NUM fresh UUIDs (default: 1)

• Issue: Fix UUID's CLI for custom namespaces for v3/v5 #148740

[python-cla-bot]

All commit authors signed the Contributor License Agreement.

[picnixz]

• Please update the docs for this.

[picnixz]

Please avoid updating your branch if the CI is green. See our devguide. It wastes CI resources

[picnixz]

I see that there are some uuid3 tests that are put after the uuid4 tests. Can you move all your uuid3 tests before the existing uuid4 tests and update the comment of the uuid3 test about uuid5 format (it is in the existing test)

[sumpygump]

@picnixz I reordered the tests and fixed the comment you noticed. Thanks.

[miss-islington-app]

Thanks @sumpygump for the PR, and @picnixz for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

[miss-islington-app]

Thanks @sumpygump for the PR, and @picnixz for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

[miss-islington-app]

Sorry, @sumpygump and @picnixz, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker f1588d460db97086f5ea28b5797ccdcfffb0307f 3.13

[bedevere-app]

GH-149152 is a backport of this pull request to the 3.14 branch.

[picnixz]

@sumpygump Can you check if 3.13 is affected please and if so do the backport? TiA!

[sumpygump]

@picnixz I checked python 3.13; this is NOT an issue in 3.13.