Skip to content

chore(windows): remove C# build solutions and projects - #130

Merged
gburd merged 1 commit into
masterfrom
chore/remove-csharp-vcxproj
Aug 3, 2026
Merged

chore(windows): remove C# build solutions and projects#130
gburd merged 1 commit into
masterfrom
chore/remove-csharp-vcxproj

Conversation

@gburd

@gburd gburd commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Final C# footprint: the .NET/C# Visual Studio solutions (BDB_dotNet*.sln, BDB_dotNet_examples*.sln) and the C# library projects (VS10/db_csharp.vcxproj, VS8/db_csharp.vcproj). These build the dropped C# binding. The C library solution Berkeley_DB_vs2010.sln (what Windows CI builds) has zero references to them and is unaffected — the windows msbuild job on this PR confirms it. Completes the C# removal in build_windows/.

The .NET/C# Visual Studio solutions (BDB_dotNet*.sln, BDB_dotNet_examples*.sln)
and the C# library projects (VS10/db_csharp.vcxproj, VS8/db_csharp.vcproj) build
the dropped C# binding. Removed. The C library solution
(Berkeley_DB_vs2010.sln, what Windows CI builds) has zero references to these and
is unaffected. Completes the C# removal in build_windows/.
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

Coccinelle convention checks

No new violations. ✅

Resolved since baseline (2) -- update dist/cocci/baseline.txt to lock these in.
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/crypto/mersenne/mt19937db.c|return (ret);
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/mp/mp_register.c|return (ret);

@gburd
gburd merged commit 6db539c into master Aug 3, 2026
47 of 48 checks passed
@gburd
gburd deleted the chore/remove-csharp-vcxproj branch August 3, 2026 15:45
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

ABI diff vs v5.3.33 (libabigail — authoritative)

Removed exported symbols (nm -D, _NNNN version suffix normalized)

None.


Advisory: libabigail/nm is the authoritative binary-ABI check; Coccinelle is complementary source-level early warning. See dist/cocci/README.md.

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