From d2a1909468ee9640bc0c73ff73070e7d0878dbc5 Mon Sep 17 00:00:00 2001 From: christophe dervieux Date: Wed, 29 Apr 2026 15:56:42 +0200 Subject: [PATCH] Fix configure warning showing binary path instead of directory for PATH The "Please ensure that is in your path" warnings emitted when the configure script cannot create a symlink were showing the path to the quarto binary itself, but PATH expects a directory. Use config.directoryInfo.bin so the warning reports the directory users should add to PATH. Closes #14426 --- package/src/common/configure.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/package/src/common/configure.ts b/package/src/common/configure.ts index a40b185fa8..c5999d98e3 100644 --- a/package/src/common/configure.ts +++ b/package/src/common/configure.ts @@ -126,9 +126,7 @@ export async function configure( info(`> Didn't create symlink at ${symlinkPath}`); if (i === symlinksFiltered.length - 1) { warning( - `\n> Please ensure that ${ - join(config.directoryInfo.bin, "quarto") - } is in your path.`, + `\n> Please ensure that ${config.directoryInfo.bin} is in your path.`, ); } } @@ -136,9 +134,7 @@ export async function configure( } else { // Just warn the user and create a symlink in our last resort warning( - `\n> Please ensure that ${ - join(config.directoryInfo.bin, "quarto") - } is in your path.`, + `\n> Please ensure that ${config.directoryInfo.bin} is in your path.`, ); } }