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

--asar-unpack = "./node_modules/node-notifier/vendor/**" not working #189

Closed
MathijsvVelde opened this issue Nov 9, 2015 · 15 comments
Closed
Labels
feature:asar Related to the ASAR feature needs info Issue reporter needs to provide more information for maintainers to take action

Comments

@MathijsvVelde
Copy link
Contributor

whadup!!!

part of my package.json:

  "devDependencies": {
    "coffee-script": "^1.10.0",
    "electron-rebuild": "^1.0.2"
  },

Executing on a Mac

electron-packager   ./app/ \
    "App Name" \
    --platform="darwin" \
    --arch="x64" \
    --version="0.34.3" \
    --out="dist/osx" \
    --icon="./assets/osx/icon.icns" \
    --app-bundle-id="com.app.name" \
    --app-version="0.0.1" \
    --ignore="node_modules/(electron-packager|electron-prebuilt)" \
    --prune \
    --asar \
    --asar-unpack="node_modules/node-notifier/vendor/" \

Even though I use the option 'prune' and the option 'ignore' on the node modules "electron-packager","electron-prebuilt", 'app.asar.unpacked' is NOT created

@Asar reported issue
@node-notifier reported issue

any idea's?

Thanks!!

@malept malept added the question ❓ Question about using Electron Packager. Not supported in this issue tracker. label Nov 9, 2015
@malept
Copy link
Member

malept commented Nov 9, 2015

Seems like you want --asar-unpack-dir, which is currently only in master.

@MathijsvVelde
Copy link
Contributor Author

I already tried that and it does not work either, sorry 😞

@malept
Copy link
Member

malept commented Nov 10, 2015

Is it possible to create a minimal testcase app in a gist that reproduces the bug?

@MathijsvVelde
Copy link
Contributor Author

like this

@MathijsvVelde
Copy link
Contributor Author

any luck testing?

@MathijsvVelde
Copy link
Contributor Author

How is it going?
Can you give me a workaround in the meantime?

@MathijsvVelde MathijsvVelde changed the title --asar-unpack = "./node_modules/node-notifier/vendor/**" not working !!BUG!! --asar-unpack = "./node_modules/node-notifier/vendor/**" not working Nov 16, 2015
@malept malept changed the title !!BUG!! --asar-unpack = "./node_modules/node-notifier/vendor/**" not working --asar-unpack = "./node_modules/node-notifier/vendor/**" not working Nov 16, 2015
@malept malept removed the question ❓ Question about using Electron Packager. Not supported in this issue tracker. label Nov 16, 2015
@malept
Copy link
Member

malept commented Nov 16, 2015

I haven't had a chance to try to reproduce it. I imagine the workaround is to just not use the asar feature, but I'm not particularly familiar with it.

@MathijsvVelde
Copy link
Contributor Author

Okay, I'm sorry to say this but, why the fuck did you bother to help me in the first place if you don't know a lot about asar?
Can you forward this issue to someone who's more familiar with asar?
Or can you tell me at least where to look, so I can try to fix it my self?
The least you can do is setting the asar dependency version to '0.8.3' (the latest), you'r currently using '0.6.1'...
There is a great chance that will help...

@sbruchmann
Copy link

@MathijsvVelde Work on your attitude towards people who are trying to help you before working on code. It will come in handy in future conversations.

@malept malept added the needs info Issue reporter needs to provide more information for maintainers to take action label Nov 16, 2015
@malept
Copy link
Member

malept commented Nov 16, 2015

why the fuck did you bother to help me in the first place if you don't know a lot about asar?

Because getting as much information about the issue (including how to reproduce it reliably) is important, especially if it turns out to be a bug. It will help anyone who decides to figure out what the problem is.

Can you forward this issue to someone who's more familiar with asar?

The asar feature (like everything in electron-packager, really) is 100% community driven. If you can't wait for someone with an interest in this feature to take a look at this issue, perhaps try some other places in the Electron community?

Or can you tell me at least where to look, so I can try to fix it my self?

This is quite literally how I would start: https://github.com/maxogden/electron-packager/search?l=javascript&q=asar&utf8=%E2%9C%93

The least you can do is setting the asar dependency version to '0.8.3' (the latest), you'r currently using '0.6.1'...

I assumed you had the latest version of asar since you said you already tried --asar-unpack-dir per #189 (comment). (The asar version change is here: https://github.com/maxogden/electron-packager/pull/174/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2L20)

@MathijsvVelde
Copy link
Contributor Author

I apologize for my attitude, @sbruchmann and @malept!
It appears to be something wrong with your npm package...

on npm un electron-packager; npm i electron-packager

└─┬ [email protected] 
  ├─┬ [email protected] 
  │ └── [email protected] 
  ├─┬ [email protected] 
  │ ├── [email protected] 
  │ ├─┬ [email protected] 
  │ │ └── [email protected] 
  │ └── [email protected] 
  ├── [email protected] 
  └── [email protected] 

It is still using [email protected] and the files aren't updated

it looks like NPM has not been updated yet.

@malept
Copy link
Member

malept commented Nov 17, 2015

As I mentioned earlier, the --unpack-asar-dir feature has not been released on NPM yet. Are you able to reproduce your problem by setting a git URL as the electron-packager source in your package.json?

@malept
Copy link
Member

malept commented Dec 20, 2015

FYI, there is now a released version of electron-packager with --unpack-asar-dir (5.2.0).

@malept malept added the feature:asar Related to the ASAR feature label Dec 20, 2015
@MathijsvVelde
Copy link
Contributor Author

👍

@malept
Copy link
Member

malept commented Mar 8, 2016

Since there hasn't been a response to whether --unpack-asar-dir solved the issue in some time, I'm closing it. If the problem still exists, please feel free to reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature:asar Related to the ASAR feature needs info Issue reporter needs to provide more information for maintainers to take action
Projects
None yet
Development

No branches or pull requests

3 participants