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.

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

Index.js is missing #12

Closed
gianpietro1 opened this issue May 3, 2017 · 17 comments
Closed

Index.js is missing #12

gianpietro1 opened this issue May 3, 2017 · 17 comments

Comments

@gianpietro1
Copy link

Hi,

Not sure why, my environment has been pulling a scss-tokenizer version without an index.js file, so I've had to manually generate it every time.

It's been called by saas-graph 2.1.1, which package.json file says: ""scss-tokenizer": "^0.2.1""

Details of my environment (I know it's kind of old, but it's an application requirement):

  • Ubuntu Trusty 14.04.4
  • Node version 0.10.25
  • NPM version 1.3.10

Regards,
Gianpietro

@xzyfer
Copy link
Member

xzyfer commented May 3, 2017 via email

@jfridye
Copy link

jfridye commented May 3, 2017

Thanks for checking into this.

I'm seeing a similar error with npm v0.10.38. This started within the past few days. We have sass-graph@2.2.2 which depends on scss-tokenizer@0.2.1.

@xzyfer
Copy link
Member

xzyfer commented May 3, 2017 via email

@feraudet
Copy link

+1

1 similar comment
@rasta-rocket
Copy link

+1

@xzyfer
Copy link
Member

xzyfer commented May 11, 2017 via email

@rasta-rocket
Copy link

npm version 1.3.10
node version v0.10.25

@jfridye
Copy link

jfridye commented May 11, 2017

npm 1.4.28
node v0.10.38

@xzyfer
Copy link
Member

xzyfer commented May 11, 2017

I have not been able to reproduce this locally with

➔ node -v
v0.10.48
➔ npm -v
1.4.29

Could y'all please show the output of listing the files in scss-tokenizer directory?

ls -lsa node_modules/node-sass/node_modules/sass-graph/node_modules/scss-tokenizer

* the path may be different on your system.

Also please try downloading the tarball directly and confirming the index.js is there. You can find the tarball by running

npm v scss-tokenizer

The end of the output should how the tarball url

  dist:
   { shasum: '07c0cc577bb7ab4d08fd900185adbf4bc844141d',
     tarball: 'https://registry.npmjs.org/scss-tokenizer/-/scss-tokenizer-0.2.1.tgz' },

@rasta-rocket
Copy link

rasta-rocket commented May 12, 2017

$ npm install node-sass
  ...

$ ls -lsa node_modules/node-sass/node_modules/sass-graph/node_modules/scss-tokenizer
total 24
4 drwxrwxr-x 4 user group 4096 May 12 09:53 .
4 drwxrwxr-x 5 user group 4096 May 12 09:53 ..
4 drwxr-xr-x 2 user group 4096 May 12 09:53 lib
4 drwxrwxr-x 4 user group 4096 May 12 09:53 node_modules
4 -rw-r--r-- 1 user group 2387 May 12 09:53 package.json
4 -rw-r--r-- 1 user group 1238 Jan 29 04:58 README.md

$ npm v scss-tokenizer | tail
dist: 
   { shasum: '07c0cc577bb7ab4d08fd900185adbf4bc844141d',
     tarball: 'https://registry.npmjs.org/scss-tokenizer/-/scss-tokenizer-0.2.1.tgz' },

$ wget https://registry.npmjs.org/scss-tokenizer/-/scss-tokenizer-0.2.1.tgz
--2017-05-12 13:13:14--  https://registry.npmjs.org/scss-tokenizer/-/scss-tokenizer-0.2.1.tgz
Resolving registry.npmjs.org (registry.npmjs.org)... 151.101.120.162
Connecting to registry.npmjs.org (registry.npmjs.org)|151.101.120.162|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 5777 (5.6K) [application/octet-stream]
Saving to: ‘scss-tokenizer-0.2.1.tgz’

100%[================================================================================================================================================================================================>] 5,777       --.-K/s   in 0s      

2017-05-12 13:13:14 (70.0 MB/s) - ‘scss-tokenizer-0.2.1.tgz’ saved [5777/5777]

$ tar -xzf scss-tokenizer-0.2.1.tgz

$ ls package/
total 20
-rw-r--r-- 1 user group   49 Apr 29 01:52 index.js
drwxrwxr-x 2 user group 4096 May 12 13:13 lib
-rw-r--r-- 1 user group 1077 Jan 28 11:17 LICENSE
-rw-r--r-- 1 user group  963 Apr 29 02:14 package.json
-rw-r--r-- 1 user group 1238 Jan 29 04:58 README.md

Also I would like to add a remark, even when I npm install scss-tokenizer there is no index.js and src directory

@xzyfer
Copy link
Member

xzyfer commented May 12, 2017

Thanks @rasta-rocket, this is fascinating.

@xzyfer
Copy link
Member

xzyfer commented May 12, 2017

Ok I've been able to reproduce this with

➔ node -v
v0.10.25
➔ npm -v
1.3.10

After some npm version bisecting I have been able to reproduce this issue up until npm@1.4.0 but not after. This conflicts with the information provided by @jfridye.

I'm going to investigate some potential workarounds

xzyfer added a commit that referenced this issue May 12, 2017
This shouldn't be necessary but may resolve an issue with npm@<1.4.0.

See #12.
@xzyfer
Copy link
Member

xzyfer commented May 12, 2017

I believe this is resolved in v0.2.3. Please confirm.

@rasta-rocket
Copy link

Also I tested the 0.2.0 version, the index.js is present

@xzyfer
Copy link
Member

xzyfer commented May 12, 2017

Fixed in d84958c

@xzyfer xzyfer closed this as completed May 12, 2017
@rasta-rocket
Copy link

Thx. Do you will update the new version on sass-graph package.json ?

@xzyfer
Copy link
Member

xzyfer commented May 12, 2017

Published sass-graph@2.2.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants