fix: fix issue preventing scripts from being made executable

jayanthkoushik · Jun 20, 2024 · dd3b3d2 · dd3b3d2
1 parent aad1765
commit dd3b3d2
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/dist/pyseed.py b/dist/pyseed.py
@@ -797,7 +797,7 @@ def init_project(config: dict[ConfigKey, Any]):
  for script_path in (project_path / scripts_dir).glob("*"):
  vprint(f"+ CHMOD+x {script_path}", file=sys.stderr)
  os.chmod(
- fpath,
+ script_path,
  stat.S_IRWXU | stat.S_IRGRP | stat.S_IXGRP | stat.S_IROTH | stat.S_IXOTH,
  )
 

diff --git a/src/pyseed.py b/src/pyseed.py
@@ -795,7 +795,7 @@ def init_project(config: dict[ConfigKey, Any]):
  for script_path in (project_path / scripts_dir).glob("*"):
  vprint(f"+ CHMOD+x {script_path}", file=sys.stderr)
  os.chmod(
- fpath,
+ script_path,
  stat.S_IRWXU | stat.S_IRGRP | stat.S_IXGRP | stat.S_IROTH | stat.S_IXOTH,
  )
 

diff --git a/tests/test_pyseed.py b/tests/test_pyseed.py
@@ -782,6 +782,10 @@ def test_init_project_writes_all_expected_files(self):
  self.assertEqual(
  (www_src_dir / "index.md").resolve(), (project_dir / "README.md").resolve()
  )
+ scripts_dir = project_dir / "scripts"
+ for fpath in scripts_dir.glob("*"):
+ with self.subTest(fpath):
+ self.assertTrue(os.access(fpath, os.X_OK))
 
  def test_init_project_writes_all_expected_files_in_barebones_mode(self):
  self.config[pyseed.ConfigKey.barebones] = True