Question about is_win32 flag about electron-packager

[malept]

I would think that if anything, this needs to be set to true when the host platform (i.e. os.platform()) is win32, not the target platform.

Perhaps. It was originally only added to win32.js (via #46), so when I refactored, I didn't want to change that behavior.

[kfranqueiro]

I've been manually testing everything I can think of and ignore is one of the last things left on my list, so hopefully I'll hit that sometime in the coming week to confirm this.

[kfranqueiro]

I've confirmed tonight that this is indeed wrong - for example, --ignore='ignore/this' will ignore correctly for windows builds on windows, but not linux/mac builds on windows. The check should be detecting against the host platform, not the platform being targeted. (Or perhaps look at path.sep and if it's not already /, replace based on it.)

I should be able to come up with a PR for this.