Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove the remaining PRODUCTION build-target usage

Browse Source 
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.
This commit is contained in:
Jonas Jenwald 2023-03-19 12:42:22 +01:00
parent 804aa896a7
commit 89b0fd9628
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
gulpfile.js
View File

@ -96,7 +96,6 @@ const AUTOPREFIXER_CONFIG = {
const BABEL_TARGETS = ENV_TARGETS.join(", "); const BABEL_TARGETS = ENV_TARGETS.join(", ");
const DEFINES = Object.freeze({ const DEFINES = Object.freeze({
PRODUCTION: true,
SKIP_BABEL: true, SKIP_BABEL: true,
TESTING: undefined, TESTING: undefined,
// The main build targets: // The main build targets:

2
web/viewer.html
View File

@ -49,7 +49,7 @@ See https://github.com/adobe-type-tools/cmap-resources
<!--<script src="resource://pdf.js/web/viewer.js"></script>--> <!--<script src="resource://pdf.js/web/viewer.js"></script>-->
<!--#elif !MINIFIED--> <!--#elif !MINIFIED-->
<!--<script src="viewer.js"></script>--> <!--<script src="viewer.js"></script>-->
<!--#elif !PRODUCTION--> <!--#elif /* Development mode. */-->
<link rel="resource" type="application/l10n" href="locale/locale.properties"> <link rel="resource" type="application/l10n" href="locale/locale.properties">
<script defer src="../node_modules/es-module-shims/dist/es-module-shims.js"></script> <script defer src="../node_modules/es-module-shims/dist/es-module-shims.js"></script>