desktop-apps
desktop-apps copied to clipboard
RPM/Deb packaging fixes
Здравствуйте!
Вам по определенным каналам был передан PDF-документ с критическими и не очень замечаниями по упаковке R7 в RPM, не знаю, дошел ли он до разработчиков. Судя по тому, что я там ошибся (показалось, что вы своими MIME-типами подменяете стандартный тип для docx, хотя вы просто создаете свой тип (https://www.onlyoffice.com/blog/2022/03/onlyoffice-forms-new-oform-docxf-formats-explained/)), а обратной связи не было, могу предположить, что его и не читали).
В этом пулл-реквесте на глаз исправлены некоторые моменты, удалось добиться отсутствия внесения изменений в ОС скриптлетами пакетов в обход пакетной системы. На глаз, потому что некогда пытаться собрать это чудо с нуля, свои правки не проверял.
В описании коммитов, где нужно, расписано подробнее, что сделано.
Из на мой взгляд критических проблема остается использования LD_PRELOAD: запущенный из-под офиса, например, браузер и запущенные из-под него программы, например, VLC, будут наследовать эту переменную. Нужно заменить на RPATH. Лучше всего флагами компилятора, а можно и утилитой patchelf.
Не могу подписать CLA, вот такая пустая картинка по ссылке. Считайте, что я не против передать все права на написанное вам.
ping
Hi @mikhailnov, please use english for pr and comments. Also I can't merge your PR cause there conflicts on branch and you don't sign CLA yet.
Ok. Commit messages are in English. I've signed the CLA, this time it seems to have succeeded.
Там ваши коллеги по Р7 уже сделали спец. сборку с учетом этих замечаний и даже исправленных rpath, этот пулл-реквест особого смысла не имеет, вы бы что ли их правки в основную ветку влили, тогда пой PR смысла не имеет. Поверх нет, может быть, найду, что поправить.
Closed due to inability to merge fixes.
Do you have plans to merge similar fixes from special r7 branch? If not, I can make a new PR.