OpenXiangShan / NEMU

Other
236 stars 90 forks source link

CI: rearrange NEMU CI #417

Closed cebarobot closed 2 months ago

cebarobot commented 3 months ago

This PR rearrange NEMU CI:

WARNING

Some basic tests failed and are set to be skipped temporarily. Please check the skip_tests in ci.yml for details.

poemonsense commented 2 months ago

I think we can default NEMU_HOME to the current path of NEMU. It has been a long time that we have to set it.

Then we don't have to explicitly set it anymore. It seems this env is never set to other values.

This may be done in another PR.

cebarobot commented 2 months ago

I think we can default NEMU_HOME to the current path of NEMU. It has been a long time that we have to set it.

Then we don't have to explicitly set it anymore. It seems this env is never set to other values.

This may be done in another PR.

Yes, NOOP_HOME and NEMU_HOME could be removed. But some path in building system relay on it. There is a lot of work to be done.