doom2d.org

Главная база плоских морпехов
It is currently 25 Jun 2024, 01:09

All times are UTC + 3 hours




Post new topic Reply to topic  [ 15 posts ] 
Author Message
PostPosted: 02 Mar 2023, 01:09 
Offline
Шерлок Холмс
User avatar

Joined: 19 Jun 2019, 23:28
Posts: 416
Location: Equestrian Wasteland.
Текущий вариант с постоянной перезаписью логов очень сильно не способствует отлову багов, неугодных айпи-адресов и прочих занимательных вещей в виде отладочного вывода или Access Violation, в том числе и всякой дичи с серверной стороны по типу двоения таблицы результатов или же игнора параметров в маплисте. Вероятно, такой реквест мог иметь место, но сходу не вспомню, существует ли тема. А так же в чате это просили.

Хочется следующий вариант: чтоб как в том же пресловутом Minecraft логи записывались в отдельной директории (условно logs) и каждый сохранялся отдельно, в названии необходимо так же дописывать дату и время записи/запуска для удобства отладки по времени.

_________________
aka TerminalHash.


Top
 Profile  
 
PostPosted: 01 Aug 2023, 17:24 
Offline
Шерлок Холмс
User avatar

Joined: 19 Jun 2019, 23:28
Posts: 416
Location: Equestrian Wasteland.
Сам предложил - сам и сделал:
https://repo.or.cz/d2df-sdl.git/commit/94ca9d5ebc01f94b5c4044adbb8ca4048802c0d2

_________________
aka TerminalHash.


Top
 Profile  
 
PostPosted: 01 Aug 2023, 18:34 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
FoxFromPripyat» Сам предложил - сам и сделал
сам и переноси значит

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
PostPosted: 01 Aug 2023, 20:03 
Offline
Шерлок Холмс
User avatar

Joined: 19 Jun 2019, 23:28
Posts: 416
Location: Equestrian Wasteland.
Чёрный Думер wrote:
FoxFromPripyat» Сам предложил - сам и сделал
сам и переноси значит

Ты где-то группу разработчиков у меня увидел? У меня только копытца У меня прав таких нет

_________________
aka TerminalHash.


Top
 Profile  
 
PostPosted: 02 Aug 2023, 02:16 
Offline
Приколист
User avatar

Joined: 04 Feb 2010, 14:42
Posts: 905
Location: Equestria
Отмечу что дата устанавливается в имени только по дефолту, а логи бесконечно создаются на каждом запуске.
Не уверен что это надо на клиенте. Надо как-то лимитировать количество лог-файлов или делать автоматическую отчистку старых логов.


Top
 Profile  
 
PostPosted: 02 Aug 2023, 02:48 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
FoxFromPripyat» Ты где-то группу разработчиков у меня увидел? У меня только копытца У меня прав таких нет
В самом деле, что-то я ступовал.

DeaDDooMER» Не уверен что это надо на клиенте. Надо как-то лимитировать количество лог-файлов или делать автоматическую отчистку старых логов.
Это полностью справедливое замечание, которое надо учесть. Поэтому пока отметил как "В работе".

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
PostPosted: 02 Aug 2023, 10:55 
Offline
Шерлок Холмс
User avatar

Joined: 19 Jun 2019, 23:28
Posts: 416
Location: Equestrian Wasteland.
DeaDDooMER wrote:
Не уверен что это надо на клиенте. Надо как-то лимитировать количество лог-файлов или делать автоматическую отчистку старых логов.

Ради клиента и задумывалось в основном. Если лимитировать или чистить, то вынести переменную в настраиваемые, мне, например, плевать на загаженность логами.

_________________
aka TerminalHash.


Top
 Profile  
 
PostPosted: 21 Mar 2024, 14:09 
Offline
Приколист
User avatar

