fusesoc/fusesoc-cores

elf-loader 1.0.2 removed, should it be upgraded?

Closed this issue · 3 comments

stffrdhrn commented

I see you recently removed elf-loader 1.0.2, that seems to have broke the mor1kx CI build:
https://travis-ci.org/github/openrisc/mor1kx/builds

Can we add elf-loader 1.0.3? I could create a PR, but I thought I would ask why you removed it without adding 1.0.3

olofk commented

This was part of a clean up in fusesoc-cores (fusesoc/elf-loader#2 (comment)). elf-loader is probably the core that has generated most bug reports over the years and is problematic on non-Linux systems. It's definitely a useful thing, but most systems load Verilog hex files directly. I have seen examples of other mechanisms for loading elf file directly and those might lean better to be used as a base for a portable elf-loader than bringing in elf-utils. So since the usage never took off outside of orpsoc, I decided to remove it from fusesoc-cores. The easiest solution for current users is to do fusesoc library add elf-loader https://github.com/fusesoc/elf-loader

stffrdhrn commented

Thanks, that makes sense. Sorry I missed your comment. Also I have already updated my scripts to use fusesoc library add ... So we are all good. Note, I am working on converting the mor1kx CI over to github workflows from travis CI so it will still be broken for a bit.

stffrdhrn commented

For the record, there error I was seeing was:

ERROR: Conflicting requirements:
323Requirements: 'mor1kx__generic == 1.1-0' <- 'elf__loader >= 1.0.3-0'
324    +mor1kx__generic-1.1-0 was ignored because it depends on missing packages
325Requirements: 'mor1kx__generic == 1.1-0'
326    Install command rule (+mor1kx__generic-1.1-0)
327
328Failed to resolve dependencies for mor1kx-generic