warn about $PATH correctly

mxcl · mxcl · commit 27629915532d · 2025-03-26T13:22:31.000-04:00
diff --git a/pkgm.ts b/pkgm.ts
@@ -189,8 +189,16 @@ async function install(args: string[], basePath: string) {
     }
   }
 
-  if (!Deno.env.get("PATH")?.split(":")?.includes(basePath)) {
-    console.error("%c! warning:", "color:yellow", `${basePath} not in $PATH`);
+  if (
+    !Deno.env.get("PATH")?.split(":")?.includes(
+      new Path(basePath).join("bin").string,
+    )
+  ) {
+    console.error(
+      "%c! warning:",
+      "color:yellow",
+      `${new Path(basePath).join("bin")} not in $PATH`,
+    );
   }
 
   return rv;

Original file line number	Diff line number	Diff line change
`@@ -189,8 +189,16 @@ async function install(args: string[], basePath: string) {`
`189`	`189`	`}`
`190`	`190`	`}`
`191`	`191`
`192`		`- if (!Deno.env.get("PATH")?.split(":")?.includes(basePath)) {`
`193`		- console.error("%c! warning:", "color:yellow", `${basePath} not in $PATH`);
	`192`	`+ if (`
	`193`	`+ !Deno.env.get("PATH")?.split(":")?.includes(`
	`194`	`+ new Path(basePath).join("bin").string,`
	`195`	`+ )`
	`196`	`+ ) {`
	`197`	`+ console.error(`
	`198`	`+ "%c! warning:",`
	`199`	`+ "color:yellow",`
	`200`	+ `${new Path(basePath).join("bin")} not in $PATH`,
	`201`	`+ );`
`194`	`202`	`}`
`195`	`203`
`196`	`204`	`return rv;`