Joined: 04 Feb 2010, 14:42
Posts: 905
Location: Equestria
Сделал автоматическое удаление логов. По-умолчанию оставляет 10 штук. Думаю этого достаточно что бы избежать случайное затерение лога с ошибкой.
Опцией --keep-logs N можно выбрать другое число.

https://repo.or.cz/d2df-sdl.git/commit/d499d2607a4377957b17555248767a1a9f6e7c86


Top
 Profile  
 
PostPosted: 21 Mar 2024, 14:29 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
DeaDDooMER» Опцией --keep-logs N можно выбрать другое число.
Сделай так, чтобы при N=0 или отсутствии числа они не вытирались вообще.

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
PostPosted: 21 Mar 2024, 14:30 
Offline
Приколист
User avatar

Joined: 04 Feb 2010, 14:42
Posts: 905
Location: Equestria
--keep-logs -1


Top
 Profile  
 
PostPosted: 21 Mar 2024, 14:41 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
DeaDDooMER» --keep-logs -1
А, уже увидел. И без -1 тоже можно, потому что в StrToIntDef() указано именно это значение, а не 0.

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
PostPosted: 21 Mar 2024, 14:45 
Offline
Приколист
User avatar

Joined: 04 Feb 2010, 14:42
Posts: 905
Location: Equestria
Зависит от условия if logLimit >= 0 then

Вообще получается не оч правильно. Оно удаляет файл после создания лога. Может при logLimit = 0 не создавать его вообще?


Top
 Profile  
 
PostPosted: 21 Mar 2024, 15:10 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
DeaDDooMER» Зависит от условия if logLimit >= 0 then
От этого как раз зависит то, будет ли удаляться при 0. :)

DeaDDooMER» Может при logLimit = 0 не создавать его вообще?
А есть ли смысл в такой настройке? Я потому и предлагал 0 вместо -1, потому что ну просто очевиднее же.

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
PostPosted: 21 Mar 2024, 16:41 
Offline
Приколист
User avatar

Joined: 04 Feb 2010, 14:42
Posts: 905
Location: Equestria
Чёрный Думер wrote:
Я потому и предлагал 0 вместо -1, потому что ну просто очевиднее же.
Не очевиднее.
--keep-logs N задаёт сколько в папке должно остаться логов (включая новый).
--keep-logs 0 => ничего не оставлять (включая новый).
Можно сделать --keep-logs all или типа того, что бы понятнее (это даже сейчас работает).
Чёрный Думер wrote:
А есть ли смысл в такой настройке?
Может я хочу всё удалить, но лень ползти непонятно куда и делать rm. :)
Ну и никак нельзя выключить запись лога вообще. Если не указан --log PATH, имя задаётся автоматически.
Хотя наверно для этого лучше сделать --no-log.


Top
 Profile  
 
PostPosted: 21 Mar 2024, 18:02 
Offline
Принципиально неуничтожаем
User avatar

Joined: 18 Oct 2009, 04:01
Posts: 6683
Location: Владивосток
Лучше оставь пока как есть, я всё равно систему логов со своей стороны уже поправил сильно, только не закоммитил ещё. Всё равно эти настройки для разработчиков в основном.
Разве что я лимит логов ещё до 15ти бы поднял.

DeaDDooMER wrote:
Не очевиднее.
--keep-logs N задаёт сколько в папке должно остаться логов (включая новый).
--keep-logs 0 => ничего не оставлять (включая новый).
Неа. Второе - это по смыслу эквивалент простого --keep-logs. У нас и так сейчас уже много где 0 означает "без лимита".

DeaDDooMER wrote:
Ну и никак нельзя выключить запись лога вообще. Если не указан --log PATH, имя задаётся автоматически.
Мой вопрос и был в том, нужно ли это вообще.

_________________
Чёрный Думер, Чёрный Думер
С монстрами сражается.
Чёрный Думер, Чёрный Думер
Рокетланчер плавится.


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 15 posts ] 

All times are UTC + 3 hours


Who is online

Users browsing this forum: No registered users and 10 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
doom2d.org, since 2007