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

upgrade: drivelist to v5.0.25 #1574

Merged
merged 1 commit into from Jul 12, 2017
Merged

upgrade: drivelist to v5.0.25 #1574

merged 1 commit into from Jul 12, 2017

Conversation

jviotti
Copy link
Contributor

@jviotti jviotti commented Jul 6, 2017

This version contains various improvements on the drive detection system
mainly on Windows. The improvements can be summarized as follows:

This PR also sets the DRIVELIST_DEBUG environment variable to enable
debug information from Windows' drivelist implementation.

Change-Type: patch
Changelog-Entry: Implement Windows drive detection using C++
Changelog-Entry: Fix various Windows .bat spawning issues
Changelog-Entry: Fix 0.0 GB Windows drive detection issues
Changelog-Entry: Cleanup drive detection temporary scripts created for other operating systems
Fixes: #1108
Fixes: #1054
Fixes: #995
Fixes: #1483
Fixes: #1142
Fixes: #1571
See: https://github.com/resin-io-modules/drivelist/blob/master/CHANGELOG.md
Signed-off-by: Juan Cruz Viotti jv@jviotti.com

@jviotti
Copy link
Contributor Author

jviotti commented Jul 6, 2017

This commit still references a drivelist unmerged branch. I'll update it once the new version is released to npm.

@jhermsmeier
Copy link
Contributor

This will also include balena-io-modules/drivelist#191 if I'm not mistaken, fixing #1571@jviotti do you want to add that to the commit message as well?

@jviotti
Copy link
Contributor Author

jviotti commented Jul 6, 2017

Oh, right, I'll add it.

@jviotti jviotti force-pushed the upgrade-drivelist branch 2 times, most recently from 65c1c38 to c2243d8 Compare July 11, 2017 21:54
@jviotti
Copy link
Contributor Author

jviotti commented Jul 11, 2017

This PR has now been updated to point to a version at npm.

@@ -22,6 +22,10 @@ const path = require('path');
const EXIT_CODES = require('../shared/exit-codes');
let mainWindow = null;

// Enable drivelist debugging information
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably worth mentioning this in the commit-message too?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@lurch
Copy link
Contributor

lurch commented Jul 11, 2017

This version contains various improvements on the drive detection system
mainly on Windows. The improvements can be summarized as follows:

- Drivelist no longer spawns a .bat script to perform its job. This
  means that a lot of spawning related issues are now fixed

- Drivelist doesn't fetch drive sizes from WMI anymore, which was known
  to incorrectly report a very small size for certain drives, causing
  the famous "0.0 GB" Windows issues (see
  balena-io-modules/drivelist#142)

- Cleanup temporary scripts after execution

This PR also sets the DRIVELIST_DEBUG environment variable to enable
debug information from Windows' drivelist implementation.

Change-Type: patch
Changelog-Entry: Implement Windows drive detection using C++
Changelog-Entry: Fix various Windows `.bat` spawning issues
Changelog-Entry: Fix 0.0 GB Windows drive detection issues
Changelog-Entry: Cleanup drive detection temporary scripts created for other operating systems
Fixes: #1108
Fixes: #1054
Fixes: #995
Fixes: #1483
Fixes: #1142
Fixes: #1571
See: https://github.com/resin-io-modules/drivelist/blob/master/CHANGELOG.md
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 77c60b9 into master Jul 12, 2017
@jviotti jviotti deleted the upgrade-drivelist branch July 12, 2017 19:18
@DarkWolfx
Copy link

Hello! So I'm having the same issue, when starting up Etcher, the error saying "Couldn't scan the drives" is showing up.
I tested the build posted above for x64 bit architecture as I have Windows 10 Pro N x64 bit, it installed but the same issue came up.
I read before you might believe would be the antivirus, so I disabled Windows Defender and firewall, same issue.

@lurch
Copy link
Contributor

lurch commented Jul 26, 2017

@DarkWolfx Maybe you're also being affected by balena-io-modules/drivelist#198 ?
If you could jump over there and help us to diagnose the issue that would be great thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment