Skip to content

restored test/debugger-mi#574

Closed
paolopas wants to merge 1 commit intobfgroup:mainfrom
paolopas:debby
Closed

restored test/debugger-mi#574
paolopas wants to merge 1 commit intobfgroup:mainfrom
paolopas:debby

Conversation

@paolopas
Copy link
Copy Markdown
Contributor

also fixed typo in debugger.cpp break help

It's always an honor to get your hands on Watanabe's stuff.

@grafikrobot
Copy link
Copy Markdown
Member

Running this locally, after rebasing on latest main commit, I get an std assert (for debug b2 build):

/usr/include/c++/15.2.1/bits/stl_vector.h:1263: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](size_type) [with _Tp = char*; _Alloc = std::allocator<char*>; reference = char*&; size_type = long unsigned int]: Assertion '__n < this->size()' failed.

And running the not-debug B2 core dumps. Can you rebase this and investigate?

@grafikrobot grafikrobot moved this to 👀 In review in BFG Tasks Apr 3, 2026
fixed typo in debugger.cpp break help
@paolopas
Copy link
Copy Markdown
Contributor Author

paolopas commented Apr 3, 2026

I tried using both the topic (rebased) branch and the current main (with changes plugged).
I tried to compile with my make and with b2 (both release and debug).

But no diagnostic messages or other issues.

@paolopas
Copy link
Copy Markdown
Contributor Author

paolopas commented Apr 3, 2026

But looking closely, the b2 build does do one bad thing. And that also explains the problems we've had in the past.

$ ./src/engine/b2 b2 debug
...patience...
...found 307 targets...
...updating 74 targets...
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/src/engine/w32_getreg.o

. . .

gcc.link .build/gcc-10/debug/cxxstd-11-iso/threading-multi/yyacc
Jamfile</home/pax/Job/B2/github/b2>.yyacc src/engine/jamgram.y
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/src/engine/tasks.o
Jamfile</home/pax/Job/B2/github/b2>.grammar src/engine/jamgram.cpp
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/src/engine/startup.o
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/src/engine/rules.o
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/src/engine/scan.o
gcc.compile.c++ .build/gcc-10/debug/cxxstd-11-iso/threading-multi/jamgram.o

. . .

gcc.link .build/gcc-10/debug/cxxstd-11-iso/threading-multi/b2

...updated 74 targets...
$ git status
On branch debby
Your branch is ahead of 'origin/debby' by 2 commits.
  (use "git push" to publish your local commits)

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   src/engine/jamgram.cpp
	modified:   src/engine/jamgram.hpp
no changes added to commit (use "git add" and/or "git commit -a")
$ git diff
diff --git a/src/engine/jamgram.cpp b/src/engine/jamgram.cpp
index afc54d43e..bea45c0c8 100644
--- a/src/engine/jamgram.cpp
+++ b/src/engine/jamgram.cpp
@@ -1,8 +1,8 @@
-/* A Bison parser, made by GNU Bison 3.8.2.  */
+/* A Bison parser, made by GNU Bison 3.5.  */
 
 /* Bison implementation for Yacc-like parsers in C
 
-   Copyright (C) 1984, 1989-1990, 2000-2015, 2018-2021 Free Software Foundation,
+   Copyright (C) 1984, 1989-1990, 2000-2015, 2018-2019 Free Software Foundation,
. . .

But apart from the modification of the jamgrams.* which is difficult to even notice, there were no problems.

@grafikrobot
Copy link
Copy Markdown
Member

But apart from the modification of the jamgrams.* which is difficult to even notice, there were no problems.

Oh, fun. I guess something in my system is making it behave differently enough. :-)

@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in BFG Tasks Apr 9, 2026
@paolopas paolopas deleted the debby branch April 10, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants