Re: [Rpm-maint] [rpm-software-management/rpm] Switch most of rpm over to C++ (PR #3028)

2024-04-09 Thread Panu Matilainen
Merged #3028 into master. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/3028#event-12402731645 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint

Re: [Rpm-maint] [rpm-software-management/rpm] Switch most of rpm over to C++ (PR #3028)

2024-04-09 Thread Panu Matilainen
:see_no_evil: -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/3028#issuecomment-2044380986 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing

Re: [Rpm-maint] [rpm-software-management/rpm] Switch most of rpm over to C++ (PR #3028)

2024-04-09 Thread Panu Matilainen
@pmatilai pushed 1 commit. 4e5b6d0c8886376d6e1ccc90e80db80f0953d3a5 Remove the WITH_CXX option, this is a one-way street -- View it on GitHub: https://github.com/rpm-software-management/rpm/pull/3028/files/70505942562da081e207ea908e090bc7e1c095ff..4e5b6d0c8886376d6e1ccc90e80db80f0953d3a5 You

Re: [Rpm-maint] [rpm-software-management/rpm] Switch most of rpm over to C++ (PR #3028)

2024-04-09 Thread Panu Matilainen
This is (very) obviously a case of "things will get worse before they get better", these casts are just unsightly but most will go away once converted to actual native C++. -- Reply to this email directly or view it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] Switch most of rpm over to C++ (PR #3028)

2024-04-09 Thread Panu Matilainen
As per https://github.com/rpm-software-management/rpm/discussions/2983, here goes. More details in commits, but to summarize: add necessary casts to most of the codebase to make it buildable with a c++ compiler, and then switch over to c++. Exempt from this conversion are - Python bindings: