search

cryptodotis / irssi-otr

LibOTR functionality in Irssi.
GNU General Public License v2.0
183 stars 24 forks source link

Fix OTR init message causing unknown command errors. #42

Closed Tronic closed 8 years ago

Tronic commented 10 years ago

Remove newline characters from libotr output and customize the OTR init message for IRC users.

Closes bug #33.

dgoulet commented 10 years ago

Hey thanks for this! I'll be looking into this pretty soon! Just wanted to ack your pull req. (and the two others).

Tronic commented 9 years ago

Any updates on this? There's also another fork called ahf-hax that looks interesting but unfortunately it is not based on mine and I have no time to properly review his work and do a merge.

DrWhax commented 9 years ago

We are working with afh to get things polished and in addition, we're looking into the possibility of merging this into Irssi upstream. Stay tuned!

anarcat commented 8 years ago

@DrWhax i have been hearing about otr being merged in irssi upstream for a while now (your message is the oldest such utterance for me). so i don't think it should keep valid patches from being merged here, on the contrary: as long as the upstream otr module is not available, this here is the real thing that people use.

i would go even further and say that improvements done on the irssi-otr native module should be merged here before the merge module is released.

dgoulet commented 8 years ago

Did some syntax fixes in 251f7b8. Big thanks!

anarcat commented 8 years ago

nice, thanks for the merge.