Re: [Rpm-maint] [rpm-software-management/rpm] Improved python support: enums are now enums with all additional goodies. (#1309)

2020-08-05 Thread KOLANICH
1 Is there any reason to define the enums in C code instead of Python? 2 Should I move them into a python file and replace by pythonic enums then, breaking the compatibility in order to declutter the namespace? -- You are receiving this because you are subscribed to this thread. Reply to this

Re: [Rpm-maint] [rpm-software-management/rpm] Improved python support: enums are now enums with all additional goodies. (#1309)

2020-08-05 Thread KOLANICH
So we can't declutter and eliminate the duplication. IMHO it makes no sense to make the cluttered enums into python enums because they are a bit faster, so the dev can decide what he needs more, convenience to debug or performance and can switch between them by using the appropriate ctors if

Re: [Rpm-maint] [rpm-software-management/rpm] Improved python support: enums are now enums with all additional goodies. (#1309)

2020-08-05 Thread Panu Matilainen
Um, what? We can't de-clutter what's there because it would break every single python-rpm user. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] OpenMP & Lua fixes for configure.ac & INSTALL (#1325)

2020-08-05 Thread Michal Domonkos
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/1325 -- Commit Summary -- * Check document need for correct OpenMP version * Bump Lua to 5.2 in configure script -- File Changes -- M INSTALL (6) M configure.ac

Re: [Rpm-maint] [rpm-software-management/rpm] packageBinaries compile error when building rpm-4.16.0-beta3 from source (#1315)

2020-08-05 Thread Michal Domonkos
Thanks for reporting this, @pixdrift. While the only solution for you is what Panu outlined above, I have at least updated the configure script so that it checks for the actual OpenMP version in the compiler, and also added a short section into the INSTALL file which states the required