Skip to content
This repository has been archived by the owner on Mar 1, 2020. It is now read-only.

TypeError: Cannot read property 'replace' of undefined #31

Open
kemayo opened this issue Oct 25, 2016 · 5 comments
Open

TypeError: Cannot read property 'replace' of undefined #31

kemayo opened this issue Oct 25, 2016 · 5 comments

Comments

@kemayo
Copy link

kemayo commented Oct 25, 2016

Looking at this, I suppose it's that the file map isn't being built (for whatever reason).

dlynch@wmf1453:~/wikimedia$ cdnjs-importer -g https://gerrit.wikimedia.org/r/p/oojs/core -p ~/src/cdnjs
info  Pulling from origin.
info  Cloning https://gerrit.wikimedia.org/r/p/oojs/core into /var/folders/28/9vnnrdl97hx9zt4z9n15nxgc0000gp/T/tmp-804249v5twSnysk6U
info  Set the source path: /dist
info  Creating the version directory: /Users/dlynch/src/cdnjs/ajax/libs/oojs/1.1.10
info  Getting the library files.
/usr/local/lib/node_modules/cdnjs-importer/lib/index.js:211
                package.filename = files[0].replace(pwd, "");
                                           ^

TypeError: Cannot read property 'replace' of undefined
    at /usr/local/lib/node_modules/cdnjs-importer/lib/index.js:211:44
@PeterDaveHello
Copy link
Contributor

@IonicaBizau are you free to take a look?

@IonicaBizau
Copy link
Contributor

This importer will have to be refactored from scratch due to the latest
changes.

On Tue, Oct 25, 2016 at 9:10 PM, Peter Dave Hello [email protected]
wrote:

@IonicaBizau https://github.com/IonicaBizau are you free to take a look?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#31 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACu085b6_doAQB3oji7Zac1tuk-DMEW9ks5q3kYEgaJpZM4KgPF-
.

@PeterDaveHello
Copy link
Contributor

@IonicaBizau do you mean 27cc194 by @pvnr0082t ?
If that'll be a problem, maybe we could just revert in the meantime.

@IonicaBizau
Copy link
Contributor

Hmm, I'm not sure, this should be anyways rewritten almost from scratch.

@PeterDaveHello
Copy link
Contributor

Wow ... it'll be a huge work, I wonder if we could have a working version right now and then rewrite one, what do you think?

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

No branches or pull requests

3 participants