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.

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

important_packages argparse breaks CI #16169

Closed
timotheecour opened this issue Nov 28, 2020 · 2 comments · Fixed by #15987
Closed

important_packages argparse breaks CI #16169

timotheecour opened this issue Nov 28, 2020 · 2 comments · Fixed by #15987

Comments

@timotheecour
Copy link
Member

important_packages argparse breaks CI

Current Output

  FAIL: argparse C
  Test "argparse" in category "nimble-packages-1"
  Failure: reBuildFailed
  package test failed
  $ nimble test
    Verifying dependencies for [email protected]
    Compiling /Users/runner/work/Nim/Nim/pkgstemp/argparse/tests/test1 (from package argparse) using c backend
  stack trace: (most recent call last)
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/src/argparse.nim(93, 12) domkParser
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/src/argparse/backend.nim(760, 27) generateDefs
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/src/argparse/backend.nim(689, 19) helpProcDef
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/src/argparse/backend.nim(685, 9) getHelpText
  /Users/runner/work/Nim/Nim/lib/pure/strutils.nim(2873, 11) stripLineEnd
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/tests/test1.nim(25, 7) template/generic instantiation of `suite` from here
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/tests/test1.nim(26, 8) template/generic instantiation of `test` from here
  /Users/runner/work/Nim/Nim/pkgstemp/argparse/tests/test1.nim(27, 22) template/generic instantiation of `newParser` from here
  /Users/runner/work/Nim/Nim/lib/pure/strutils.nim(2873, 11) Error: attempt to access a nil address kind: rkInt
         Tip: 1 messages have been suppressed, use --verbose to show them.
       Error: Execution failed with exit code 256
          ... Command: /Users/runner/work/Nim/Nim/bin/nim c --noNimblePath -d:NimblePkgVersion=1.0.0 --hints:off -r --path:. /Users/runner/work/Nim/Nim/pkgstemp/argparse/tests/test1
  
  PASS: arraymancer C                                                (35.84 sec)

Additional Information

in unrelated PR https://github.com/nim-lang/Nim/pull/16166/checks?check_run_id=1466652591

timotheecour added a commit to timotheecour/Nim that referenced this issue Nov 28, 2020
@ringabout
Copy link
Member

ringabout commented Nov 28, 2020

This is caused by:
#15939
Every ^1 usage for string in VM may be affected.

@timotheecour
Copy link
Member Author

timotheecour commented Nov 28, 2020

right, i just noticed too: here's git bisect output:

527e792827497002ded742ad815c567a4a056471 is the first bad commit
commit 527e792827497002ded742ad815c567a4a056471
Author: hlaaftana <[email protected]>
Date:   Thu Nov 12 13:44:21 2020 +0300

    make var string return var char w/ BackwardsIndex (#15461)

    * make var string return var char w/ BackwardsIndex

    fixes #14497

    * work around VM bug
    * properly workaround again

 lib/pure/pathnorm.nim    | 6 +++---
 lib/system.nim           | 1 +
 tests/stdlib/tstring.nim | 7 +++++++
 3 files changed, 11 insertions(+), 3 deletions(-)
bisect run success

my PR #15987 fixes that

timotheecour added a commit to timotheecour/Nim that referenced this issue Nov 28, 2020
Araq pushed a commit that referenced this issue Dec 3, 2020
…s `addr(mystring[ind])` (index + index assignment) (#15987)

* fix #15939, fix #15464 VM now supports `addr(mystring[ind])` (index + index assignment), var char return etc
* cleanups
* cstring tests
* add test for bug #15464
* improve test coverage
mildred pushed a commit to mildred/Nim that referenced this issue Jan 11, 2021
nim-lang#16226 VM now supports `addr(mystring[ind])` (index + index assignment) (nim-lang#15987)

* fix nim-lang#15939, fix nim-lang#15464 VM now supports `addr(mystring[ind])` (index + index assignment), var char return etc
* cleanups
* cstring tests
* add test for bug nim-lang#15464
* improve test coverage
ardek66 pushed a commit to ardek66/Nim that referenced this issue Mar 26, 2021
nim-lang#16226 VM now supports `addr(mystring[ind])` (index + index assignment) (nim-lang#15987)

* fix nim-lang#15939, fix nim-lang#15464 VM now supports `addr(mystring[ind])` (index + index assignment), var char return etc
* cleanups
* cstring tests
* add test for bug nim-lang#15464
* improve test coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants