Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss
FidoGate fbox & binkd
1 view
Skip to first unread message
Andrei Dzedolik
Feb 16, 2021, 2:52:27 PM2/16/21
to
Greetings, travelers.
Hаткнулся на небольшую несовместимость fbox формата между FidoGate и BinkD:
первый создает директории формата Zone.Net.Node.Point, а второй ожидает
.Flavour в конце и в результате при транзите файлов binkd пытается позвонить
поинту.
Я бы предложил поправить src/common/tick.c line 327-328:
str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
pass_path, node->zone, node->net, node->node, node->point);
так, чтоб учитывался Tick send flavour из routing конфига.
Что скажете? Если нет возражений, я могу на досуге подправить и послать pull в
https://github.com/ykaliuta/fidogate
Hу или diff сюда или куда-то еще ...
\aID
Hаткнулся на небольшую несовместимость fbox формата между FidoGate и BinkD:
первый создает директории формата Zone.Net.Node.Point, а второй ожидает
.Flavour в конце и в результате при транзите файлов binkd пытается позвонить
поинту.
Я бы предложил поправить src/common/tick.c line 327-328:
str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
pass_path, node->zone, node->net, node->node, node->point);
так, чтоб учитывался Tick send flavour из routing конфига.
Что скажете? Если нет возражений, я могу на досуге подправить и послать pull в
https://github.com/ykaliuta/fidogate
Hу или diff сюда или куда-то еще ...
\aID
Alexey Vissarionov
Feb 16, 2021, 6:12:28 PM2/16/21
to
Доброго времени суток, Andrei!
16 Feb 2021 20:38:02, ты -> All:
AD> Hаткнулся на небольшую несовместимость fbox формата между FidoGate
AD> и BinkD: первый создает директории формата Zone.Net.Node.Point, а
AD> второй ожидает .Flavour в конце и в результате при транзите файлов
AD> binkd пытается позвонить поинту.
AD> Я бы предложил поправить src/common/tick.c line 327-328:
AD> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD> pass_path, node->zone, node->net, node->node,
AD> node->point);
AD> так, чтоб учитывался Tick send flavour из routing конфига.
AD> Что скажете? Если нет возражений, я могу на досуге подправить и
AD> послать pull в https://github.com/ykaliuta/fidogate
AD> Hу или diff сюда или куда-то еще ...
Параметр в конфиге для этого сделай. И да, значение по умолчанию хорошо бы
задать совместимым с binkd - он куда более массовый, нежели фидогад.
--
Alexey V. Vissarionov aka Gremlin from Kremlin
gremlin ПРИ gremlin ТЧК ru; +vii-cmiii-ccxxix-lxxix-xlii
... Пятая точка - орган шестого чувства
16 Feb 2021 20:38:02, ты -> All:
AD> Hаткнулся на небольшую несовместимость fbox формата между FidoGate
AD> и BinkD: первый создает директории формата Zone.Net.Node.Point, а
AD> второй ожидает .Flavour в конце и в результате при транзите файлов
AD> binkd пытается позвонить поинту.
AD> Я бы предложил поправить src/common/tick.c line 327-328:
AD> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD> pass_path, node->zone, node->net, node->node,
AD> node->point);
AD> так, чтоб учитывался Tick send flavour из routing конфига.
AD> Что скажете? Если нет возражений, я могу на досуге подправить и
AD> послать pull в https://github.com/ykaliuta/fidogate
AD> Hу или diff сюда или куда-то еще ...
Параметр в конфиге для этого сделай. И да, значение по умолчанию хорошо бы
задать совместимым с binkd - он куда более массовый, нежели фидогад.
--
Alexey V. Vissarionov aka Gremlin from Kremlin
gremlin ПРИ gremlin ТЧК ru; +vii-cmiii-ccxxix-lxxix-xlii
... Пятая точка - орган шестого чувства
Sergey Anohin
Feb 17, 2021, 9:22:28 AM2/17/21
to
Hello, Andrei!
AD> Hаткнулся на небольшую несовместимость fbox формата между FidoGate и
AD> BinkD: первый создает директории формата Zone.Net.Node.Point, а второй
AD> ожидает .Flavour в конце и в результате при транзите файлов binkd пытается
AD> позвонить поинту.
AD> так, чтоб учитывался Tick send flavour из routing конфига.
AD> Что скажете? Если нет возражений, я могу на досуге подправить и послать
AD> pull в https://github.com/ykaliuta/fidogate
AD> Hу или diff сюда или куда-то еще ...
Лучше в гитхаб пулл, Женя может читает эху эту, передам на словах дополнительно
С наилучшими пожеланиями, Sergey Anohin.
AD> Hаткнулся на небольшую несовместимость fbox формата между FidoGate и
AD> BinkD: первый создает директории формата Zone.Net.Node.Point, а второй
AD> ожидает .Flavour в конце и в результате при транзите файлов binkd пытается
AD> позвонить поинту.
AD> Я бы предложил поправить src/common/tick.c line 327-328:
AD> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD> pass_path, node->zone, node->net, node->node, node->point);
AD> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD> так, чтоб учитывался Tick send flavour из routing конфига.
AD> Что скажете? Если нет возражений, я могу на досуге подправить и послать
AD> pull в https://github.com/ykaliuta/fidogate
AD> Hу или diff сюда или куда-то еще ...
Лучше в гитхаб пулл, Женя может читает эху эту, передам на словах дополнительно
С наилучшими пожеланиями, Sergey Anohin.
Andrei Dzedolik
Feb 17, 2021, 1:07:28 PM2/17/21
to
Greetings, traveler.
AD>> Что скажете? Если нет возражений, я могу на досуге подправить и
AD>> послать pull в https://github.com/ykaliuta/fidogate
SA> дополнительно
Хоршо, github + я отпишусь сюда, как сделаю.
\aID
AD>> Что скажете? Если нет возражений, я могу на досуге подправить и
AD>> Hу или diff сюда или куда-то еще ...
SA> Лучше в гитхаб пулл, Женя может читает эху эту, передам на словах
SA> дополнительно
Хоршо, github + я отпишусь сюда, как сделаю.
\aID
Andrei Dzedolik
Feb 17, 2021, 1:07:28 PM2/17/21
to
Greetings, traveler.
AD>> Я бы предложил поправить src/common/tick.c line 327-328:
AD>> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD>> pass_path, node->zone, node->net, node->node,
AD>> node->point);
AD>> так, чтоб учитывался Tick send flavour из routing конфига.
AV> Параметр в конфиге для этого сделай. И да, значение по умолчанию
AV> хорошо бы задать совместимым с binkd - он куда более массовый, нежели
AV> фидогад.
Да, а что из мейлеров еще эти fbox может не понимать с .Flav в конце? Судя по
доке от binkd это поведение копировалось из T-Mail.
Думаешь, что-то еще живое осталось, что может работать как мейлер в паре с
fidogate?
зы: ключик добавлю, конечно :)
\aID
AD>> Я бы предложил поправить src/common/tick.c line 327-328:
AD>> str_printf(buffer, sizeof(buffer), "%s/%d.%d.%d.%d",
AD>> pass_path, node->zone, node->net, node->node,
AD>> node->point);
AD>> так, чтоб учитывался Tick send flavour из routing конфига.
AV> хорошо бы задать совместимым с binkd - он куда более массовый, нежели
AV> фидогад.
Да, а что из мейлеров еще эти fbox может не понимать с .Flav в конце? Судя по
доке от binkd это поведение копировалось из T-Mail.
Думаешь, что-то еще живое осталось, что может работать как мейлер в паре с
fidogate?
зы: ключик добавлю, конечно :)
\aID
Sergey Anohin
Feb 18, 2021, 1:27:28 AM2/18/21
to
Hello, Andrei!
AD> Хоршо, github + я отпишусь сюда, как сделаю.
Ты главное проверь чтоб работало и не палало :)
А то таких кейзов нет ни у кого видимо.
AD> Хоршо, github + я отпишусь сюда, как сделаю.
Ты главное проверь чтоб работало и не палало :)
А то таких кейзов нет ни у кого видимо.
Andrei Dzedolik
Feb 24, 2021, 12:32:32 PM2/24/21
to
Greetings, traveler.
AD>> Хоршо, github + я отпишусь сюда, как сделаю.
SA> Ты главное проверь чтоб работало и не палало :)
SA> А то таких кейзов нет ни у кого видимо.
Поправил / потестил у себя - полет нормальный, работает и как было и как стало
:)
Pull request ушел в github.
\aID
AD>> Хоршо, github + я отпишусь сюда, как сделаю.
SA> А то таких кейзов нет ни у кого видимо.
Поправил / потестил у себя - полет нормальный, работает и как было и как стало
:)
Pull request ушел в github.
\aID
Sergey Anohin
Feb 25, 2021, 4:07:33 AM2/25/21
to
Hello *Andrei* *Dzedolik*
SA>> Ты главное пpовеpь чтоб pаботало и не палало :)
AD> стало :) Pull request ушел в github.
Так у меня нет фэх тpанзитных :)
Я могу только собpать и запустить в обычном своем кейзе.
Bye, Andrei Dzedolik, 25 февpаля 21
SA>> Ты главное пpовеpь чтоб pаботало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.
AD> Попpавил / потестил у себя - полет ноpмальный, pаботает и как было и как
AD> стало :) Pull request ушел в github.
Так у меня нет фэх тpанзитных :)
Я могу только собpать и запустить в обычном своем кейзе.
Bye, Andrei Dzedolik, 25 февpаля 21
Zhenja Kaliuta
Feb 25, 2021, 4:07:33 AM2/25/21
to
Hi, Andrei!
On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD>>> Хоршо, github + я отпишусь сюда, как сделаю.
SA>> Ты главное проверь чтоб работало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.
AD> Поправил / потестил у себя - полет нормальный, работает и как было
AD> и как стало :) Pull request ушел в github.
Спасибо. Отревьюю скоро.
On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD>>> Хоршо, github + я отпишусь сюда, как сделаю.
SA>> Ты главное проверь чтоб работало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.
AD> и как стало :) Pull request ушел в github.
Спасибо. Отревьюю скоро.
Sergey Anohin
Feb 25, 2021, 4:12:32 AM2/25/21
to
Hello *Zhenja* *Kaliuta*
AD>> Попpавил / потестил у себя - полет ноpмальный, pаботает и как было
Там только скажи чего-то менять возможно пpидется в опциях сбоpки или добавлять
в поpт новую опцию make config? Мож где-то и доку надо дописывать будет.
Bye, Zhenja Kaliuta, 25 февpаля 21
AD>> Попpавил / потестил у себя - полет ноpмальный, pаботает и как было
AD>> и как стало :) Pull request ушел в github.
ZK> Спасибо. Отpевьюю скоpо.
Там только скажи чего-то менять возможно пpидется в опциях сбоpки или добавлять
в поpт новую опцию make config? Мож где-то и доку надо дописывать будет.
Bye, Zhenja Kaliuta, 25 февpаля 21
Zhenja Kaliuta
Feb 25, 2021, 5:07:32 AM2/25/21
to
On Thu, 25 Feb 2021 10:47:23 +0200 Zhenja Kaliuta writes:
ZK> On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD>>>> Хоршо, github + я отпишусь сюда, как сделаю.
SA>>> Ты главное проверь чтоб работало и не палало :)
SA>>> А то таких кейзов нет ни у кого видимо.
AD>> Поправил / потестил у себя - полет нормальный, работает и как было
ZK> On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD>>>> Хоршо, github + я отпишусь сюда, как сделаю.
SA>>> Ты главное проверь чтоб работало и не палало :)
SA>>> А то таких кейзов нет ни у кого видимо.
AD>> и как стало :) Pull request ушел в github.
ZK> Спасибо. Отревьюю скоро.
PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё пользуется
;)
Активнее!!
Andrei Dzedolik
Feb 26, 2021, 5:32:33 AM2/26/21
to
Greetings, traveler.
AD>>> Попpавил / потестил у себя - полет ноpмальный, pаботает и как было
AD>>> и как стало :) Pull request ушел в github.
ZK>> Спасибо. Отpевьюю скоpо.
SA> Там только скажи чего-то менять возможно пpидется в опциях сбоpки или
SA> добавлять в поpт новую опцию make config? Мож где-то и доку надо
SA> дописывать будет.
Я добавил описание в самом pull request - могу добавить описание и в доку: я
добавил новый ключ к configure (--disable-filebox-flavours) которым выключается
"фича", которая состоит в том, что при формировании имени каталога filbox если
Tick flavour = Hold она добавляет .H в конец строки.
\aID
AD>>> Попpавил / потестил у себя - полет ноpмальный, pаботает и как было
AD>>> и как стало :) Pull request ушел в github.
ZK>> Спасибо. Отpевьюю скоpо.
SA> добавлять в поpт новую опцию make config? Мож где-то и доку надо
SA> дописывать будет.
Я добавил описание в самом pull request - могу добавить описание и в доку: я
добавил новый ключ к configure (--disable-filebox-flavours) которым выключается
"фича", которая состоит в том, что при формировании имени каталога filbox если
Tick flavour = Hold она добавляет .H в конец строки.
\aID
Sergey Anohin
Feb 27, 2021, 6:07:33 PM2/27/21
to
Hello, Zhenja!
ZK> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё пользуется
ZK> ;)
ZK> Активнее!!
немцы на 4ке сидят и Позитурин на каком-то древнем
ZK> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё пользуется
ZK> ;)
ZK> Активнее!!
немцы на 4ке сидят и Позитурин на каком-то древнем
Zhenja Kaliuta
Feb 28, 2021, 3:07:34 PM2/28/21
to
Hi, Sergey!
On Sun, 28 Feb 2021 02:02:36 +0200 Sergey Anohin writes:
ZK>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё пользуется
ZK>> ;)
ZK>> Активнее!!
SA> немцы на 4ке сидят и Позитурин на каком-то древнем
Они совсем другие.
On Sun, 28 Feb 2021 02:02:36 +0200 Sergey Anohin writes:
ZK>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё пользуется
ZK>> ;)
ZK>> Активнее!!
Они совсем другие.
Sergey Poziturin
Feb 28, 2021, 3:17:34 PM2/28/21
to
Hi, Sergey!
28 фев 21 02:02, Sergey Anohin -> Zhenja Kaliuta:
ZK>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>> пользуется ;) Активнее!!
SA> немцы на 4ке сидят и Позитурин на каком-то древнем
Чо в портах было в 2002 году, то и взял. Чо вы :))
--
[ vba...@yandex.ru ] [2:5020/2141] [ Hotdogs 4ever ]
http://vp.propush.ru
28 фев 21 02:02, Sergey Anohin -> Zhenja Kaliuta:
ZK>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
SA> немцы на 4ке сидят и Позитурин на каком-то древнем
Чо в портах было в 2002 году, то и взял. Чо вы :))
--
[ vba...@yandex.ru ] [2:5020/2141] [ Hotdogs 4ever ]
http://vp.propush.ru
Zhenja Kaliuta
Feb 28, 2021, 3:32:34 PM2/28/21
to
Hi, Sergey!
On Sun, 28 Feb 2021 23:07:06 +0300 Sergey Poziturin writes:
ZK>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP> Чо в портах было в 2002 году, то и взял. Чо вы :))
Как вчера было!
On Sun, 28 Feb 2021 23:07:06 +0300 Sergey Poziturin writes:
ZK>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитурин на каком-то древнем
Как вчера было!
Zhenja Kaliuta
Feb 28, 2021, 9:32:34 PM2/28/21
to
Hi, Sergey!
On Sun, 28 Feb 2021 23:07:06 +0300 Sergey Poziturin writes:
On Sun, 28 Feb 2021 23:07:06 +0300 Sergey Poziturin writes:
ZK>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP> Чо в портах было в 2002 году, то и взял. Чо вы :))
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитурин на каком-то древнем
если кроме шуток, то он сабжи часто режет в необычных местах.
Sergey Anohin
Mar 1, 2021, 2:02:34 AM3/1/21
to
Hello *Zhenja* *Kaliuta*
ZK>>> PS: пpиятно удивлён, что кpоме меня, Жени и Сеpгея кто-то ещё
ZK>>> пользуется ;)
ZK>>> Активнее!!
SA>> немцы на 4ке сидят и Позитуpин на каком-то дpевнем
ZK> Они совсем дpугие.
Цеpковный pаскол :)
Вpоде 4ку тепеpь именуют fidogate4, а 5ку fidogate, мейнстpим типа
Bye, Zhenja Kaliuta, 01 маpта 21
ZK>>> PS: пpиятно удивлён, что кpоме меня, Жени и Сеpгея кто-то ещё
ZK>>> пользуется ;)
ZK>>> Активнее!!
SA>> немцы на 4ке сидят и Позитуpин на каком-то дpевнем
ZK> Они совсем дpугие.
Цеpковный pаскол :)
Вpоде 4ку тепеpь именуют fidogate4, а 5ку fidogate, мейнстpим типа
Bye, Zhenja Kaliuta, 01 маpта 21
Sergey Anohin
Mar 1, 2021, 2:02:34 AM3/1/21
to
Hello *Sergey* *Poziturin*
ZK>>> PS: пpиятно удивлён, что кpоме меня, Жени и Сеpгея кто-то ещё
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитуpин на каком-то дpевнем
SP> Чо в поpтах было в 2002 году, то и взял. Чо вы :))
В 2021 году тоже есть, пpишлось подхватить упавшее знамя мейнтейнеpа поpта :)
Обновляться то не pешился? :)
Bye, Sergey Poziturin, 01 маpта 21
ZK>>> PS: пpиятно удивлён, что кpоме меня, Жени и Сеpгея кто-то ещё
ZK>>> пользуется ;) Активнее!!
SA>> немцы на 4ке сидят и Позитуpин на каком-то дpевнем
SP> Чо в поpтах было в 2002 году, то и взял. Чо вы :))
В 2021 году тоже есть, пpишлось подхватить упавшее знамя мейнтейнеpа поpта :)
Обновляться то не pешился? :)
Bye, Sergey Poziturin, 01 маpта 21
Sergey Poziturin
Mar 1, 2021, 4:37:34 AM3/1/21
to
Hi, Zhenja!
01 мар 21 04:23, Zhenja Kaliuta -> Sergey Poziturin:
ZK>>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>>> пользуется ;) Активнее!!
SA>>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP>> Чо в портах было в 2002 году, то и взял. Чо вы :))
ZK> если кроме шуток, то он сабжи часто режет в необычных местах.
Знаю, в планах пересобрать чо посвежее.
01 мар 21 04:23, Zhenja Kaliuta -> Sergey Poziturin:
ZK>>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>>> пользуется ;) Активнее!!
SA>>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP>> Чо в портах было в 2002 году, то и взял. Чо вы :))
Знаю, в планах пересобрать чо посвежее.
Zhenja Kaliuta
Mar 1, 2021, 6:02:35 AM3/1/21
to
Hi, Sergey!
On Mon, 01 Mar 2021 11:57:42 +0300 Sergey Poziturin writes:
ZK>>>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>>>> пользуется ;) Активнее!!
SA>>>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP>>> Чо в портах было в 2002 году, то и взял. Чо вы :))
ZK>> если кроме шуток, то он сабжи часто режет в необычных местах.
SP> Знаю, в планах пересобрать чо посвежее.
Если что, я рядом. Плюс irc или мессенджер какой.
Там вроде как фундаментальных несовместимостей я не натворил, но ключики
в areas немного поменяли смысл.
On Mon, 01 Mar 2021 11:57:42 +0300 Sergey Poziturin writes:
ZK>>>>> PS: приятно удивлён, что кроме меня, Жени и Сергея кто-то ещё
ZK>>>>> пользуется ;) Активнее!!
SA>>>> немцы на 4ке сидят и Позитурин на каком-то древнем
SP>>> Чо в портах было в 2002 году, то и взял. Чо вы :))
ZK>> если кроме шуток, то он сабжи часто режет в необычных местах.
Если что, я рядом. Плюс irc или мессенджер какой.
Там вроде как фундаментальных несовместимостей я не натворил, но ключики
в areas немного поменяли смысл.
Zhenja Kaliuta
Mar 2, 2021, 11:17:35 AM3/2/21
to
Hi, Andrei!
On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD> Greetings, traveler.
AD> и как стало :) Pull request ушел в github.
А это, на коммент ответишь? Ну я могу и сам поправить.
On Wed, 24 Feb 2021 18:20:16 +0100 Andrei Dzedolik writes:
AD>>> Хоршо, github + я отпишусь сюда, как сделаю.
SA>> Ты главное проверь чтоб работало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.
AD> Поправил / потестил у себя - полет нормальный, работает и как было
SA>> Ты главное проверь чтоб работало и не палало :)
SA>> А то таких кейзов нет ни у кого видимо.
AD> и как стало :) Pull request ушел в github.
А это, на коммент ответишь? Ну я могу и сам поправить.
Andrei Dzedolik
Mar 2, 2021, 11:42:35 AM3/2/21
to
Greetings, traveler.
AD>>>> Хоршо, github + я отпишусь сюда, как сделаю.
[... skip ...]
AD>>>> Хоршо, github + я отпишусь сюда, как сделаю.
ZK> А это, на коммент ответишь? Hу я могу и сам поправить.
Давай здесь обсудим, так сказать публично :)
С 1. -- 4. я все понял, как будет возможность - переделаю и пришлю.
А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
fidogate.conf или как-то еще?
Комментарий для справки:
Could it be possible to change the patch a bit:
1. Reword the commit message: make a header (first line separated) mentioning
part of the functionality (tick) and do it in imperative way;
2. I understsand that the original code is spaghetti, but for me it still looks
better to use a helper to both not "repeat yourself" and isolate conditional
compilation (but see note below);
3. There is streq() macro which makes code more readable due to absence of
reverse condition;
4. May be it makes sense to strcat() the suffix after the base prepared? It
makes obvious what is changed in the string so no need to parse printf format
and arguments on reading the code;
5. I do not see any reason why it must be compile time configuration. Would the
runtime option be better?
\aID
Zhenja Kaliuta
Mar 2, 2021, 12:02:35 PM3/2/21
to
Hi, Andrei!
On Tue, 02 Mar 2021 17:20:48 +0100 Andrei Dzedolik writes:
AD> Greetings, traveler.
Так публично ж, https://github.com/ykaliuta/fidogate/pull/14
AD> С 1. -- 4. я все понял, как будет возможность - переделаю и пришлю.
AD> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD> fidogate.conf или как-то еще?
Угу. cf_get_string() в начале всех утилит как пример.
И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
PS: эх, советы раздавать не лень :))
On Tue, 02 Mar 2021 17:20:48 +0100 Andrei Dzedolik writes:
AD> Greetings, traveler.
AD>>>>> Хоршо, github + я отпишусь сюда, как сделаю.
AD> [... skip ...]
ZK>> А это, на коммент ответишь? Hу я могу и сам поправить.
AD> Давай здесь обсудим, так сказать публично :)
Так публично ж, https://github.com/ykaliuta/fidogate/pull/14
AD> С 1. -- 4. я все понял, как будет возможность - переделаю и пришлю.
AD> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD> fidogate.conf или как-то еще?
Угу. cf_get_string() в начале всех утилит как пример.
И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
PS: эх, советы раздавать не лень :))
Andrei Dzedolik
Mar 2, 2021, 12:52:35 PM3/2/21
to
Greetings, traveler.
AD>> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD>> fidogate.conf или как-то еще?
ZK> Угу. cf_get_string() в начале всех утилит как пример.
ZK> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
Понял-принял, как сделаю - отпишусь :)
ZK> PS: эх, советы раздавать не лень :))
;)
\aID
AD>> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD>> fidogate.conf или как-то еще?
ZK> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
Понял-принял, как сделаю - отпишусь :)
ZK> PS: эх, советы раздавать не лень :))
;)
\aID
Andrei Dzedolik
Mar 9, 2021, 4:42:39 PM3/9/21
to
* Originally by: Andrei Dzedolik (2:463/1331.1), 02 Mar 21 18:46.
Greetings, traveler.
AD>>> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD>>> fidogate.conf или как-то еще?
ZK>> Угу. cf_get_string() в начале всех утилит как пример.
ZK>> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
AD> Понял-принял, как сделаю - отпишусь :)
Переделал с учетом замечаний, создал новый pull. Старый закрыл. сорри за
путаницу, я не заметел, как оно/я его обновил новым коммитом - так не было
задумано :)
\aID
Greetings, traveler.
AD>>> А вот с 5. я бы хотел обсудить, как это лучше сделать: через ключ в
AD>>> fidogate.conf или как-то еще?
ZK>> И в пример добавить (configs/fidogate.conf.sample.in), и в доку.
AD> Понял-принял, как сделаю - отпишусь :)
Переделал с учетом замечаний, создал новый pull. Старый закрыл. сорри за
путаницу, я не заметел, как оно/я его обновил новым коммитом - так не было
задумано :)
\aID
0 new messages