MajorDoMo

Здравствуйте!
У меня несколько "неудобных" вопросов по программному коду Majordomo.
Сразу отмечу что являюсь давним разработчиком на php/ruby/js.

Начну с проблем, которые увидел сразу:

1) Невозможно запустить Majordome, если брать исходный код проекта с github.com. Вообще нет инсталляции первоначальных таблиц бд (например settings). Однако /load_settings.php и /test.php упорно пытаются сделать выборку из таблицы settings, при ее отсутствии, и никак не обрабатывая проблему. Все это еще утяжеляется тем что при возникновении ошибки при работе с бд, вызывается регистрации ошибки (/lib/mysql.class.php, строки 132 и 270) и попытка опять же записать это в бд, все это вызывает рекурсию и как результат segmentation fault в php.

2) Зачем такая "свалка" папок в корне (в DOCUMENT_ROOT)? Ведь можно все грамотно/удобно содержать в папках vendor (для внешнего кода типа bootstrap, highlights, jquery-mobile, etc) и в assets (или media) для js/css/ttf и прочее.

3) Почему в Majordomo часть рабочих ссылок (адресов/вызовов) со слешем на конце, а часть с .html на конце? Нет четкой логики. И нельзя посмотреть все рабочие "роутинги".

4) Почему нет сборщика в единые файлы js и css? Хотя бы сборщика. Это бы убрало "портянки" в шаблонах. И так же бы увеличило производительность.

5) Зачем использовать/советовать mplayer (с кучей зависимостей)? Все работает с aplay/mpg123. Кстати, для VLC лучше использовать пакет vlc-nox - без зависимостей пакетов для X-сервера.

6) При первом старте можно проверять все что необходимо Majordomo для работы - модули веб-сервера (различные), работу htaccess (или другого rewrite), правильность работы бд, права на папки и файлы. Все это уберет многие повторяющиеся проблемы, которые постоянно всплывают на форуме.

Здравствуйте,

Вопросы риторические из серии "почему так, если можно лучше"
Отмечу, что очень давно разработка не моё основное занятие, поэтому красота кода на уровне времён бума дот комов. Это не оправдание, просто лично у меня есть определённые приоритеты в достаточно ограниченном временном ресурсе. Нисколько ни умаляю важность более грамотной организации, но, боюсь, она моими силами не будет реализовано ибо их просто на всё не хватит (в контексте всех моих проектов, а не только конкретно этого). Если есть желание и возможность, то, как говорится, "welcome on board" -- для того и исходники все на github-е.

sergejey писал(а):Вопросы риторические из серии "почему так, если можно лучше"

Вообще-то это как-то похоже на "отмазку". Особенно в вопросе под номером 1 (про бд и segmentation fault).
Типо "леплю как попало, без тестов, не серчайте, не профи".

Продукт только тогда привлечет граммотных разработчиков, когда в продукте не будет таких "камней" как в вопросе про бд и про установку/проверку. И Вы, как зачинатель продукта, должны это понимать.
Разработчик, который заинтересовался проектом (в силу разных причин), должен спокойно взять код проекта с github, почитать readme и развернуть проект. А не вычитывать форум, не рыскать по сайту, и не догадываться что недостающие таблицы бд нужно взять из архива с сайта. Это нонсенс для хорошего открытого проекта.

Теперь у меня вопросы иного плана.

- Вы исправите (и когда) проблему/баг, которую описал я в пункте 1? Или примите merge request как можно быстрее?

- Что будет если (допустим я) приведу код Majordomo к четкой структуре и вызовам?

- Кто конкретно будет принимать решение о merge в основную ветку?

- Будете ли Вы как создатель/разработчик следовать внесенным изменениям и новому code style?

igk1972 писал(а):Вообще-то это как-то похоже на "отмазку"

Не похоже. Я не пытаюсь оправдаться, не вижу в этом необходимости.

igk1972 писал(а):Теперь у меня вопросы иного плана.
- Вы исправите (и когда) проблему/баг, которую описал я в пункте 1?

Да, когда до неё дойдёт очередь.

igk1972 писал(а): - Что будет если (допустим я) приведу код Majordomo к четкой структуре и вызовам?

Код Majordomo станет с чёткой структурой и вызовами.

igk1972 писал(а): - Кто конкретно будет принимать решение о merge в основную ветку?

Конкретно я.

igk1972 писал(а): - Будете ли Вы как создатель/разработчик следовать внесенным изменениям и code style?

Буду стараться

Я, признаться, не очень понимаю сути претензий. Да, можно сделать лучше, ну ок.

sergejey писал(а):

igk1972 писал(а):Теперь у меня вопросы иного плана.
- Вы исправите (и когда) проблему/баг, которую описал я в пункте 1?

Да, когда до неё дойдёт очередь.

А конкретно? Это же баг, а не функция.
И где можно ознакомится с текущим todo или timeline release?

sergejey писал(а):

igk1972 писал(а): - Кто конкретно будет принимать решение о merge в основную ветку?

Конкретно я.
Я, признаться, не очень понимаю сути претензий. Да, можно сделать лучше, ну ок.

Вот это очень настораживает

igk1972 писал(а):А конкретно? Это же баг, а не функция.
И где можно ознакомится с текущим todo или timeline release?

