fix: avoid rc sourcing loops

Author: aminya

cspell.config.yaml

@@ -30,10 +30,9 @@ words:
   - choco
   - clangd
   - cmake
+  - cmakeformat
   - cmakelang
   - cmakelint
-  - llvmorg
-  - cmakeformat
   - cobertura
   - copr
   - CPATH
@@ -51,6 +50,7 @@ words:
   - dyld
   - eabi
   - envosman
+  - envosmanrc
   - esac
   - esbuild
   - esmodule
@@ -75,6 +75,7 @@ words:
   - libstdc
   - libtinfo
   - liuli
+  - llvmorg
   - mdimporterdir
   - memoizee
   - mkdirp

dist/legacy/setup-cpp.js

@@ -1,6 +1,6 @@
 {
   "name": "envosman",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Manage environment variables, PATH, and rc files",
   "repository": "https://github.com/aminya/setup-cpp",
   "homepage": "https://github.com/aminya/setup-cpp/tree/master/packages/envosman",

dist/legacy/setup-cpp.js.map

@@ -1,12 +1,13 @@
 import { promises } from "fs"
+import { resolve } from "path"
 import { grantUserWriteAccess } from "admina"
 import { info, warning } from "ci-log"
 import memoize from "memoizee"
 import { pathExists } from "path-exists"
 import { untildifyUser } from "untildify-user"
 const { appendFile, readFile, writeFile } = promises
 
-export const defaultRcPath = untildifyUser("~/.bashrc")
+export const defaultRcPath = untildifyUser("~/.envosmanrc")
 
 /**
  * Options for adding an rc file
@@ -20,15 +21,25 @@ export type RcOptions = {
 }
 
 async function sourceRCInRc_(options: RcOptions) {
+  const bashrc = untildifyUser("~/.bashrc")
+  const profile = untildifyUser("~/.profile")
+
+  const rcPath = resolve(options.rcPath)
+
+  // avoid source loops
+  if (rcPath === bashrc || rcPath === profile) {
+    return
+  }
+
   const sourceRcString = options.guard === undefined
-    ? `\nsource "${options.rcPath}"\n`
-    : `\n# ${options.guard}\nif [[ "$SOURCE_${options.guard.toUpperCase()}RC" != 0 && -f "${options.rcPath}" ]]; then source "${options.rcPath}"; fi\n`
+    ? `\nsource "${rcPath}"\n`
+    : `\n# ${options.guard}\nif [[ "$SOURCE_${options.guard.toUpperCase()}RC" != 0 && -f "${rcPath}" ]]; then source "${rcPath}"; fi\n`
 
   try {
     await Promise.all([
       addRCHeader(options),
-      addSourceToTargetRc(sourceRcString, untildifyUser("~/.bashrc")),
-      addSourceToTargetRc(sourceRcString, untildifyUser("~/.profile")),
+      addSourceToTargetRc(sourceRcString, bashrc),
+      addSourceToTargetRc(sourceRcString, profile),
     ])
   } catch (err) {
     warning(`Failed to add ${sourceRcString} to .profile or .bashrc. You should add it manually: ${err}`)