commit:     5f7ad4a0e3ddd944c094ebc9719fdd5982ea76f6
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Wed Jun 14 21:38:12 2023 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Thu Jun 15 21:13:50 2023 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5f7ad4a0

ruby-ng.eclass: add _ruby_get_use_targets comment

I don't see us adding a non-ruby* target any time soon so this should be fine.

Closes: https://github.com/gentoo/gentoo/pull/31428
Signed-off-by: Sam James <sam <AT> gentoo.org>

 eclass/ruby-ng.eclass | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/eclass/ruby-ng.eclass b/eclass/ruby-ng.eclass
index 2f14c22ace37..5cea44101593 100644
--- a/eclass/ruby-ng.eclass
+++ b/eclass/ruby-ng.eclass
@@ -105,6 +105,8 @@ ruby_implementation_depend() {
 _ruby_get_all_impls() {
        _RUBY_GET_ALL_IMPLS=()
 
+       # XXX: Please update _ruby_get_use_targets if adding a non-'ruby*'
+       # target.
        local i found_valid_impl
        for i in ${USE_RUBY}; do
                case ${i} in
@@ -350,6 +352,7 @@ _ruby_get_use_targets() {
        _ruby_set_globals_invalidate_if_stale
 
        local impls="${_RUBY_GET_ALL_IMPLS[@]}"
+       # XXX: This assumes all targets begin with 'ruby'.
        _RUBY_GET_USE_TARGETS="${impls//ruby/ruby_targets_ruby}"
 }
 

Reply via email to