[python] don't attempt systemdep install if no deps

vlinkz · vlinkz · commit 9eb810408d95 · 2025-06-16T10:12:05.000-07:00
diff --git a/internal/backends/python/python.go b/internal/backends/python/python.go
@@ -333,6 +333,8 @@ func commonInstallDotReplitNixDeps(ctx context.Context, pkgs []api.PkgName, spec
 			}
 		}
 	}
+	
+	originalLen := len(allPkgs)
 
 	for _, pkg := range pkgs {
 		deps := nix.PythonNixDeps(string(pkg))
@@ -350,11 +352,17 @@ func commonInstallDotReplitNixDeps(ctx context.Context, pkgs []api.PkgName, spec
 		}
 	}
 
+	if len(allPkgs) == originalLen {
+		// No new packages were added, so we can skip writing to .replit
+		return
+	}
+
 	keys := []string{}
 	for key := range allPkgs {
 		keys = append(keys, key)
 	}
 	slices.Sort(keys)
+
 	value, err := json.Marshal(keys)
 	if err != nil {
 		util.DieSubprocess("failed to marshal JSON: %s", err)

Original file line number	Diff line number	Diff line change
`@@ -333,6 +333,8 @@ func commonInstallDotReplitNixDeps(ctx context.Context, pkgs []api.PkgName, spec`
`333`	`333`	`}`
`334`	`334`	`}`
`335`	`335`	`}`
	`336`	`+`
	`337`	`+ originalLen := len(allPkgs)`
`336`	`338`
`337`	`339`	`for _, pkg := range pkgs {`
`338`	`340`	`deps := nix.PythonNixDeps(string(pkg))`
`@@ -350,11 +352,17 @@ func commonInstallDotReplitNixDeps(ctx context.Context, pkgs []api.PkgName, spec`
`350`	`352`	`}`
`351`	`353`	`}`
`352`	`354`
	`355`	`+ if len(allPkgs) == originalLen {`
	`356`	`+ // No new packages were added, so we can skip writing to .replit`
	`357`	`+ return`
	`358`	`+ }`
	`359`	`+`
`353`	`360`	`keys := []string{}`
`354`	`361`	`for key := range allPkgs {`
`355`	`362`	`keys = append(keys, key)`
`356`	`363`	`}`
`357`	`364`	`slices.Sort(keys)`
	`365`	`+`
`358`	`366`	`value, err := json.Marshal(keys)`
`359`	`367`	`if err != nil {`
`360`	`368`	`util.DieSubprocess("failed to marshal JSON: %s", err)`