operatingSystems.linux = false, still downloads linux drivers

[SiKing]

I am using version 1.0.0. I have:
<operatingSystems>
<windows>true</windows>
<linux>false</linux>
<osx>false</osx>
</operatingSystems>
Just tried this on my Linux machine, and it downloads the Linux binaries!?!?

Further, it errors with: ".bz2 is an unsupported archive type"

Am I doing this wrong?

[Ardesco]

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

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

[SiKing]

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 notifications@github.com 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
.

[Ardesco]

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.

[SiKing]

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.

[Ardesco]

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.

[Ardesco]

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

[SiKing]

Works great! :)

On 21 April 2014 07:39, Mark Collin notifications@github.com 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
.