ds300 / patch-package

Fix broken node modules instantly 🏃🏽‍♀️💨
MIT License
10.48k stars 295 forks source link

patch-package with yarn dlx error #535

Open YadBro opened 1 month ago

YadBro commented 1 month ago

What is yarn dlx?

Simply similar npx but in yarn version. Idk if this issue from yarn or from patch-package. But I will still add this issue for who want using yarn dlx

How to reproduce?

  1. Create some patch to whatever package you want
  2. Run yarn dlx patch-package <some-package>
  3. And you will getting this:

here my case: yarn dlx patch-package react-native-snap-carousel ➤ YN0000: ┌ Resolution step ➤ YN0061: │ rimraf@npm:2.7.1 is deprecated: Rimraf versions prior to v4 are no longer supported ➤ YN0061: │ glob@npm:7.2.3 is deprecated: Glob versions prior to v9 are no longer supported ➤ YN0061: │ inflight@npm:1.0.6 is deprecated: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful. ➤ YN0000: └ Completed in 1s 62ms ➤ YN0000: ┌ Fetch step ➤ YN0000: └ Completed ➤ YN0000: ┌ Link step ➤ YN0000: └ Completed in 0s 288ms ➤ YN0000: Done with warnings in 1s 405ms

patch-package 8.0.0 • Creating temporary folder • Installing react-native-snap-carousel@3.9.1 with yarn

{ status: 1, signal: null, output: [ null, <Buffer 1b 5b 33 31 6d 1b 5b 31 6d 55 6e 6b 6e 6f 77 6e 20 53 79 6e 74 61 78 20 45 72 72 6f 72 1b 5b 32 32 6d 1b 5b 33 39 6d 3a 20 55 6e 73 75 70 70 6f 72 74 ... 142 more bytes>,

], pid: 11348, stdout: , stderr: , error: null } /private/var/folders/j8/p7gb5ydj5cb4wpzqjgf33zfh0000gn/T/xfs-b0e03d33/dlx-11343/node_modules/patch-package/dist/makePatch.js:395 throw e; ^ { status: 1, signal: null, output: [ null, Buffer(192) [Uint8Array] [ 27, 91, 51, 49, 109, 27, 91, 49, 109, 85, 110, 107, 110, 111, 119, 110, 32, 83, 121, 110, 116, 97, 120, 32, 69, 114, 114, 111, 114, 27, 91, 50, 50, 109, 27, 91, 51, 57, 109, 58, 32, 85, 110, 115, 117, 112, 112, 111, 114, 116, 101, 100, 32, 111, 112, 116, 105, 111, 110, 32, 110, 97, 109, 101, 32, 40, 34, 45, 45, 105, 103, 110, 111, 114, 101, 45, 115, 99, 114, 105, 112, 116, 115, 34, 41, 46, 10, 10, 36, 32, 121, 97, 114, 110, 32, 105, 110, 115, 116, 97, ... 92 more items ], Buffer(0) [Uint8Array] [] ], pid: 11348, stdout: Buffer(192) [Uint8Array] [ 27, 91, 51, 49, 109, 27, 91, 49, 109, 85, 110, 107, 110, 111, 119, 110, 32, 83, 121, 110, 116, 97, 120, 32, 69, 114, 114, 111, 114, 27, 91, 50, 50, 109, 27, 91, 51, 57, 109, 58, 32, 85, 110, 115, 117, 112, 112, 111, 114, 116, 101, 100, 32, 111, 112, 116, 105, 111, 110, 32, 110, 97, 109, 101, 32, 40, 34, 45, 45, 105, 103, 110, 111, 114, 101, 45, 115, 99, 114, 105, 112, 116, 115, 34, 41, 46, 10, 10, 36, 32, 121, 97, 114, 110, 32, 105, 110, 115, 116, 97, ... 92 more items ], stderr: Buffer(0) [Uint8Array] [], error: null } - Node.js version: v20.13.1 - Yarn version: v3.6.4 ✨**But if I run with npx this will not issue**
YadBro commented 1 month ago

