https://bugs.openldap.org/show_bug.cgi?id=10137
Issue ID: 10137 Summary: Ease redefining the MDB_IDL_LOGN value Product: LMDB Version: 0.9.30 Hardware: All OS: All Status: UNCONFIRMED Keywords: needs_review Severity: normal Priority: --- Component: liblmdb Assignee: bugs@openldap.org Reporter: renault.cle@gmail.com Target Milestone: ---
Created attachment 992 --> https://bugs.openldap.org/attachment.cgi?id=992&action=edit The patch do apply to midl.h
Hello,
I would like to slightly change the midl.h c file to make it possible to change the MDB_IDL_LOGN define without having to fork on my side. I know it can be redefined to reduce the amount of memory allocated by LMDB. I am using the latest `mdb.master` branch version of LMDB.
https://github.com/mozilla/lmdb/pull/2
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #1 from Howard Chu hyc@openldap.org --- Since you are not the original author of this patch, we cannot accept it as-is.
It's a simple enough patch, but you should resubmit a different one written by yourself, and with the requisite IPR notice attached, as documented in the OpenLDAP Contributing guidelines.
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #2 from kero renault.cle@gmail.com --- Created attachment 993 --> https://bugs.openldap.org/attachment.cgi?id=993&action=edit The patch to allow users to define MDB_IDL_LOGN
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #3 from kero renault.cle@gmail.com --- Here's a patch (in git-format-patch format, per the Guidelines for Contributing) that changes the code to allow users to define MDB_IDL_LOGN.
(I haven't provided a Notice of Origin nor Rights Statement, as, per the Guidelines for Contributing, this patch doesn't represent "significant blocks of new code (10 lines or greater)." It's a one-line change.)
Thank you very much, please tell me if you need anything else.
https://bugs.openldap.org/show_bug.cgi?id=10137
Howard Chu hyc@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED
--- Comment #4 from Howard Chu hyc@openldap.org --- In git mdb.master / mdb.master3
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #5 from kero renault.cle@gmail.com --- Thank you very much, Howard. Nice to see this reaction time!
https://bugs.openldap.org/show_bug.cgi?id=10137
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |0.9.32 Keywords|needs_review |
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #6 from Quanah Gibson-Mount quanah@openldap.org --- mdb.master:
• 30288c72 by Clément Renault at 2023-11-27T16:24:43+00:00 ITS#10137 LMDB: Allow users to define MDB_IDL_LOGN
mdb.master3:
• 22a41169 by Clément Renault at 2023-11-27T16:25:15+00:00 ITS#10137 LMDB: Allow users to define MDB_IDL_LOGN
https://bugs.openldap.org/show_bug.cgi?id=10137
--- Comment #7 from Quanah Gibson-Mount quanah@openldap.org --- mdb.RE/0.9:
• 95670e37 by Clément Renault at 2024-01-16T20:30:01+00:00 ITS#10137 LMDB: Allow users to define MDB_IDL_LOGN
-
openldap-its＠openldap.org