LINUXTALKS.CO

вопрос синхронизации с апстримом

 

L


1

1

смотрите, на оригинальнмом лоре появился разделитель в треде по дням! эта фича закомичена после нашего форка. @cocucka, отсюда вопрос, как обстоят у нас дела с синхронизацией с апстримом? хочу такие же некромаркеры! я не могу их просто так перенести, т.к. maxcom постепенно переписывает весь код с java на scala. мне удается бекпортировать изменения из апстрима maxcom, но у нас они не собираются с ошибкой (-ами) скалы.

★★☆☆☆

cocucka> С апстримом я не буду синхронизироваться. Слишком много геморроя с интеграцией изменений. Мб портирую интересные штуки, если они не совсем в разрез с моим кодом будут.

такое твое мнение?

crypt    
★★☆☆☆
FreeBSD / Chrome

Мастхев фичи (клуб и блоги) я запилю в ближайшие дни

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

p.s.

да, я лично maxcomовский scala понимаю еще хуже, чем java. но я все-таки подниму этот вопрос.

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome

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

Банально, если он выкатит изменение в DDL и я выкачу свои, то миграционные патчи уже будут конфликтовать.

cocucka    
★★★★★★★★★★
Linux / Chrome

Ты уж определись, либо крестик снимать и тупо держать переименованный ЛОР, либо трусы надеть и допиливать движок до нормального состояния.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от cocucka

отсюда есть два следствия:

а) можно дропнуть все то легаси, что накопилось, вроде кучи каких-то SQL данных и тестов.

б) встает вопрос в сабже. вот мы видим классную фичу в апстриме. сам ты ее не будешь делать, потому что некогда. он ее уже сделал. в итоге проект теряет.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

так вот у меня к тебе вопрос и есть: что мы делаем-то? до нормального состояния мы его не допиливаем, потому что все, что я вижу на текущий момент - это хаки и замена хардкода.

при этом синхронизация с апстримом и классные фичи оттуда тоже отсутствуют.

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 3)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

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

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

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

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 3)

FreeBSD / Chrome
Ответ на: комментарий от crypt

вот мы видим классную фичу в апстриме

Не увидишь ты там особо классных фич. Ту хрень с датой я могу добавить легко, а срань типа постинга в телеграм топ-10 постов и ограничение по странам нахрен не нужна.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от cocucka

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

вопрос, будешь ли ты это вообще делать через год, как и мелкие бампы версий и прочее.

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

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 2)

FreeBSD / Chrome
Ответ на: комментарий от crypt

потому что все, что я вижу на текущий момент - это хаки и замена хардкода.

А что ты хотел чистый код за две недели? Инстанс подняли? Подняли. Баги выявили? Выявили. Косяки в текущей реализации нашли? Нашли. То что я пушу быстрые исправления, это не значит, что это вся моя цель. Моя цель сейчас запустить инстанс и определиться с фронтом работ. Плюс некоторые мастхев фичи вроде клуба и другой структуры разделов.

З.Ы. Классных фич в апстриме я не вижу.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от cocucka

перед этим он добавлял какие-то мелочи для модераторов, чтобы им было удобнее. не будешь ты всю эту хрень делать. забьешь со словами: «и так сканает! есть куча более прикольных вещей.» я посмотрел, изменения макскома очень дотошные. в этом проекте этого не будет.

а ведь это вцелом будет определять качество сайта и его юзабилити.

во-вторых, он тоже может копировать хорошие фичи отсюда, если раскрутимся. единственное, почему он до сих пор не сделал PM - это отсутствие фантазии. если возникнет такая гонка, я не уверен, что у нашего проекта есть ресурс для переимплеементации фич. но когда кодовая база синхронизирована с апстримом «лямзить» фичи проще.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от crypt

Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.

Банально убрать хардкод путей и сделать нормальный конфиг – уже геморрой с интеграцией (не сильный, но всё ж время займёт). Плюс код надо тестировать, проверять, а не ебашить на прод в режиме нонстоп, надеясь что оно заработает.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от crypt

Макском ещё месяцок фичи попилит и опять забъёт на пару лет, так всегда было. Он сейчас активизировался из-за войны и боязни проблем.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от cocucka

А что ты хотел чистый код за две недели?

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

Баги выявили? Выявили.

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

Классных фич в апстриме я не вижу.

разделение треда по датам («некромаркеры») - это очень удобно. сразу видно, когда подняли некротред, а можно бы и вообще разделение по дням сделать, как во все современных мессенджерах.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

Макском ещё месяцок фичи попилит и опять забъёт на пару лет, так всегда было. Он сейчас активизировался из-за войны и боязни проблем.

значит, тем легче следовать за апстримом. да, работает неравномерно и нам это наруку.

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 3)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.

ок, точки над i. тогда нахрена весь тот легаси в sql с 1998 тащить?) можно rebase делать?:)

уже геморрой с интеграцией (не сильный, но всё ж время займёт).

а так у нас получается тоже время замерло на 10 дней. зато я за выходные добавил нижнюю панель, в середине разработки PM. ты готов это такими же темпами делать? мне кажется, PM - это must have. тесты ты сейчас игнорируешь вообще при сборке. почему бы не сделать мне возможность подымать новые инстансы для теста просто прописывая нужный домен в конфиге?

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от crypt

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

На локалхосте оно без проблем взлетает. Я же как-то тестирую.

cocucka    
★★★★★★★★★★
Linux / Firefox
Ответ на: комментарий от cocucka

А что ты хотел чистый код за две недели?

если даже у меня получается что-то наваять за 2 дня. то уж тебе-то точно ничего не стоило активнее править код. включая, те же самые PM.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

На локалхосте оно без проблем взлетает. Я же как-то тестирую.

на проде я тещу сразу с nginx. для меня в этом смысл тестирования всего стека. я сразу какой-то домен буду добавлять. как я буду отлавливать баги, связанные с путями, которые я склонен менять.

я уж не говорю о том, что когда я начну менять хардкод имени сервера, то у меня это в git status все полезет.

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 5)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

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

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от crypt

У меня свое мнение насчёт приоритетов. И уж ПМ не первостепенная фича для меня, тем более что её надо качественно продумывать, а не реализовывать на скорую руку.

cocucka    
★★★★★★★★★★
Linux / Firefox
Ответ на: комментарий от cocucka

если ты с апстримом синхронизироваться вообще не собираешься, т.к. делаешь ставку на другие технологии, то тогда есть смысл сделать чистый git. занахрена это все тащить за 20 лет, включая sql миграции?

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

У меня свое мнение насчёт приоритетов. И уж ПМ не первостепенная фича для меня, тем более что её надо качественно продумывать, а не реализовывать на скорую руку.

ок, то есть все остальное можно реализовывать на скорую руку, а pm нельзя?? я думаю, что модифицировать post сообщений, с тем чтобы с уведомлением приходило тело сообщения - это не бином ньютона. как раз можно сделать быстро. люди хоть емейлами смогут обмениваться.

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

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 5)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

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

не только макском сейчас приходится сейчас взяться за код.

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от crypt

Давай программировать буду я? А ты админить? Можем спорить хоть до второго пришествия, но изначально обговаривалось, что проект мой, а ты даёшь хостинг (и то на время бета периода) и помогаешь с настройкой.

P.S. Время на споры тратить больше не буду. Вечером посмотрю твои изменения и доделаю таки клуб.

cocucka    
★★★★★★★★★★
Linux / Chrome
Ответ на: комментарий от cocucka

окей. мне просто неудобно перед народом. как и было договорено, через 2 недели после 28го (в эту субботу) я закрываю клуб. независимо, будет у тебя готов раздел или нет.

вопрос про rebase git'a в силе. зачем тащить легаси?

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 7)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

Давай программировать буду я? А ты админить?

я обеими руками «за». ты главное это делай)

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от crypt

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

А вот вбросить по поводу ШГ - это я всегда пожалуйста.

sorrow    
★★★★★★★★★★★
Windows / Firefox
Ответ на: комментарий от sorrow

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

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome

cocucka, я сегодня читаю чейнжлог maxcom'a. сегодня перенес.

commit df271eac2e916444daad59af28699f1bd0e2a909 (HEAD -> dev)
Author: crypt <root@linuxtalks>
Date:   Tue Jun 14 14:18:50 2022 +0700

       maxcom: при модификации тега переиндексим комменты
    
        без этого статистика странная получается
crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

Не увидишь ты там особо классных фич.

он собирается внедрять какой-то http firewall. хз что это ка.

