On Mon, Jan 16, 2017 at 03:21:29PM +0100, Danny Milosavljevic wrote: > Hi, > > > I should have probably added a different phase for the > > "icebox/icebox_vlog.py" substitution, to make clear it's the only path > > which is a run-time reference. The installed icebox_vlog, can't find the > > chipdb.txt file, if we don't make the substitution. > > Yeah, I think how you did it now is best. > > LGTM! > > I've built it and it works. > > I'll leave some time for others to comment. Afterwards, I'll commit it > to master (since it'll cause rebuild of one icestorm-dependent > package).
LGTM! Thanks Theodoros and Danny!