diff --git a/Lib/pdb.py b/Lib/pdb.py index 7b08d2bb70183d..772e28d21849cd 100644 --- a/Lib/pdb.py +++ b/Lib/pdb.py @@ -3634,6 +3634,10 @@ def parse_args(): opt_module = parser.parse_args(args[:2]) opts.module = opt_module.module args = args[2:] + elif args[0] == '--': + args.pop(0) + if not args: + parser.error("missing script or module to run") elif args[0].startswith('-'): # Invalid argument before the script name. invalid_args = list(itertools.takewhile(lambda a: a.startswith('-'), args)) diff --git a/Lib/test/test_pdb.py b/Lib/test/test_pdb.py index 0e23cd6604379c..5eadfa6ed2cb88 100644 --- a/Lib/test/test_pdb.py +++ b/Lib/test/test_pdb.py @@ -4173,6 +4173,20 @@ def test_run_script_with_args(self): stdout, stderr = self.run_pdb_script(script, commands, script_args=["--bar", "foo"]) self.assertIn("['--bar', 'foo']", stdout) + def test_run_script_with_double_dash(self): + script = "import sys; print(sys.argv)" + commands = "continue\nquit" + filename = 'main.py' + with open(filename, 'w') as f: + f.write(script) + self.addCleanup(os_helper.unlink, filename) + stdout, _ = self._run_pdb(["--", filename, "-c", "example"], commands) + self.assertIn(f"['{filename}', '-c', 'example']", stdout) + stdout, _ = self._run_pdb(["-c", "continue", "--", filename, "-c", "example"], "quit") + self.assertIn(f"['{filename}', '-c', 'example']", stdout) + stdout, stderr = self._run_pdb(["--"], "", expected_returncode=2) + self.assertIn("pdb: error: missing script or module to run", stderr) + def test_breakpoint(self): script = """ if __name__ == '__main__': @@ -4752,6 +4766,26 @@ def foo(self): stdout, stderr = self.run_pdb_script(script, commands) self.assertIn("The specified object 'C.foo' is not a function", stdout) + def test_end_of_options_separator(self): + # gh-148615: Test parsing when '--' separator is used + script = "import sys; print(f'ARGS: {sys.argv[1:]}')" + with open(os_helper.TESTFN, 'w', encoding='utf-8') as f: + f.write(script) + stdout, _ = self._run_pdb(['--', os_helper.TESTFN, '-foo'], 'c\nq') + self.assertIn("ARGS: ['-foo']", stdout) + stdout, _ = self._run_pdb(['-c', 'continue', '--', os_helper.TESTFN, '-c', 'foo'], 'q') + self.assertIn("ARGS: ['-c', 'foo']", stdout) + stdout, stderr = self._run_pdb(['--'], 'q', expected_returncode=2) + self.assertIn("missing script or module to run", stderr) + stdout, stderr = self._run_pdb(['-x', '--', os_helper.TESTFN], 'q', expected_returncode=2) + self.assertIn("unrecognized arguments: -x", stderr) + stdout, _ = self._run_pdb([os_helper.TESTFN, '--', 'arg'], 'c\nq') + self.assertIn("ARGS: ['--', 'arg']", stdout) + with os_helper.temp_cwd(): + with open('mymod.py', 'w', encoding='utf-8') as f: + f.write(script) + stdout, _ = self._run_pdb(['-m', 'mymod', '--', 'arg'], 'c\nq') + self.assertIn("ARGS: ['--', 'arg']", stdout) class ChecklineTests(unittest.TestCase): def setUp(self): diff --git a/Misc/NEWS.d/next/Library/2026-04-15-16-08-12.gh-issue-148615.Uvx50R.rst b/Misc/NEWS.d/next/Library/2026-04-15-16-08-12.gh-issue-148615.Uvx50R.rst new file mode 100644 index 00000000000000..f023f0141889a6 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2026-04-15-16-08-12.gh-issue-148615.Uvx50R.rst @@ -0,0 +1 @@ +Fix :mod:`pdb` to accept standard -- end of options separator. Reported by haampie. Patched by Shrey Naithani.