merging some bugfixes #163

Слито

sarah слито 1 коммит(ов) из bigmerge в master

2018-11-22 01:05:24 +00:00

erinn прокомментировал(а)

2018-11-22 00:48:41 +00:00

Владелец

we need to be able to set the AIF before launching individual peers, so this splits that out

we need to be able to set the AIF before launching individual peers, so this splits that out

buildbot прокомментировал(а)

2018-11-22 00:49:00 +00:00

Участник

Drone Build Status: failure

https://build.openprivacy.ca/cwtch.im/cwtch/278

Drone Build Status: failure https://build.openprivacy.ca/cwtch.im/cwtch/278

buildbot прокомментировал(а)

2018-11-22 01:03:26 +00:00

Участник

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/cwtch/280

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch/280

sarah закрыл этот запрос на слияние

2018-11-22 01:05:23 +00:00

dan прокомментировал(а)

2018-11-22 05:43:33 +00:00

Владелец

i did away with startPeer in app
Listen now does the go function thing, so you can call peer.Listen directly and itll fork

i did away with startPeer in app Listen now does the go function thing, so you can call peer.Listen directly and itll fork

dan прокомментировал(а)

2018-11-22 06:52:22 +00:00

Владелец

so yeah
lines 113-114

 		p.Init(app.mn)
 		p.Listen()

start and launch the peer. I got rid of app.startPeer cus its wrong and terrible

to be fair, LaunchPeers will basically nop
because
.Listen go calls .listenFn inside a look to handle socket fails
and you added cp.started = true inside listenFn
so LaunchPeers should always find them all started already

so yeah lines 113-114 ``` p.Init(app.mn) p.Listen() ``` start and launch the peer. I got rid of app.startPeer cus its wrong and terrible to be fair, LaunchPeers will basically nop because .Listen go calls .listenFn inside a look to handle socket fails and you added `cp.started = true` inside listenFn so LaunchPeers should always find them all `started` already

Войдите, чтобы присоединиться к обсуждению.

Нет рецензентов

Нет меток

ready-for-implementation

Нет этапа

Нет назначенных лиц

3 участников

Уведомления

Срок выполнения

Срок действия недействителен или находится за пределами допустимого диапазона. Пожалуйста, используйте формат 'гггг-мм-дд'.

Срок выполнения не установлен.

Зависимости

Зависимостей нет.

Ссылка: cwtch.im/cwtch#163

Описание отсутствует.