Это не приоритетный баг в виду того, что касается в основном разработчиков, а не конечных пользователей.
todo-листа и timeline release нет. точнее, нет публичного ибо он не связан с кодом как таковым.

igk1972 писал(а):

sergejey писал(а):

igk1972 писал(а): - Кто конкретно будет принимать решение о merge в основную ветку?

Конкретно я.
Я, признаться, не очень понимаю сути претензий. Да, можно сделать лучше, ну ок.

Вот это очень настораживает

Настороженность это не то чувство, с которым имеет смысл подходить к активному участию. Если действительно хотите (в первую очередь) пользоваться системой и (во вторую очередь) помочь с развитием, то было бы здорово подойти к этому с некоторым уважением и, возможно, пониманием обстоятельств эволюции проекта.

sergejey писал(а):

igk1972 писал(а):А конкретно? Это же баг, а не функция.
И где можно ознакомится с текущим todo или timeline release?

Это не приоритетный баг в виду того, что касается в основном разработчиков, а не конечных пользователей.

То есть Вы не допускаете мысли что конечный "пользователь" может устанавливать Majordomo с github?
Или желать сделать deb-пакет для своего дистрибутива, а в пакетах не должно быть установки руками бд и прочего - все автоматически или через first boot мастера.

sergejey писал(а):todo-листа и timeline release нет. точнее, нет публичного ибо он не связан с кодом как таковым

То есть я как разработчик, желающий помочь, должен только догадываться о планах развития Majordomo.


Про упомянутый баг. Вам только кажется что он не касается конечного пользователя. Пример:
- пользователь устанавливает Majordomo на раздел где (вдруг) мало свободного места (такое вполне реально).
- при нехватке места mysql отпадает с ошибками, либо вообще падает, но без рестарта.
- majordomo же "работает" без бд, с постоянными segmentation fault на уровне модуля apachе, или процесса php-fpm.
- пользователь вообще ничего не видит, так как Majordomo вообще не отдает ни одной страницы (error 500).

После этого будете продолжать утверждать что баг второстепенный?

Ок, убедили, баг важный. Спасибо за помощь.

Чёт я вообще не понимаю смысла "наезда" на Сергея.
Уважаемый, igk1972, в случае если Вы так щепетильно относитесь к подобного рода "багам"(из Ваших слов), почему тогда Вы присутствуете здесь? Найдите проект без изъянов и бесплатно. Есть сомнения по этому поводу... Я уверен что 99% людей которые пользуются данным проектом, считают его самым лучшим и гибким в своей области. Для них тот "баг", как Вы его называете, и не баг вовсе. Все действуют согласно чёткой инструкции, а тот кто хочет поэкспериментировать, берёт и экспериментирует, а не пытается как-то рассказать разработчику что "он живёт не правильно". И в случае если нуждается, в помощи то он приходит на форум и спрашивает об этом, и все любезно стараются ему помочь, в том числе и разработчик лично.
ИМХО. Подобные "наезды" только портят ситуацию в целом, и после подобного рода заявлений со стороны потребителя, разработчик просто отказывается продолжать поддерживать проект и развивать его. Это я сейчас не про Сергея и MDM, а в целом по личным ситуациям.
И повторюсь словами Сергея, на то и исходники на Git-е что бы каждый желающий внёс свою лепту.
А то, что кому-то там впадлу увидеть ссылку на сайт в файле: "README.md", и перейти по ней, и уже на сайте зайти на страницу: "Установка\Настройка", так это его личные проблемы. 80% пользователей MDM устанавливают оную по странице которую я указал ранее. И не сталкиваются с трудностями. А если человеку впадлу глянуть на форум, где всегда помогут, повторюсь "...так это его личные проблемы".
Не знаю на сколько в реальности обоснованы "претензии" которые Вы предъявляете Сергею, но я лично считаю что уж если Вы такой грамотный(без иронии), так в чём проблема? Дерзайте! Я лично желаю Вам только успехов! И замечу что это не сарказм.

warik писал(а):Чёт я вообще не понимаю смысла "наезда" на Сергея.
Уважаемый, igk1972, в случае если Вы так щепетильно относитесь к подобного рода "багам"(из Ваших слов), почему тогда Вы присутствуете здесь? Найдите проект без изъянов и бесплатно. Есть сомнения по этому поводу... Я уверен что 99% людей которые пользуются данным проектом, считают его самым лучшим и гибким в своей области. Для них тот "баг", как Вы его называете, и не баг вовсе. Все действуют согласно чёткой инструкции, а тот кто хочет поэкспериментировать, берёт и экспериментирует, а не пытается как-то рассказать разработчику что "он живёт не правильно". И в случае если нуждается, в помощи то он приходит на форум и спрашивает об этом, и все любезно стараются ему помочь, в том числе и разработчик лично.

Никакого "наезда" не было. Оставьте эти домыслы при себе.

Было четкое указание на проблему. И описание возникающей ситуации. И пояснение, после непонимания картины в целом.
Так же подмечены недостатки, увиденные сразу. Как раз в помощь проекту. Как взгляд со стороны, который полезен при малом количестве разработчиков.

То что у проекта не было code review - это и так видно. И Сергей это не отрицает.

Кроме того, нормальная мысль - узнать мнение главного разработчика на указанные проблемы, прежде чем начинать работы по изменению/переделке проекта, и их открытой публикации.