Skip to content

Commit 2c3f729

Browse files
authored
Expand info_commands in setup.py
Expanding our list of info commands to include those needed for the ticket. The only command provided that won't work is a blank python setup.py. If they use a source distributed that hasn't had the version changed from develop, there will also be a warning. The rest of the info output looks good.
1 parent 1576f06 commit 2c3f729

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

setup.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,15 @@ def is_info_command():
6262
Check if we're running a command that only needs package metadata
6363
and doesn't require Flux to be installed.
6464
"""
65-
info_commands = {'egg_info', 'sdist', '--help', '--help-commands', '--version'}
65+
info_commands = {
66+
"egg_info",
67+
"sdist",
68+
"--name",
69+
"--description",
70+
"--help",
71+
"--help-commands",
72+
"--version",
73+
}
6674
return any(cmd in sys.argv for cmd in info_commands)
6775

6876

@@ -348,7 +356,6 @@ def setup():
348356
global has_flux_security
349357

350358
if is_info_command():
351-
print("Running info command, skipping Flux detection and header preparation")
352359
cffi_modules = None
353360
else:
354361
# Always set the install root to the environment

0 commit comments

Comments
 (0)