Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make spec tests pass against a known version of all shells #2

Closed
kootenpv opened this issue Dec 7, 2016 · 4 comments
Closed

Make spec tests pass against a known version of all shells #2

kootenpv opened this issue Dec 7, 2016 · 4 comments

Comments

@kootenpv
Copy link

kootenpv commented Dec 7, 2016

No description provided.

@andychu
Copy link
Contributor

andychu commented Dec 7, 2016

The examples are all in the tests here:

https://github.com/oilshell/oil/tree/master/tests

which I talk about here

http://www.oilshell.org/blog/2016/11/21.html

Unimplemented features have failing tests; I plan to change that to a different system to prevent regressions.

@andychu andychu changed the title Maybe add some examples Make all spec tests green and prevent regressions Dec 7, 2016
@andychu
Copy link
Contributor

andychu commented Mar 3, 2017

This is done on at least one machine... still need to figure out the environment/version issue.

http://lists.oilshell.org/private.cgi/oil-dev-oilshell.org/2017-March/000004.html

@andychu andychu changed the title Make all spec tests green and prevent regressions Make spec tests pass against a known version of all shells Mar 26, 2017
@andychu
Copy link
Contributor

andychu commented Mar 26, 2017

bash/dash/mksh/zsh

@andychu
Copy link
Contributor

andychu commented Sep 26, 2017

issue #42 is more specific about this

@andychu andychu closed this as completed Sep 26, 2017
@andychu andychu mentioned this issue Dec 9, 2017
Merged
@andychu andychu mentioned this issue Feb 19, 2019
Closed
@andychu andychu mentioned this issue Apr 30, 2019
Closed
@andychu andychu mentioned this issue Jun 7, 2019
Closed
This was referenced Mar 15, 2020
Merged
Merged
@andychu andychu mentioned this issue Oct 10, 2020
Closed
@andychu andychu mentioned this issue Feb 5, 2021
Closed
andychu pushed a commit that referenced this issue Feb 6, 2021
[my_runtime_test] Fix bug caught on continuous build.
ca5de9e 
Why didn't this happen on my own machine?

http://travis-ci.oilshell.org/srht-jobs/2021-02-06__19-34-07.wwz/_tmp/toil/logs/cpp-unit-all.txt

--> COLLECT with 0 roots
i = -2147483648
.AddressSanitizer:DEADLYSIGNAL
=================================================================
==3052==ERROR: AddressSanitizer: SEGV on unknown address 0x7f8666c53034 (pc 0x55618e7f6d5b bp 0x7ffc429f1190 sp 0x7ffc429f1180 T0)
==3052==The signal is caused by a READ memory access.
    #0 0x55618e7f6d5a in gc_heap::str_equals(gc_heap::Str*, gc_heap::Str*) /home/build/oil/mycpp/gc_heap.cc:260
    #1 0x55618e7dafd0 in str_replace_test /home/build/oil/mycpp/my_runtime_test.cc:162
    #2 0x55618e7e7bd6 in main /home/build/oil/mycpp/my_runtime_test.cc:779
    #3 0x7f866924109a in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2409a)
    #4 0x55618e7d94b9 in _start (/home/build/oil/mycpp/_bin/my_runtime_test.asan+0xd4b9)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/build/oil/mycpp/gc_heap.cc:260 in gc_heap::str_equals(gc_heap::Str*, gc_heap::Str*)
@andychu andychu mentioned this issue Jan 6, 2022
Closed
@watk watk mentioned this issue Jul 21, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@andychu @kootenpv