do not test --version
`llvm-strip-7` is a sane valid strip command that LLVM 7 ships, albeit it does not understand `--version`. It is a bad idea to check that option. Instead just see if the command actually strips something. A copy of `/bin/sh` should suffice. That file must be ubiquitous.
This commit is contained in:
parent
44d25c6528
commit
cc2caa6edf
Notes:
git
2020-08-22 00:34:10 +09:00
@ -59,11 +59,17 @@ describe 'RbConfig::CONFIG' do
|
|||||||
out.should_not be_empty
|
out.should_not be_empty
|
||||||
end
|
end
|
||||||
|
|
||||||
|
require 'tempfile'
|
||||||
it "['STRIP'] exists and can be executed" do
|
it "['STRIP'] exists and can be executed" do
|
||||||
strip = RbConfig::CONFIG.fetch('STRIP')
|
strip = RbConfig::CONFIG.fetch('STRIP')
|
||||||
out = `#{strip} --version`
|
Tempfile.open('sh') do |dst|
|
||||||
$?.should.success?
|
File.open('/bin/sh', 'rb') do |src|
|
||||||
out.should_not be_empty
|
IO.copy_stream(src, dst)
|
||||||
|
dst.flush
|
||||||
|
out =`#{strip} #{dst.to_path}`
|
||||||
|
$?.should.success?
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
Loading…
x
Reference in New Issue
Block a user