Make use of the race detector #604

New Issue

GiteaMirror · 2025-11-02T03:29:42-06:00

GiteaMirror commented

2025-11-02 03:29:42 -06:00

Originally created by @typeless on GitHub (Apr 1, 2017).

The race detector is helpful for discovering race conditions in concurrent programs. The easiest way I can think of is to add a new target or some kind of env. variable in the Makefile to enabe -race. We could also begin running try.gitea.io with the race detector enabled.

Edit:this is not just a proposal, but also a call for ideas about how we are going to do it.

Originally created by @typeless on GitHub (Apr 1, 2017). The race detector is helpful for discovering race conditions in concurrent programs. The easiest way I can think of is to add a new target or some kind of env. variable in the Makefile to enabe `-race`. We could also begin running try.gitea.io with the race detector enabled. Edit:this is not just a proposal, but also a call for ideas about how we are going to do it.

GiteaMirror added the issue/confirmed topic/build labels 2025-11-02 03:29:42 -06:00

GiteaMirror closed this issue

2025-11-02 03:29:42 -06:00

GiteaMirror commented

2025-11-02 03:29:43 -06:00

@tboerger commented on GitHub (Apr 1, 2017):

An env variable which is enabled on the ci makes sense

@tboerger commented on GitHub (Apr 1, 2017): An env variable which is enabled on the ci makes sense

GiteaMirror commented

2025-11-02 03:29:43 -06:00

@strk commented on GitHub (Apr 1, 2017):

How about GOFLAGS ?

@strk commented on GitHub (Apr 1, 2017): How about GOFLAGS ?

GiteaMirror commented

2025-11-02 03:29:43 -06:00

@typeless commented on GitHub (Apr 4, 2017):

@strk ~~I don't understand how to apply GOFLAGS to the Makefile. Can you elaborate?~~

I submitted a PR for it.

@typeless commented on GitHub (Apr 4, 2017): @strk ~I don't understand how to apply GOFLAGS to the Makefile. Can you elaborate?~ I submitted a PR for it.

GiteaMirror commented

2025-11-02 03:29:43 -06:00

@strk commented on GitHub (Apr 4, 2017):

Yes, I meant something like that.
I'm not sure $(GOFLAGS) would expand to the env variable, give it a try.
If it doesn't work, try ${GOFLAGS}

@strk commented on GitHub (Apr 4, 2017): Yes, I meant something like that. I'm not sure $(GOFLAGS) would expand to the env variable, give it a try. If it doesn't work, try ${GOFLAGS}

GiteaMirror commented

2025-11-02 03:29:43 -06:00

@stale[bot] commented on GitHub (Feb 17, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Feb 17, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

GiteaMirror commented

2025-11-02 03:29:44 -06:00

@stale[bot] commented on GitHub (Apr 20, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Apr 20, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

GiteaMirror commented

2025-11-02 03:29:44 -06:00

@stale[bot] commented on GitHub (May 4, 2019):

This issue has been automatically closed because of inactivity. You can re-open it if needed.

@stale[bot] commented on GitHub (May 4, 2019): This issue has been automatically closed because of inactivity. You can re-open it if needed.

GiteaMirror referenced this issue

2025-11-02 11:47:34 -06:00

[PR #604] [MERGED] docs: update zh-tw and zh-hk translations. #15483

Sign in to join this conversation.