Remove log4net, newtonsoft deps from deb package #3900
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
Trying to install the .deb on Debian 12 fails with a message indicating log4net is missing:
Cause
The latest release of Debian seems to no longer include this package; "bookworm" doesn't show up in the package search listing:
https://packages.debian.org/search?searchon=names&keywords=liblog4net1.2-cil
However, our build process repacks log4net into
ckan.exe
, so we shouldn't need it to be installed in the OS:Changes
Now log4net and newtonsoft are no longer listed as requirements for the .deb, because they are both repacked into the
ckan.exe
assembly. This should allow the .deb to be installed on Debian 12 and any other .deb-based OS that drops this package.Fixes #3896.