Skip to content
This repository has been archived by the owner on Sep 22, 2022. It is now read-only.

Devel #145

Merged
merged 12 commits into from
Jan 7, 2019
Merged

Devel #145

merged 12 commits into from
Jan 7, 2019

Conversation

tnt4brain
Copy link

This is a bootstrapping .spec file for building a complete RPM package for CentOS 7 Could you please review it?

@erthink
Copy link
Owner

erthink commented Jan 11, 2018

I apologize for the delay. I am so busy, but hope in the near future your improvements will ne taken.

@erthink
Copy link
Owner

erthink commented Jan 18, 2018

Приветствую.

Я постараюсь влить pull-request в выходные.

Пока же могу высказать замечания:

  1. Во многих местах s/openldap/reopenldap/.
  2. Должно быть reldap-devel, а не ldap-devel:
    • текущий состав библиотек не совместим с ldap-devel из opendap;
    • все библиотеки слиты в libreldap;
    • изменено API некоторых функций...;
  3. Непосредственно сейчас reldap-devel не следует собирать:
    • будет рефакториг для Refactoring API of libreldap to avoid false-positives from Coverity #142 с потерей совместимости по API;
    • в следствии Migration to the current version libmdbx. #141 будет изменено API связанное с пулом тредов, что также сломает совместимость;
    • маловероятно что reldap-devel сейчас действительно кому-то нужен, но это может иметь смысл после двух предыдущих пунктов;
    • поэтому я бы предпочел не собирать reldap-devel, а буквально закоментировать это в спеке добавив это пояснение.

@tnt4brain
Copy link
Author

tnt4brain commented Jan 6, 2019

Happy New Year!
Codetree updated to reflect that force-push last fall, so it can be merged to upstream.

С Новым годом!
Код обновлён в соответствии с осенним force-push'ем, так что может быть влит в основной репозиторий.

@erthink erthink merged commit 6ca54d7 into erthink:devel Jan 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants