Skip to content

Fix #26: string_methods.cpp does not support ChaiScript 6+ - #29

Merged
lefticus merged 1 commit into
ChaiScript:masterfrom
leftibot:fix/issue-26-string-methods-cpp-does-not-support-chai
Apr 16, 2026
Merged

lefticus merged 1 commit into
ChaiScript:masterfrom
leftibot:fix/issue-26-string-methods-cpp-does-not-support-chai

Conversation

@leftibot

Copy link
Copy Markdown
Contributor

Automated fix by @leftibot.

What changed

Fix #26: Update to ChaiScript 6.1.0 and fix vector_type API
ChaiScript 6+ changed the vector_type signature from returning a ModulePtr
to taking a Module& parameter and returning void. The ChaiScript constructor
also no longer accepts a ModulePtr for stdlib. Updated bootstrap() to register
the vector_type for std::vectorstd::string directly within the module, and
updated tests to use the new ChaiScript 6+ API. Fixed Catch SIGSTKSZ
compile error on newer glibc where SIGSTKSZ is no longer a constant expression.
Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Files

 cmake/chaiscript.cmake                       | 2 +-
 include/chaiscript/extras/string_methods.hpp | 5 ++++-
 tests/catch.hpp                              | 6 +++---
 tests/math.cpp                               | 4 +---
 tests/string_methods.cpp                     | 5 +----
 5 files changed, 10 insertions(+), 12 deletions(-)

Closes #26

Triggered by @lefticus.

ChaiScript 6+ changed the vector_type signature from returning a ModulePtr
to taking a Module& parameter and returning void. The ChaiScript constructor
also no longer accepts a ModulePtr for stdlib. Updated bootstrap() to register
the vector_type for std::vector<std::string> directly within the module, and
updated tests to use the new ChaiScript 6+ API. Fixed Catch SIGSTKSZ
compile error on newer glibc where SIGSTKSZ is no longer a constant expression.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

@RobLoach RobLoach left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks about right.

@lefticus
lefticus merged commit fbe744e into ChaiScript:master Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

string_methods.cpp does not support ChaiScript 6+

3 participants