brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.51k stars 2.26k forks source link

Fix create_dist on Linux/macOS ASAN #30253

Open wknapik opened 1 year ago

wknapik commented 1 year ago

This works correctly on Windows

It also looks like symbols generation is disabled now with ASAN?

We haven't been running create_dist in ASAN pipelines for a while and just re-enabled it

wknapik commented 1 year ago

cc @rebron @bsclifton

rebron commented 1 year ago

cc: @goodov Can you take a look when you get a chance?

wknapik commented 1 year ago

Slack thread

Related https://github.com/brave/devops/issues/10027

goodov commented 1 year ago

why do we want to run create_dist in ASAN pipelines? this stage creates installers, archives symbols and do other unrelated to ASAN tests things.

wknapik commented 1 year ago

If we want to run ASAN in PRs (which we do if performance is good enough), then we wouldn't want to get rid of the create_dist step, otherwise something that breaks it might get merged. And we wouldn't get installers, etc. There's discussion about this in the linked thread and issue and it looks like most people are in favor of just not running create_dist in PRs. I think at least @mherrmann is still looking into it.

Otherwise, if we want to implement system tests and run them with ASAN, or fuzz the whole browser, we'd need at least part of what's done in create_dist to work.

Anyway, this is something that used to work (and still does on Windows).