search

robik / ConsoleD

Because colors are awesome.
63 stars 11 forks source link

terminal.d no longer compiles with latest druntime git HEAD #11

Closed quickfur closed 9 years ago

quickfur commented 10 years ago

terminal.d no longer compiles with latest druntime git HEAD, due to signal handlers now being marked nothrow. I tried to fix this but it ended up with a gigantic mess due to the invasiveness of nothrow. :-(

quickfur commented 10 years ago

Quick and dirty hack: pull https://github.com/robik/ConsoleD/pull/12

adamdruppe commented 10 years ago

I think I fixed this on my misc repo github.com/adamdruppe/arsd

does that work? My diff was a lot smaller though, I just swallowed any exception in the handler.

quickfur commented 10 years ago

On Sat, May 17, 2014 at 07:32:11PM -0700, Adam D. Ruppe wrote:

I think I fixed this on my misc repo github.com/adamdruppe/arsd

does that work? My diff was a lot smaller though, I just swallowed any exception in the handler. [...]

Oh. I didn't realize you had your own version of terminal.d in your arsd repo. Should I be using that one instead of the ConsoleD one? I don't actually use any of the other ConsoleD stuff.

--T

adamdruppe commented 10 years ago

On Mon, May 26, 2014 at 11:31:41AM -0700, H. S. Teoh wrote:

Oh. I didn't realize you had your own version of terminal.d in your arsd repo. Should I be using that one instead of the ConsoleD one? I don't actually use any of the other ConsoleD stuff.

They're the same file on my computer, I just don't push to consoled as much (since there's not the other stuff there...)

quickfur commented 10 years ago

On Mon, May 26, 2014 at 11:53:16AM -0700, Adam D. Ruppe wrote:

They're the same file on my computer, I just don't push to consoled as much (since there's not the other stuff there...) [...]

Ah, I see.

Well I switched over to just using arsd, since I'm already using arsd.eventloop anyway, and it's less of a hassle to use a single external repo than it is to use two different ones.

Thanks!

--T

quickfur commented 9 years ago

Hmm, seems I forgot to resolve this issue. It's fixed now anyway.