Skip to content

Make the test runner work - #36

Merged
eNascimento178 merged 2 commits into
exercism:mainfrom
IsaacG:fix
May 19, 2026
Merged

eNascimento178 merged 2 commits into
exercism:mainfrom
IsaacG:fix

Conversation

@IsaacG

@IsaacG IsaacG commented May 18, 2026

Copy link
Copy Markdown
Member
  • The shared library (libj.so) demands an executable stack, which modern Linux systems block for security (Data Execution Prevention). This is fixed by clearing the executable flag on the library using patchelf --clear-execstack libj.so.
  • Use Dockerfile ADD to download the tar file.
  • Split into a build and run layer; set up /opt/j9.5 in build then copy it to the run layer
  • Remove dependency on sponge in the runner.

@IsaacG IsaacG added the x:module/test-runner Work on Test Runners label May 18, 2026
@IsaacG
IsaacG requested a review from a team as a code owner May 18, 2026 00:14
@IsaacG IsaacG added x:size/medium Medium amount of work x:rep/medium Medium amount of reputation labels May 18, 2026
@IsaacG
IsaacG requested review from eNascimento178 and removed request for a team May 18, 2026 00:22
@IsaacG

IsaacG commented May 18, 2026

Copy link
Copy Markdown
Member Author

Switching to j9.7 causes a bunch of test failures so I left that alone for now.

@IsaacG

IsaacG commented May 19, 2026

Copy link
Copy Markdown
Member Author

+cc @eNascimento178 for maintainer approval

@eNascimento178

Copy link
Copy Markdown
Contributor

The j9.7 failures are likely caused by this section in run.ijs at line 34:

helper_files_code=: noun define
after_all=: monad define
  require 'convert/json'
  'descrPath orderPath tasksPath'=: ([: <@jpath '~temp/'&,@,&'.json')each ;: 'descriptions order tasks'
  (descrPath 1!:2~ enc_json) descriptions
  (orderPath 1!:2~ enc_json)^:((#\ descriptions) -.@-: >) order
  (tasksPath 1!:2~ enc_json)^:(-.@-:&'') tasks
)

However, since the download and installation process for j9.7 is different, I’ll update it in a later commit, this weekend.

@eNascimento178
eNascimento178 merged commit 42179b3 into exercism:main May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:module/test-runner Work on Test Runners x:rep/medium Medium amount of reputation x:size/medium Medium amount of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants