[taler-wallet-core] branch master updated: build system: get rid of prepare scripts

[gnunet]

This is an automated email from the git hooks/post-receive script.

dold pushed a commit to branch master
in repository wallet-core.

The following commit(s) were added to refs/heads/master by this push:
     new f56a56353 build system: get rid of prepare scripts
f56a56353 is described below

commit f56a563533e28b27affc098ccff77ae4cc60e509
Author: Florian Dold <florian@dold.me>
AuthorDate: Tue Jun 6 15:25:53 2023 +0200

    build system: get rid of prepare scripts
    
    These don't seem to be the right location to compile packages.
    The newest version of pnpm runs all those scripts on "pnpm install",
    even if we filter for only one package.  That results in a build
    performance issue.
---
 Makefile                                        | 4 ++--
 packages/anastasis-core/package.json            | 1 -
 packages/anastasis-webui/package.json           | 1 -
 packages/idb-bridge/package.json                | 1 -
 packages/merchant-backend-ui/package.json       | 1 -
 packages/pogen/package.json                     | 1 -
 packages/taler-util/package.json                | 1 -
 packages/taler-wallet-cli/package.json          | 1 -
 packages/taler-wallet-core/package.json         | 1 -
 packages/taler-wallet-webextension/package.json | 1 -
 packages/web-util/package.json                  | 1 -
 11 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/Makefile b/Makefile
index 1a23b0a64..20e8d886c 100644
--- a/Makefile
+++ b/Makefile
@@ -106,7 +106,7 @@ webextension-dev:
 
 .PHONY: embedded
 embedded:
-       pnpm install --frozen-lockfile --ignore-scripts --filter 
@gnu-taler/taler-wallet-embedded...
+       pnpm install --frozen-lockfile --filter 
@gnu-taler/taler-wallet-embedded...
        pnpm run --filter @gnu-taler/taler-wallet-embedded... compile
        @echo built 
packages/taler-wallet-embedded/dist/taler-wallet-core-qjs.mjs
 
@@ -120,7 +120,7 @@ install: compile
 
 # Install taler-wallet-cli and taler-harness
 install-tools:
-       pnpm install --frozen-lockfile --ignore-scripts --filter 
@gnu-taler/taler-wallet-cli... --filter @gnu-taler/taler-harness...
+       pnpm install --frozen-lockfile --filter @gnu-taler/taler-wallet-cli... 
--filter @gnu-taler/taler-harness...
        pnpm run --filter @gnu-taler/taler-wallet-cli... --filter 
@gnu-taler/taler-harness... compile
        make -C packages/taler-wallet-cli install-nodeps
        make -C packages/taler-harness install-nodeps
diff --git a/packages/anastasis-core/package.json 
b/packages/anastasis-core/package.json
index 62a9b0bde..92957b885 100644
--- a/packages/anastasis-core/package.json
+++ b/packages/anastasis-core/package.json
@@ -6,7 +6,6 @@
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "scripts": {
-    "prepare": "tsc",
     "compile": "tsc",
     "pretty": "prettier --write src",
     "test": "tsc && ava",
diff --git a/packages/anastasis-webui/package.json 
b/packages/anastasis-webui/package.json
index afde50fed..eff5ba04d 100644
--- a/packages/anastasis-webui/package.json
+++ b/packages/anastasis-webui/package.json
@@ -8,7 +8,6 @@
     "build": "./build.mjs",
     "compile": "tsc && ./build.mjs",
     "dev": "./dev.mjs",
-    "prepare": "pnpm compile",
     "lint": "eslint 'src/**/*.{js,jsx,ts,tsx}'",
     "test": "./test.mjs && mocha --require source-map-support/register 
--enable-source-maps 'dist/**/*test.js'",
     "pretty": "prettier --write src"
diff --git a/packages/idb-bridge/package.json b/packages/idb-bridge/package.json
index 263a095bf..89f191655 100644
--- a/packages/idb-bridge/package.json
+++ b/packages/idb-bridge/package.json
@@ -11,7 +11,6 @@
   "private": false,
   "scripts": {
     "test": "tsc && ava",
-    "prepare": "tsc",
     "compile": "tsc",
     "clean": "rimraf dist lib tsconfig.tsbuildinfo",
     "pretty": "prettier --write src"
diff --git a/packages/merchant-backend-ui/package.json 
b/packages/merchant-backend-ui/package.json
index 03bb1e894..23e5007e9 100644
--- a/packages/merchant-backend-ui/package.json
+++ b/packages/merchant-backend-ui/package.json
@@ -6,7 +6,6 @@
   "scripts": {
     "compile": "tsc && ./build.mjs",
     "build": "pnpm compile",
-    "prepare": "tsc",
     "render-examples": "ts-node -O '{\"module\": \"commonjs\"}' -T 
render-examples.ts dist/pages dist/examples",
     "lint-check": "eslint '{src,tests}/**/*.{js,jsx,ts,tsx}'",
     "lint-fix": "eslint --fix '{src,tests}/**/*.{js,jsx,ts,tsx}'",
diff --git a/packages/pogen/package.json b/packages/pogen/package.json
index 09d3ccda5..ae1dfac3b 100644
--- a/packages/pogen/package.json
+++ b/packages/pogen/package.json
@@ -7,7 +7,6 @@
   "author": "Florian Dold",
   "license": "GPL-2.0+",
   "scripts": {
-    "prepare": "tsc",
     "compile": "tsc"
   },
   "devDependencies": {
diff --git a/packages/taler-util/package.json b/packages/taler-util/package.json
index f78791752..ba3a22d31 100644
--- a/packages/taler-util/package.json
+++ b/packages/taler-util/package.json
@@ -52,7 +52,6 @@
     }
   },
   "scripts": {
-    "prepare": "tsc",
     "compile": "tsc",
     "test": "tsc && ava",
     "clean": "rimraf dist lib tsconfig.tsbuildinfo",
diff --git a/packages/taler-wallet-cli/package.json 
b/packages/taler-wallet-cli/package.json
index 643a24deb..55f9adecb 100644
--- a/packages/taler-wallet-cli/package.json
+++ b/packages/taler-wallet-cli/package.json
@@ -16,7 +16,6 @@
   },
   "type": "module",
   "scripts": {
-    "prepare": "true",
     "compile": "./build-node.mjs",
     "test": "tsc",
     "clean": "rimraf lib dist tsconfig.tsbuildinfo",
diff --git a/packages/taler-wallet-core/package.json 
b/packages/taler-wallet-core/package.json
index 164198f7c..caf212b37 100644
--- a/packages/taler-wallet-core/package.json
+++ b/packages/taler-wallet-core/package.json
@@ -12,7 +12,6 @@
   "author": "Florian Dold",
   "license": "GPL-3.0",
   "scripts": {
-    "prepare": "tsc",
     "compile": "tsc",
     "pretty": "prettier --write src",
     "test": "tsc && ava",
diff --git a/packages/taler-wallet-webextension/package.json 
b/packages/taler-wallet-webextension/package.json
index f2d4e2923..002d888a2 100644
--- a/packages/taler-wallet-webextension/package.json
+++ b/packages/taler-wallet-webextension/package.json
@@ -13,7 +13,6 @@
     "test": "./test.mjs && mocha --require source-map-support/register 
'dist/test/**/*.test.js' 'dist/test/**/test.js'",
     "test:coverage": "nyc pnpm test",
     "compile": "./patch-linaria.sh && tsc && ./build.mjs",
-    "prepare": "tsc",
     "dev": "./dev.mjs",
     "pretty": "prettier --write src",
     "i18n:extract": "pogen extract",
diff --git a/packages/web-util/package.json b/packages/web-util/package.json
index d2a8a3144..0fc76c0d6 100644
--- a/packages/web-util/package.json
+++ b/packages/web-util/package.json
@@ -27,7 +27,6 @@
     }
   },
   "scripts": {
-    "prepare": "tsc && ./build.mjs",
     "compile": "tsc && ./build.mjs",
     "clean": "rimraf dist lib tsconfig.tsbuildinfo",
     "pretty": "prettier --write src"

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.