ruby-ng.eclass: Replace unnecessary 'eval ls' with array fnexp
authorMichał Górny <mgorny@gentoo.org>
Thu, 23 Feb 2017 19:08:24 +0000 (20:08 +0100)
committerMichał Górny <mgorny@gentoo.org>
Wed, 8 Mar 2017 07:35:35 +0000 (08:35 +0100)
commit57f836431666db3fd058e6f4e93f7231fcac748c
treefac73b3042a7eed56ebda8e179757f2327a1f78a
parent8e1bff5146ca3f6305ce7349b963c8fa19bc9b63
ruby-ng.eclass: Replace unnecessary 'eval ls' with array fnexp

Replace the unnecessary use of 'eval ls -d ...' with much simpler
and safer filename expansion via bash array. The 'eval' was completely
unnecessary in the original code; however, the late addition of quoting
would have broken it if eval did not implicitly discard the quotes.
The 'ls -d' was unnecessary as well since bash performs filename
expansion before passing the parameter to 'ls'.

Furthermore, a check for accidental multiple expansion has been added.
A complementary check for failed expansion can not be added since
the function is called in src_unpack() as well and the expansion fails
then.
eclass/ruby-ng.eclass