-
Notifications
You must be signed in to change notification settings - Fork 167
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.
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
detect-libc returns glibc
on Linux
#203
Comments
kudos to @killagu |
It looks like npm-install does not validate the |
@cyjake No need to rename the package name, the error should be introduced by here, not getting the abi correctly. |
https://github.com/parcel-bundler/parcel-css/blob/master/scripts/build-npm.js#L7 changing from |
There is no |
well, guess the only option left is to alias the abi naming in the build script. AFAICT, both cnpm and yarn support that. I can take a look at it tomorrow. |
Hi, just to be sure, this is related to the change in #188 that can cause If not I'll open a separate issue. |
@onigoetz Yes it's same issue. |
A string like
We can view the details via rustc: So there is no need to rename it, use |
[admin@14e2f8a21ed0 /home/admin]
$ll node_modules/@parcel/
total 4
lrwxrwxrwx 1 admin admin 34 Jun 17 09:31 css -> ../_@parcel_css@1.10.1@@parcel/css
lrwxrwxrwx 1 admin admin 62 Jun 17 09:31 css-linux-x64-gnu -> ../_@parcel_css-linux-x64-gnu@1.10.1@@parcel/css-linux-x64-gnu verified 💯 |
hence tries to locale package @parcel/css-linux-x64-glibc,
Should
@parcel/css-linux-x64-gnu
be renamed as@parcel/css-linux-x64-glibc
, or both should be published?The text was updated successfully, but these errors were encountered: