Skip to content

add LIBSEARPC_ASYNC_API_WORKS define#81

Open
stsp wants to merge 1 commit into
haiwen:masterfrom
stsp:async_id
Open

add LIBSEARPC_ASYNC_API_WORKS define#81
stsp wants to merge 1 commit into
haiwen:masterfrom
stsp:async_id

Conversation

@stsp

@stsp stsp commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Commit ed7b992 fixed async API, but now the application needs a check that the API is usable.

Add LIBSEARPC_ASYNC_API_WORKS define.

Commit ed7b992 fixed async API, but now the application needs
a check that the API is usable.

Add LIBSEARPC_ASYNC_API_WORKS define.
@stsp

stsp commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Guys, its absolutely problematic
to use libsearpc w/o this define,
because older versions didn't free
private data in searpc_client_generic_callback(),
and now they do again, as before
ed7b992
One can't make a reliable work-around
for all versions, so please, add
this define.

stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
Due to haiwen/libsearpc#81

While configure check was added in the prev commit, it doesn't work
when cross-compiling.
stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
Due to haiwen/libsearpc#81

While configure check was added in the prev commit, it doesn't work
when cross-compiling.
stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
libsearpc upstream applied
haiwen/libsearpc#79
but not
haiwen/libsearpc#81
which made it impossible to detect if the lib is fixed or not.
The fix itself was backward-incompatible with any work-around, so using
the lib is no longer possible at all because you never know if it needs
a work-around or not.

The simplest fix is to fork the broken searpc-client.c and add a fixed
version of it into the build.
stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
libsearpc upstream applied
haiwen/libsearpc#79
but not
haiwen/libsearpc#81
which made it impossible to detect if the lib is fixed or not.
The fix itself was backward-incompatible with any work-around, so using
the lib is no longer possible at all because you never know if it needs
a work-around or not.

The simplest fix is to fork the broken searpc-client.c and add a fixed
version of it into the build.
stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
libsearpc upstream applied
haiwen/libsearpc#79
but not
haiwen/libsearpc#81
which made it impossible to detect if the lib is fixed or not.
The fix itself was backward-incompatible with any work-around, so using
the lib is no longer possible at all because you never know if it needs
a work-around or not.

The simplest fix is to fork the broken searpc-client.c and add a fixed
version of it into the build.
stsp added a commit to dosemu2/dosemu2 that referenced this pull request Jul 23, 2026
libsearpc upstream applied
haiwen/libsearpc#79
but not
haiwen/libsearpc#81
which made it impossible to detect if the lib is fixed or not.
The fix itself was backward-incompatible with any work-around, so using
the lib is no longer possible at all because you never know if it needs
a work-around or not.

The simplest fix is to fork the broken searpc-client.c and add a fixed
version of it into the build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant