Improve indentation in Makefile.aldec
See original GitHub issueHi,
While debugging a libcocotbvhpi.dll missing issue for cocotb1.3, I noticed that some parts of the Makefile.aldec is incomplete. Like the below:
else ifeq ($(TOPLEVEL_LANG),vhdl)
is anyone working on this?
We recently migrated to cocotbv1.3 and unable to get our Aldec RivieraPro simulations progressing due to the libcocotbvhpi.dll missing issue. In that debugging process, I noticed the Makefile.aldec has some gotchas which might be root cause.
Cheers Binu
Issue Analytics
- State:
- Created 4 years ago
- Comments:7 (5 by maintainers)
Top Results From Across the Web
makefile - Why doesn't Make allow indented "if" statements?
The first solution is a terrible pain if you don't have an editor that shows whitespace characters, but if you do, the simplest...
Read more >Emacs: lisp/progmodes/vhdl-mode.el - Fossies
A hint: This file contains one or more very long lines, so maybe it is better readable using the pure text view mode...
Read more >Sigasi Studio Manual
A project is a collection of VHDL and/or (System)Verilog files that belong ... Sigasi Studio will automatically increase or decrease the indentation level....
Read more >HDL free tools - Campera Electronic Systems
Collection and links of VHDL, Verilog, System Verilog and Tcl free and useful ... version control management, makefile generation and simulator link).
Read more >CODING GUIDELINES - Springer Link
guidelines are designed to enhance code maintainability and read- ... Put a Makefile with a default 'all' target in every source directory.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Hi yes I can look into this afterwards.
So to be clear, the request is just to make the indentation clearer here:
https://github.com/cocotb/cocotb/blob/2d11eeac80e97f2f356662b029228690dfc990d5/cocotb/share/makefiles/simulators/Makefile.aldec#L84-L105
Note that it’s very important to use spaces and not tabs for this type of indent, else the makefile will do the wrong thing.