Ru-MaNGOS

Вернуться   Ru-MaNGOS > Ядро > Патчи

Важная информация

Патчи Если кто-то хочет выложить не свой готовый патч - не забудьте указать автора и источник.

Если кто-то хочет задать вопрос по патчу - велкам.

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 20.03.2011, 22:58   #1
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию [patch/dev] use DungeonEncounter.dbc

Если кому интересно, то я сегодня закончил портирование кода от Shauren, использующего DungeonEncounter.dbc. К сожалению, в чистом ядре кода, которому оно может понадобиться, нет вообще (у меня оно используется пока только для сообщения о биндинге групп, планируется в LFG), поэтому статус патча не определен. Да и процесс тестирования только пошел.
Патч размазан на несколько коммитов в моем форке, 2 основных:
загрузка данных об энкаунтерах:
https://github.com/rsa/mangos/commit...9aeeff41d140e7
save/load маски выполненных энкаунтеров, ее установка по спеллам или киллам (внимание! эта часть сделана совсем не как у Shauren и вообще не требует скриптов для работы)
https://github.com/rsa/mangos/commit...fe44a7da162a49
ну и пример использования (с небольшим фиксом предыдущего коммита). very simple.
https://github.com/rsa/mangos/commit...2925163f5b552e
При наличии интереса team - сделаю для чистого ядра. Правда, поскольку интереса ни разу не было, то и делать пока не приходилось.
rsa вне форума   Ответить с цитированием
6 пользователя(ей) сказали cпасибо:
Den (22.03.2011), KiriX (21.03.2011), Konctantin (21.03.2011), Laise (10.08.2011), partizanes (30.06.2011), PSZ (21.03.2011)
Старый 10.08.2011, 11:39   #2
Laise
MaNGOS Dev
 
Аватар для Laise
 
Регистрация: 09.03.2010
Сообщений: 33
Сказал(а) спасибо: 27
Поблагодарили 26 раз(а) в 11 сообщениях
Laise На верном пути
По умолчанию

Просьба сделать для чистого ядра ^_^ и если не трудно запостить на гетмангосе тоже
Laise вне форума   Ответить с цитированием
Старый 10.08.2011, 12:21   #3
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

Сорри, для чистого ядра сделать не выйдет, точнее выйдет - но только ту версию что в теме, а текущая уже здорово переработана. Примерно 3-4 предварительных имплементации придется сделать, строчек по 20 каждая. И их еще надо найти Работа (в любом варианте) довольно приличная, и без веских причин заниматься ей не очень охота - последние несколько "патчей на чистое", что я делал по просьбам мангос-девов, так и остались висеть в воздухе.
rsa вне форума   Ответить с цитированием
Старый 10.08.2011, 12:43   #4
Laise
MaNGOS Dev
 
Аватар для Laise
 
Регистрация: 09.03.2010
Сообщений: 33
Сказал(а) спасибо: 27
Поблагодарили 26 раз(а) в 11 сообщениях
Laise На верном пути
По умолчанию

Цитата:
Сообщение от rsa Посмотреть сообщение
Сорри, для чистого ядра сделать не выйдет, точнее выйдет - но только ту версию что в теме, а текущая уже здорово переработана. Примерно 3-4 предварительных имплементации придется сделать, строчек по 20 каждая. И их еще надо найти Работа (в любом варианте) довольно приличная, и без веских причин заниматься ей не очень охота - последние несколько "патчей на чистое", что я делал по просьбам мангос-девов, так и остались висеть в воздухе.
если данная имплементация является не правильной - то тогда нет смысла что либо с ней делать. А какие именно нужны причины ? Сразу коммит - это вряд ли, это достаточно большое изменение чтобы дать какое то время на его тестирование.В общем вам решать, делать патч или нет, только скидывать на *разработчикам на все пофиг* потом не надо
Laise вне форума   Ответить с цитированием
Старый 10.08.2011, 13:13   #5
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

Цитата:
Сообщение от Laise Посмотреть сообщение
если данная имплементация является не правильной - то тогда нет смысла что либо с ней делать. А какие именно нужны причины ? Сразу коммит - это вряд ли, это достаточно большое изменение чтобы дать какое то время на его тестирование.В общем вам решать, делать патч или нет, только скидывать на *разработчикам на все пофиг* потом не надо
она не неправильная, она неполная. в принципе конечно с нее можно начать, благо ее код в мангосе использовать сейчас просто совсем негде, а соответственно результаты тестирования будут блестящими не может плохо работать код, к которому нет обращений
"скидывать" ни на кого я ничего не собираюсь. весь мой код полностью открыт - заходите и берите. а вот прикладывать дополнительные усилия по прикручиванию - хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.
PS вот кстати еще один коммит из этой же серии:
https://github.com/rsa/mangos/commit...49d72a57ae4f71
уже не настолько нужен, но без него ЛФГ будет неполной. это уже чисто мое изобретение.
rsa вне форума   Ответить с цитированием
Старый 10.08.2011, 13:32   #6
Laise
MaNGOS Dev
 
Аватар для Laise
 
Регистрация: 09.03.2010
Сообщений: 33
Сказал(а) спасибо: 27
Поблагодарили 26 раз(а) в 11 сообщениях
Laise На верном пути
По умолчанию

Цитата:
Сообщение от rsa;
хочется иметь уверенность что от них будет польза коммунити, работать на корзину я и для себя умею. как отмечено выше, несколько предыдущих попыток закончились именно корзиной, причем без каких-то явных причин.
Цитата:
Сообщение от rsa;
полную гарантию даст только страховой полис (с) Бендер.
---
может это лучше уже отдельным коммитом сразу перед самим лфг добавлять, хотя хз, надо ж хотя б основы закинуть
Laise вне форума   Ответить с цитированием
Старый 10.08.2011, 14:09   #7
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

Браво
ну тогда еще несколько обязательных коммитиков в кучу (часть из них можно реализовать по другому но все равно придется). опять же, это оригинальные версии - текущие обычно переработаны.
https://github.com/rsa/mangos/commit...56c7d541173b82
https://github.com/rsa/mangos/commit...d40590c4ed07e7
https://github.com/rsa/mangos/commit...1e5c9e32d8885d
https://github.com/rsa/mangos/commit...ea057678527367 - этот частично уже в мэйнстриме
https://github.com/rsa/mangos/commit...7977f2855b0028

не смог найти - патч на обновление крич после смерти (сейчас новая версия от SilverIce)
rsa вне форума   Ответить с цитированием
2 пользователя(ей) сказали cпасибо:
Laise (10.08.2011), Ranger (10.09.2011)
Ответ


Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.



Текущее время: 08:50. Часовой пояс GMT +3.


ru-mangos.ru - Русское сообщество MaNGOS
Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума Ru-MaNGOS будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot