Variant binaries under build/<variant>/ cannot find the stdlib (load_file lib/... fails)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 62/100
Research direction
The stdlib root is computed from the executable's path (/proc/self/exe/../lib) in the C source, so start there and find where that lookup is built. Add a fallback that also tries exe_dir/../../lib when the binary sits under build/, or have make create a lib symlink in build/. Done when a stdlib-loading program run directly from build/release/eigenscript loads, and a test covers that case.
Written by the indexing model from the issue text.
Description
The stdlib root is exe-relative (/proc/self/exe/../lib). For a variant binary run directly (build/<variant>/eigenscript) it resolves to the nonexistent build/lib, so any program that does load_file "lib/..." fails at load. Seen in the #1665 per-layer review: test_tasks and test_arena_escape fail from build/<arm>/ and pass through the src/eigenscript hardlink. Measurements that call variant binaries by explicit path, which #1676 forces, hit this.
Done when
- A binary run from
build/<variant>/resolves the repolib/. For example, the lookup also triesexe_dir/../../libwhenexe_dirsits underbuild/, ormakeplaces alibsymlink inbuild/. - A test runs one stdlib-loading program from
build/release/eigenscriptdirectly.
- Dominant language
- C
- Stars
- 3
- Forks
- 7
- Avg merge
- 8h 41m
- Merged PRs (30d)
- 254
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- Ships a Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from InauguralSystems/EigenScript
-
eigenscript.h declares 8 observer functions that no longer existPossibly taken @facelessdevhack claimed this 2 days ago. Openarea:observer enhancement found-by:cold-read good first issue
Difficulty 1/5 Under an hour Newbie friendliness 88/100
InauguralSystems/EigenScript#1781 ·
Maintainers usually reply within 1 day
-
Amalgamation compiles with warnings at -O0 (-Wreturn-type) and -O1 (-Wmaybe-uninitialized)Possibly taken @facelessdevhack claimed this 2 days ago. Openarea:embed bug found-by:code-review
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
InauguralSystems/EigenScript#1775 ·
Maintainers usually reply within 1 day
-
UB: (int) cast of a user double outside int range in dispatch / vm_index_is_int / flat_index_atOpenarea:runtime-vm
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
InauguralSystems/EigenScript#1748 ·
Maintainers usually reply within 1 day
-
scan_int_tokens line/col fields are untestedPossibly taken @facelessdevhack claimed this 2 days ago. Openarea:stdlib found-by:critic kind:gate-defect
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
InauguralSystems/EigenScript#1693 ·
Maintainers usually reply within 1 day
-
area:gfx bug
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
InauguralSystems/EigenScript#1684 ·
Maintainers usually reply within 1 day
All issues in InauguralSystems/EigenScript
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
AFLplusplus/AFLplusplus#2903 ·
-
category:port-update
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
colbymchenry/codegraph#2634 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
Maintainers usually reply within 8 days