operatingSystems.linux = false, still downloads linux drivers about driver-binary-downloader-maven-plugin HOT 7 CLOSED

.bz2 support was added in 1.0.1 (released last night).

The OS thing looks like a bug that i'll look into today.

from driver-binary-downloader-maven-plugin.

Definitely broken. Back on my Windows box I tried windows=false and
osx=true, and it still downloads the Windows binaries.

Tried it also with version 1.0.1 - that just errors with nullpointer
exception. :(

On 3 April 2014 00:39, Mark Collin [email protected] wrote:

.bz2 support was added in 1.0.1 (released last night).

The OS thing looks like a bug that i'll look into today.

Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-39421158
.

from driver-binary-downloader-maven-plugin.

Sorry for the delay, I've just got time to look at this.

I can't reproduce this locally, do you have:

<onlyGetDriversForHostOperatingSystem>false</onlyGetDriversForHostOperatingSystem>

set? If not the above will default to true and it will only try to download drivers for the current OS. I'd like to see a log showing the null pointer error, I'm not sure where that is coming from.

from driver-binary-downloader-maven-plugin.

No rush, I understand you are doing this in your spare time. :)

I do not have the onlyGetDriversForHostOperatingSystem declared. My entire pom is at https://svn.code.sf.net/p/genieos/code-0/trunk/demos/pom.xml

For the nullpointer I opened a separate ticket: issue #13.

from driver-binary-downloader-maven-plugin.

if <onlyGetDriversForHostOperatingSystem> isn't specified it defaults to true. So if you run this on your Linux machine it will only download the Linux binaries. I've added some logging to make this clearer on the console.

Looks like this is working as specified.

from driver-binary-downloader-maven-plugin.

This was missing from the (very basic) documentation in readme.md. It's now been added in.

from driver-binary-downloader-maven-plugin.

Works great! :)

On 21 April 2014 07:39, Mark Collin [email protected] wrote:

This was missing from the (very basic) documentation in readme.md. It's
now been added in.

Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-40940067
.

from driver-binary-downloader-maven-plugin.