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 人の参加者

通知

期日

期日が正しくないか範囲を超えています。 'yyyy-mm-dd' の形式で入力してください。

期日は未設定です。

依存関係

依存関係が設定されていません。

リファレンス: cwtch.im/cwtch#163

説明はありません。