+  @Bean
+  def httpFirewall: StrictHttpFirewall = {
+    val firewall = new StrictHttpFirewall
+
+    firewall.setAllowedHeaderValues { header =>
+      val parsed = new String(header.getBytes(StandardCharsets.ISO_8859_1), StandardCharsets.UTF_8)
+
+      allowed.matcher(parsed).matches
+    }
+
+    firewall
+  }
crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot.

там какой-то бамп версий spring у него. в связи с безопасностью, наверное. не знаю, важно это или нет. я не стал переносить, т.к. сразу несколько файлов затрагивает.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

допиливать движок до нормального состояния.

*this

Crocodoom    
★★★★★
Android / Chrome
Ответ на: комментарий от cocucka

Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.

В продолжение этого: Известные баги (комментарий)

Может действительно проще будет развернуть второй инстанс-скелет LOR’а с базовой функциональностью (профили/темы/комменты) на Spring Boot и твоём_любимом_шаблонизаторе вместо JSP и потихоньку переносить туда код и логику из оригинального движка? Натянуть похожий дизайн.

В итоге движок сразу получится без всего этого тяжёлого Legacу-наследия и гибридизации со Scala, которая многим непонятна. И ты сможешь добавлять в него фичи гораздо активнее.

Как думаешь? Или всё-таки намерен джедайствовать со всем этим?

EXL    
★★★★★
Windows / Chrome
Ответ на: комментарий от EXL

Идея сама по себе неплоха и я тоже подумывал о подобном. Вопрос только в нужности всего этого, ведь если не будет аудитории, то всё в трубу вылетит.

Мы подняли ЛТ на лоровском движке т.к. он привычен для народа с ЛОРа и поднять его оказалось не так уж и сложно. Понятное дело, что тупо лор один-в-один с другими модераторами никому не нужен, поэтому мы тут вдыхаем свежую струю в концепт форума линуксоидов и в плане модерации, и в плане функционала.

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

cocucka    
★★★★★★★★★★
Linux / Firefox
Ответ на: комментарий от cocucka

У лора норм движок, из него можно сделать офигенно удобный сайт, если упростить структуру и придать более динамичный характер. Сделать трекер главной страницей, например.

alexferman    
★★★★★
Linux / Firefox
Ответ на: комментарий от sudopacman

Я всегда читал как Ииийпук

sorrow    
★★★★★★★★★★★
Windows / Firefox
Ответ на: комментарий от cocucka

код начал загнивать:(

[WARNING] /root/lor_crypt/lorsource/src/test/scala/ru/org/linux/search/ElasticsearchIndexServiceIntegrationSpec.scala:76: warning: object ElasticsearchClientUri in package elastic4s is deprecated (since 6.3.3): Use ElasticNodeEndpoint

обратил на это внимание, т.к. как раз вчера для эластика правил. и что мне с этим делать? подправить существующий scala или тут вообще все надо на java переписывать?

я понимаю, что это ворнинг, но …

crypt    
★★☆☆☆
Последнее исправление: crypt (всего исправлений: 1)

FreeBSD / Chrome
Ответ на: комментарий от cocucka

кстати, макском активно комитит эти несколько дней

Date:   Sat Jun 18 10:43:51 2022 +0300
Date:   Sat Jun 18 09:20:52 2022 +0300
Date:   Fri Jun 17 20:37:36 2022 +0300
Date:   Fri Jun 17 20:19:48 2022 +0300
Date:   Fri Jun 17 19:32:52 2022 +0300
Date:   Fri Jun 17 18:50:11 2022 +0300
Date:   Fri Jun 17 13:46:22 2022 +0300
Date:   Fri Jun 17 13:36:25 2022 +0300
Date:   Thu Jun 16 23:14:50 2022 +0300
Date:   Thu Jun 16 23:13:15 2022 +0300
crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от Crocodoom

нет, там в основном правила, косметика и багфиксы. просто нам бы багфиксы тоже не помешали.

crypt    
★★☆☆☆
FreeBSD / Chrome
Ответ на: комментарий от cocucka

зацени:

commit 75ef1e3b960f75815d3e8849ca166cc1034b103f
Author: Maxim Valyanskiy <[email protected]>
Date:   Thu Jul 7 00:01:02 2022 +0300

    предупреждение об ответе на комментарий
    
    предупреждаем при ответе на комментарий в случае если
    автор комментария больше не может писать в топик из-за
    каких-то ограничений
crypt    
★★☆☆☆
FreeBSD / Chrome
Ограничение на отправку комментариев: только для зарегистрированных пользователей, score>=90