From 89b0fd96280264ffd7ec4afe8f1ea5ec32dd513c Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Sun, 19 Mar 2023 12:42:22 +0100
Subject: [PATCH] Remove the remaining `PRODUCTION` build-target usage

After the previous patch we now have only *a single* `PRODUCTION` occurrence in the entire code-base, more specifically in the `web/viewer.html` file.
This special build-target can be replaced with any condition that always evaluate to `false`, such as e.g. a comment.

*Please note:* This patch might be considered too hacky, hence I completely understand if it's rejected.
---
 gulpfile.js     | 1 -
 web/viewer.html | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/gulpfile.js b/gulpfile.js
index 116b33ab7..65149ab38 100644
--- a/gulpfile.js
+++ b/gulpfile.js
@@ -96,7 +96,6 @@ const AUTOPREFIXER_CONFIG = {
 const BABEL_TARGETS = ENV_TARGETS.join(", ");
 
 const DEFINES = Object.freeze({
-  PRODUCTION: true,
   SKIP_BABEL: true,
   TESTING: undefined,
   // The main build targets:
diff --git a/web/viewer.html b/web/viewer.html
index 4c0583c67..4845807c3 100644
--- a/web/viewer.html
+++ b/web/viewer.html
@@ -49,7 +49,7 @@ See https://github.com/adobe-type-tools/cmap-resources
 <!--<script src="resource://pdf.js/web/viewer.js"></script>-->
 <!--#elif !MINIFIED-->
 <!--<script src="viewer.js"></script>-->
-<!--#elif !PRODUCTION-->
+<!--#elif /* Development mode. */-->
     <link rel="resource" type="application/l10n" href="locale/locale.properties">
 
     <script defer src="../node_modules/es-module-shims/dist/es-module-shims.js"></script>