MEGA65 / mega65-core

MEGA65 FPGA core
Other
244 stars 88 forks source link

ghdl/ghdl_mcode:error: importation has failed due to compilation error #388

Closed UdayZorawarSingh closed 3 years ago

UdayZorawarSingh commented 3 years ago

When I try to do make simulate in the development branch, I receive following error:

src/vhdl/colourram.vhdl:28:35:error: primary expression expected

Screen Shot 2021-05-18 at 8 33 41 pm

When I had a look at the line number and column number, it pointed towards the closing parenthesis as shown in the image below

Screen Shot 2021-05-18 at 8 32 49 pm

When I looked at the template file for src/vhdl/colourram.vhdl which is present at location src/tools/makerom/colourram_template.vhdl, I found that value of ROMDATA is not present in the generated file

Screen Shot 2021-05-18 at 8 44 17 pm

How could this be resolved? I am using Ubuntu 18.04.5.

waym0002 commented 3 years ago

I can get it to compile by removing the assignment completely, i.e. putting ram : ram_t; there instead. This seems to work as later on in the vhdl the ram variable gets assigned to so it doesn't have to be assigned at first seemingly.

gardners commented 3 years ago

The problem here is that exomizer has not been installed, and so when it tries to make the COLOURRAM.BIN file to then autogenerate colourram.vhdl, it all fails. It would be great to have exomizer also setup as a git sub-module and a rule that checks for its existence, so that this problem can be totally avoided.

Paul.

On Thu, 20 May 2021 at 00:44, Luke Waymark @.***> wrote:

I can get it to compile by removing the assignment completely, i.e. putting ram : ram_t; there instead. This seems to work as later on in the vhdl the ram variable gets assigned to so it doesn't have to be assigned at first seemingly.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/MEGA65/mega65-core/issues/388#issuecomment-844202782, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFCOT2TUA5F4I5ASOPAKGDTOPIV5ANCNFSM45CKEAJQ .

UdayZorawarSingh commented 3 years ago

Thank you Paul. It's working now.