From c74f5b858627562e52b1182df7fe134468a40550 Mon Sep 17 00:00:00 2001
From: Frederick Wagner <frederick@frederickwagner.net>
Date: Wed, 4 May 2022 10:51:11 -0400
Subject: [PATCH] Don't mention .env file if LoadDotenv isn't enabled

---
 README.md                 | 13 +++++--------
 internal/cmd/cmd_allow.go |  5 ++++-
 internal/cmd/cmd_deny.go  |  5 ++++-
 man/direnv.toml.1         |  2 +-
 man/direnv.toml.1.md      |  2 +-
 5 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/README.md b/README.md
index 0cc8ceeef..3be7fb276 100644
--- a/README.md
+++ b/README.md
@@ -18,14 +18,11 @@ current directory.
 
 ## How it works
 
-Before each prompt, direnv checks for the existence of a `.envrc` or `.env`
-file in the current and parent directories. If the file exists (and is
-authorized), it is loaded into a **bash** sub-shell and all exported
-variables are then captured by direnv and then made available to the current
-shell.
-
-If both `.envrc` and `.env` files exists, the `.envrc` will always be chosen
-first.
+Before each prompt, direnv checks for the existence of a `.envrc` file (and
+[optionally](man/direnv.toml.1.md#load_dotenv) a `.env` file) in the current
+and parent directories. If the file exists (and is authorized), it is loaded
+into a **bash** sub-shell and all exported variables are then captured by
+direnv and then made available to the current shell.
 
 It supports hooks for all the common shells like bash, zsh, tcsh and fish.
 This allows project-specific environment variables without cluttering the
diff --git a/internal/cmd/cmd_allow.go b/internal/cmd/cmd_allow.go
index 4bfee5605..d7cf255a4 100644
--- a/internal/cmd/cmd_allow.go
+++ b/internal/cmd/cmd_allow.go
@@ -65,7 +65,10 @@ func cmdAllowAction(env Env, args []string, config *Config) (err error) {
 	if err != nil {
 		return err
 	} else if rc == nil {
-		return fmt.Errorf(".envrc or .env file not found")
+		if config.LoadDotenv {
+			return fmt.Errorf(".envrc or .env file not found")
+		}
+		return fmt.Errorf(".envrc file not found")
 	}
 	return rc.Allow()
 }
diff --git a/internal/cmd/cmd_deny.go b/internal/cmd/cmd_deny.go
index 15409a188..9630bba37 100644
--- a/internal/cmd/cmd_deny.go
+++ b/internal/cmd/cmd_deny.go
@@ -35,7 +35,10 @@ func cmdDenyAction(env Env, args []string, config *Config) (err error) {
 	if err != nil {
 		return err
 	} else if rc == nil {
-		return fmt.Errorf(".envrc or .env file not found")
+		if config.LoadDotenv {
+			return fmt.Errorf(".envrc or .env file not found")
+		}
+		return fmt.Errorf(".envrc file not found")
 	}
 	return rc.Deny()
 }
diff --git a/man/direnv.toml.1 b/man/direnv.toml.1
index 065ea2d11..22bbc56b8 100644
--- a/man/direnv.toml.1
+++ b/man/direnv.toml.1
@@ -54,7 +54,7 @@ If set to \fB\fCtrue\fR, stdin is disabled (redirected to /dev/null) during the
 
 .SS \fB\fCload_dotenv\fR
 .PP
-Also look for and load \fB\fC\&.env\fR files on top of the \fB\fC\&.envrc\fR files.
+Also look for and load \fB\fC\&.env\fR files on top of the \fB\fC\&.envrc\fR files. If both \fB\fC\&.envrc\fR and \fB\fC\&.env\fR files exist, the \fB\fC\&.envrc\fR will always be chosen first.
 
 .SS \fB\fCstrict_env\fR
 .PP
diff --git a/man/direnv.toml.1.md b/man/direnv.toml.1.md
index 72c6dc8f6..05dcae490 100644
--- a/man/direnv.toml.1.md
+++ b/man/direnv.toml.1.md
@@ -44,7 +44,7 @@ If set to `true`, stdin is disabled (redirected to /dev/null) during the `.envrc
 
 ### `load_dotenv`
 
-Also look for and load `.env` files on top of the `.envrc` files.
+Also look for and load `.env` files on top of the `.envrc` files. If both `.envrc` and `.env` files exist, the `.envrc` will always be chosen first.
 
 ### `strict_env`