Skip to content

Commit

Permalink
temporarily disable ruby examples due to swig bug (#4246)
Browse files Browse the repository at this point in the history
  • Loading branch information
karlnapf committed Apr 14, 2018
1 parent 2442845 commit ec0103a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
4 changes: 4 additions & 0 deletions examples/meta/CMakeLists.txt
Expand Up @@ -121,6 +121,10 @@ LIST(APPEND DISABLED_INTERFACES INTERFACE_R)
# see https://github.com/shogun-toolbox/shogun/issues/3018
LIST(APPEND DISABLED_INTERFACES INTERFACE_LUA)

# temporarily disabled. While the interface fully works, the new API does not
# there is a potential bug in swig that needs fixing first
# see https://github.com/shogun-toolbox/shogun/issues/4177
LIST(APPEND DISABLED_INTERFACES INTERFACE_RUBY)

FOREACH(interface ${AVAILABLE_INTERFACES})
list(FIND DISABLED_INTERFACES ${interface} disabled)
Expand Down
5 changes: 0 additions & 5 deletions examples/meta/ruby/CMakeLists.txt
@@ -1,10 +1,5 @@
STRING(REGEX REPLACE "(.*)/narray.*$" "\\1" NARRAY_PATH ${NARRAY_LIB})

LIST(APPEND EXCLUDED_RUBY_META_EXAMPLES
base_api-put_get
base_api-factory
)

# add test case for each generated example
# (not generated yet so have to fake filenames from META_EXAMPLES list)
FOREACH(META_EXAMPLE ${META_EXAMPLES})
Expand Down
2 changes: 1 addition & 1 deletion tests/meta/CMakeLists.txt
Expand Up @@ -63,7 +63,7 @@ FOREACH(REFERENCE_FILE ${META_INTEGRATION_REFERENCES})
#AddMetaIntegrationTest(r INTERFACE_R) # currently we have the r meta examples disabled, so no generated results that can be tested
AddMetaIntegrationTest(octave INTERFACE_OCTAVE)
AddMetaIntegrationTest(csharp INTERFACE_CSHARP)
AddMetaIntegrationTest(ruby INTERFACE_RUBY)
#AddMetaIntegrationTest(ruby INTERFACE_RUBY) # disabled due to potential swig bug, see https://github.com/shogun-toolbox/shogun/issues/4177
AddMetaIntegrationTest(scala INTERFACE_SCALA)
#AddMetaIntegrationTest(lua INTERFACE_LUA) # currently doesn't have meta examples
ENDIF()
Expand Down

0 comments on commit ec0103a

Please sign in to comment.