Skip to content

Conversation

@satokaz
Copy link

@satokaz satokaz commented Jan 24, 2016

Hi,

Add a fix that to support Oracle Solaris 11. (Check the Solaris 11.3)

  • install-binary option is not supported. nodejs.org binary does not work in solaris 11.
  • io.js is not supported

thanks,
Kazuyuki

nodebrew Outdated
Copy link
Owner

Choose a reason for hiding this comment

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

Use error_and_exit

Copy link

@Alhadis Alhadis left a comment

Choose a reason for hiding this comment

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

Proof-reading.

}

sub _cmd_install_binary {
error_and_exit("Does not supported on Oracle Solaris. This option supports only illumos kernel distribution. \n") if $^O eq 'solaris';
Copy link

Choose a reason for hiding this comment

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

Suggested change
error_and_exit("Does not supported on Oracle Solaris. This option supports only illumos kernel distribution. \n") if $^O eq 'solaris';
error_and_exit("Node.js does not support Oracle Solaris. Only illumos-based distributions are supported.\n") if $^O eq 'solaris';

my $tarball_path = "$src_dir/$target_name.tar.gz";

if ($^O eq 'solaris' && $self->get_type eq 'iojs') {
error_and_exit("io.js does not supported on $^O \n");
Copy link

Choose a reason for hiding this comment

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

Suggested change
error_and_exit("io.js does not supported on $^O \n");
error_and_exit("io.js is not supported on $^O \n");

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants