Skip to content

Commit

Permalink
scripts: xtensa-lx106 compiler no longer supports debugging?
Browse files Browse the repository at this point in the history 
I'm getting this error:

	xtensa-lx106-elf-gcc: warning: target system does not support debug output

which seems rather odd. I guess we'll try disabling debug mode and see if that helps.

Signed-off-by: Keith Packard <keithp@keithp.com>
  • Loading branch information
@keith-packard
keith-packard committed Jan 23, 2023
1 parent c4bc2d0 commit 711dbd2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/do-lx106-configure
View file
Expand Up @@ -33,4 +33,4 @@
# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
# OF THE POSSIBILITY OF SUCH DAMAGE.
#
exec "$(dirname "$0")"/do-configure xtensa-lx106-elf "$@"
exec "$(dirname "$0")"/do-configure xtensa-lx106-elf "$@" -Ddebug=false

4 comments on commit 711dbd2

@paravoid
Copy link

@paravoid paravoid commented on 711dbd2 Mar 11, 2023 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming this was on Debian with a version between 12.2.0-9+12 and 12.2.0-14+13: see https://bugs.debian.org/1032564 which was fixed just now.

@keith-packard
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sorry for dropping the ball here; I should have filed a bug upstream too. When this hits testing, I'll revert this commit and poke CI to regenerate the toolchain image it uses and see if it works.

@paravoid
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sorry for dropping the ball here; I should have filed a bug upstream too. When this hits testing, I'll revert this commit and poke CI to regenerate the toolchain image it uses and see if it works.

FYI this has now hit testing :)

@keith-packard
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pr #455 -- we'll see if this works now.

Please sign in to comment.