View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009163 | Taler | wallet-core | public | 2024-09-03 13:57 | 2024-10-06 17:54 |
| Reporter | ericherman | Assigned To | root | ||
| Priority | low | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.14 | Fixed in Version | 0.14 | ||
| Summary | 0009163: building wallet-core requires root access? | ||||
| Description | The README clearly states that pnpm is a requirement: https://git.taler.net/wallet-core.git/tree/README#n8 However, in my testing, it seems "make install" does not work with a locally installed `npm install pnpm`, (the PATH includes `./node_modules/.bin`), as it fails with the error: $ make install pnpm install --frozen-lockfile Scope: all 20 workspace projects ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with package.json Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile" make: *** [Makefile:270: install] Error 1 | ||||
| Steps To Reproduce | # we do not have pnpm installed globally, no: sudo npm install -g pnpm mkdir -pv ~/builds/wallet-core git clone git://git.taler.net/wallet-core.git cd wallet-core export PATH=$(pwd)/node_modules/.bin:$PATH ./bootstrap npm install pnpm ./configure --prefix=~/builds/wallet-core make install | ||||
| Additional Information | 1) It is noteworthy that when a `sudo npm install -g pnpm` is performed, then this error does not occur. 2) Reviewing the Makefile, it seems that `--frozen-lockfile` is a strong requirement (not intended to be over-written). | ||||
| Tags | No tags attached. | ||||
|
|
It's actually possible to install pnpm via npm without root access: $ npm config set prefix $HOME/.npm-global $ export PATH=$HOME.npm-global/bin:$PATH $ npm install -g pnpm I'm wondering, are there any installation instructions where we should add this? |
|
|
I've added this info to the README now. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-09-03 13:57 | ericherman | New Issue | |
| 2024-09-03 13:57 | ericherman | Status | new => assigned |
| 2024-09-03 13:57 | ericherman | Assigned To | => Florian Dold |
| 2024-10-04 15:41 | Florian Dold | Assigned To | Florian Dold => root |
| 2024-10-04 15:41 | Florian Dold | Status | assigned => feedback |
| 2024-10-04 15:41 | Florian Dold | Note Added: 0023440 | |
| 2024-10-04 15:41 | Florian Dold | Target Version | => 1.0 |
| 2024-10-04 15:43 | Florian Dold | Status | feedback => resolved |
| 2024-10-04 15:43 | Florian Dold | Resolution | open => fixed |
| 2024-10-04 15:43 | Florian Dold | Note Added: 0023441 | |
| 2024-10-06 17:54 | Christian Grothoff | Product Version | => git (master) |
| 2024-10-06 17:54 | Christian Grothoff | Fixed in Version | => 0.14 |
| 2024-10-06 17:54 | Christian Grothoff | Target Version | 1.0 => 0.14 |
