Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Strange file created on arm64 #59

Open
dbcesar opened this issue Aug 11, 2022 · 2 comments · May be fixed by #60
Open

Strange file created on arm64 #59

dbcesar opened this issue Aug 11, 2022 · 2 comments · May be fixed by #60

Comments

@dbcesar
Copy link

dbcesar commented Aug 11, 2022

Hello everyone,

I'm using [email protected], which requires [email protected] on a ubuntu 18.04 with arm64 architecture I end up having a file with

user@manager-web$ nodejs --version
v16.16.0
user@manager-web$ npm -v
8.16.0
user@manager-web$ ls node_modules/cwebp-bin/ -lah
total 52K
drwxr-xr-x   4 user user 4.0K Aug 11 10:48  .
drwxr-xr-x 537 user user  20K Aug 11 10:20  ..
-rw-r--r--   1 user user    0 Aug 11 10:18 ''$'\001\225\001''@'$'\b\v''@@8'
-rwxr-xr-x   1 user user  219 Aug 11 10:18  cli.js
-rw-r--r--   1 user user   62 Aug 11 10:18  index.js
drwxr-xr-x   2 user user 4.0K Aug 11 10:18  lib
-rw-r--r--   1 user user 1.1K Aug 11 10:18  license
-rw-r--r--   1 user user  924 Aug 11 10:18  package.json
-rw-r--r--   1 user user  915 Aug 11 10:18  readme.md
drwxr-xr-x   6 user user 4.0K Aug 11 10:20  vendor

Do you have any idea where is that coming from? I can confirm that using same setup, but on a x86_64 architecture I don't have this file.

Thanks in advance
gilbsgilbs added a commit to gilbsgilbs/cwebp-bin that referenced this issue Aug 23, 2022
@gilbsgilbs
fix dangling file created at installation on linux/arm64
c41cf7c 
fixes imagemin#59
@gilbsgilbs gilbsgilbs linked a pull request Aug 23, 2022 that will close this issue
Open
@gilbsgilbs
Copy link

It comes from this weird bug. It even causes a crash when installing with yarn instead of npm. I pushed a workaround in #60.

@dbcesar
Copy link
Author

dbcesar commented Aug 30, 2022

Thanks for replying. I was afraid to be the only one experiencing it. I'm waiting for the PR to be merged so I can test it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix crash at installation on linux platforms with non-intel archs gilbsgilbs/cwebp-bin
2 participants
@gilbsgilbs @dbcesar