Oh yeah I forgot, I also see usage from this using yarn also: https://github.com/ds300/patch-package?tab=readme-ov-file#making-patches

I tried without dlx so: yarn patch-package react-native-snap-carousel

still getting error, and the error same as my issue up there

nfantone commented 1 month ago

FWIW, I'm having the same with npx.

❯ npx patch-package some-module
Need to install the following packages:
patch-package@8.0.0
Ok to proceed? (y)

npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated rimraf@2.7.1: Rimraf versions prior to v4 are no longer supported
npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported
patch-package 8.0.0
• Creating temporary folder
• Installing some-module@1.17.0 with yarn

{
  status: 1,
  signal: null,
  output: [
    null,
    <Buffer 1b 5b 33 31 6d 1b 5b 31 6d 55 6e 6b 6e 6f 77 6e 20 53 79 6e 74 61 78 20 45 72 72 6f 72 1b 5b 32 32 6d 1b 5b 33 39 6d 3a 20 55 6e 73 75 70 70 6f 72 74 ... 185 more bytes>,
    <Buffer >
  ],
  pid: 19642,
  stdout: <Buffer 1b 5b 33 31 6d 1b 5b 31 6d 55 6e 6b 6e 6f 77 6e 20 53 79 6e 74 61 78 20 45 72 72 6f 72 1b 5b 32 32 6d 1b 5b 33 39 6d 3a 20 55 6e 73 75 70 70 6f 72 74 ... 185 more bytes>,
  stderr: <Buffer >,
  error: null
}

/Users/user.name/.npm/_npx/1a782c0cdc01f9d4/node_modules/patch-package/dist/makePatch.js:395
        throw e;
        ^
{
  status: 1,
  signal: null,
  output: [
    null,
    Buffer(235) [Uint8Array] [
       27,  91,  51,  49, 109,  27,  91,  49, 109,  85, 110, 107,
      110, 111, 119, 110,  32,  83, 121, 110, 116,  97, 120,  32,
       69, 114, 114, 111, 114,  27,  91,  50,  50, 109,  27,  91,
       51,  57, 109,  58,  32,  85, 110, 115, 117, 112, 112, 111,
      114, 116, 101, 100,  32, 111, 112, 116, 105, 111, 110,  32,
      110,  97, 109, 101,  32,  40,  34,  45,  45, 105, 103, 110,
      111, 114, 101,  45, 115,  99, 114, 105, 112, 116, 115,  34,
       41,  46,  10,  10,  36,  32, 121,  97, 114, 110,  32, 105,
      110, 115, 116,  97,
      ... 135 more items
    ],
    Buffer(0) [Uint8Array] []
  ],
  pid: 19642,
  stdout: Buffer(235) [Uint8Array] [
     27,  91,  51,  49, 109,  27,  91,  49, 109,  85, 110, 107,
    110, 111, 119, 110,  32,  83, 121, 110, 116,  97, 120,  32,
     69, 114, 114, 111, 114,  27,  91,  50,  50, 109,  27,  91,
     51,  57, 109,  58,  32,  85, 110, 115, 117, 112, 112, 111,
    114, 116, 101, 100,  32, 111, 112, 116, 105, 111, 110,  32,
    110,  97, 109, 101,  32,  40,  34,  45,  45, 105, 103, 110,
    111, 114, 101,  45, 115,  99, 114, 105, 112, 116, 115,  34,
     41,  46,  10,  10,  36,  32, 121,  97, 114, 110,  32, 105,
    110, 115, 116,  97,
    ... 135 more items
  ],
  stderr: Buffer(0) [Uint8Array] [],
  error: null
}

Node.js v20.18.0
TuanTran-Tsk commented 1 week ago

any update?

YadBro commented 1 week ago

If you are using node, try to using as same my node version (v20.13.1). I've tried using npx in that version and not issue for me