Package Details: dosbox-debug 0.74.3-1

Git Clone URL: https://aur.archlinux.org/dosbox-debug.git (read-only, click to copy)
Package Base: dosbox-debug
Description: Emulator with builtin DOS for running DOS games (includes debugger for DOS apps)
Upstream URL: http://dosbox.sourceforge.net/
Licenses: GPL
Conflicts: dosbox
Provides: dosbox
Submitter: None
Maintainer: Malvineous
Last Packager: Malvineous
Votes: 3
Popularity: 0.153695
First Submitted: 2009-02-06 14:38 (UTC)
Last Updated: 2022-06-12 04:51 (UTC)

Required by (67)

Sources (2)

Latest Comments

adriaticc commented on 2024-05-31 05:53 (UTC) (edited on 2024-05-31 10:12 (UTC) by adriaticc)

This build will fail, complaining of access to begy members of a struct WINDOW which has no definition. Ncurses is typically built in a way which makes that structure an opaque implementation detail, so the following patch makes use of the recommended access pattern:

--- dosbox-0.74-3/src/debug/debug_gui.cpp   2019-05-02 10:01:19.000000000 -0700
+++ ../debug_gui.cpp    2024-05-30 22:45:46.306714061 -0700
@@ -146,15 +146,15 @@
        attrset(COLOR_PAIR(PAIR_BLACK_BLUE));
    }
    /* Show the Register bar */
-   mvaddstr(dbg.win_reg->_begy-1,0, "---(Register Overview                   )---");
+   mvaddstr(getbegy(dbg.win_reg)-1,0, "---(Register Overview                   )---");
    /* Show the Data Overview bar perhaps with more special stuff in the end */
-   mvaddstr(dbg.win_data->_begy-1,0,"---(Data Overview   Scroll: page up/down)---");
+   mvaddstr(getbegy(dbg.win_data)-1,0,"---(Data Overview   Scroll: page up/down)---");
    /* Show the Code Overview perhaps with special stuff in bar too */
-   mvaddstr(dbg.win_code->_begy-1,0,"---(Code Overview   Scroll: up/down     )---");
+   mvaddstr(getbegy(dbg.win_code)-1,0,"---(Code Overview   Scroll: up/down     )---");
    /* Show the Variable Overview bar */
-   mvaddstr(dbg.win_var->_begy-1,0, "---(Variable Overview                   )---");
+   mvaddstr(getbegy(dbg.win_var)-1,0, "---(Variable Overview                   )---");
    /* Show the Output OverView */
-   mvaddstr(dbg.win_out->_begy-1,0, "---(OutPut/Input    Scroll: home/end    )---");
+   mvaddstr(getbegy(dbg.win_out)-1,0, "---(OutPut/Input    Scroll: home/end    )---");
    attrset(0);
 }

Forks like dosbox-x seem to have kludged this too.

Malvineous commented on 2018-07-29 03:28 (UTC)

It was named "dosbox-debug" because that's the name of the command you run. I figured it would be more confusing if the package name was different to what gets installed.

The reason for compiling both was because I was hoping at one point that it could be incorporated into the official Arch repos to make it easier to install the debug version, but that plan didn't eventuate. I think it is also better to keep the versions consistent to ensure the debug version is always the same as the normal version, otherwise you risk inconsistencies when switching between the two (e.g. since they can read different config files based on their version numbers.)

eschwartz commented on 2018-07-05 23:26 (UTC)

Wouldn't it make more sense to only package the debugger build, since the rest is identical to the dosbox package from the official repos?

You could either depend on the official package or provide additional copies of the pixmap and manpage forked to refer to the debug build...

I'd also suggest renaming the package to "dosbox-debugger" since it is more accurate.