Skip to content
This repository was archived by the owner on Feb 13, 2024. It is now read-only.

Read ndk installdir from android class instead of default value #38

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions manifests/ndk.pp
Original file line number Diff line number Diff line change
@@ -18,7 +18,7 @@
$ndk_version = $android::params::ndk_version
)
{
include android::params
include android
include wget

$base_path = "http://dl.google.com/android/ndk/${ndk_version}"
@@ -36,7 +36,7 @@
} ->
exec { 'run-androidndk':
command => "${ndk_installer} -y",
cwd => $android::params::installdir,
cwd => $android::paths::installdir,
}

}
2 changes: 1 addition & 1 deletion manifests/sdk.pp
Original file line number Diff line number Diff line change
@@ -59,7 +59,7 @@
ensure_packages($::osfamily ? {
# list 64-bit version and use latest for installation too so that the same version is applied to both
'RedHat' => ['glibc.i686','zlib.i686','libstdc++.i686','zlib','libstdc++'],
'Debian' => ['ia32-libs'],
'Debian' => ['lib32stdc++6','lib32z1'],
default => [],
})
}
6 changes: 4 additions & 2 deletions spec/classes/sdk_spec.rb
Original file line number Diff line number Diff line change
@@ -29,7 +29,8 @@
:operatingsystem => 'Ubuntu',
:osfamily => 'Debian'
} }
it { should contain_package('ia32-libs') }
it { should contain_package('lib32stdc++6') }
it { should contain_package('lib32z1') }
end

context '64bit Debian, amd64 architecture', :compile do
@@ -38,7 +39,8 @@
:osfamily => 'Debian',
:architecture => 'amd64'
} }
it { should contain_package('ia32-libs') }
it { should contain_package('lib32stdc++6') }
it { should contain_package('lib32z1') }
end

end