Sentry is reborn!

News
1

I will start maintaining Sentry on Crystal-China.

Lots of bugfixs and nearly half of the code was rewritten in recent two days.

For details, check the README.md in the page.

For interested users, help wanted for refine the English grammar error in README.md, comment, code, anyway, PR or issue always welcome.

3 Likes
2

That’s great. Please also fix Windows support, I have a PR Add initial Windows support by sdogruyol · Pull Request #68 · samueleaton/sentry · GitHub

3

Thank you, i saw you PR.

      {% if flag?(:win32) %}
        if (app_process = @app_process)
          app_process.terminate
          # app_process.wait  # do we need add this too?
        end
      {% end %}

Do we need add app_process.wait waiting terminate completely before run build generate new file?

4

I don’t think so, I’m not on Windows for a while now. Maybe you can try on a Windows 11 VM

1 Like
5

Yes, you are right, I create a PR for fix window support, it works a tiny hello world web server project, please have a look if i’ve missed something? I’m still a beginner when using Crystal on Windows.

image
image931×478 17.2 KB

6

Thanks for sdogruyol’s help, windows binary is available for download now.

2 Likes