search

bastibl / instant-gnuradio

Customizable GNU Radio Environment for Live Images and VMs
158 stars 34 forks source link

Fix the deprecated syntax iso_checksum error #45

Closed stkrzysiak closed 2 years ago

stkrzysiak commented 2 years ago

This is the result of running packer fix base.json. Unfortunately packer fixed a lot more than the original deprecated field(iso_checksum_type). I can resubmit this PR just fixing the field name only but I figure packer wants this format. Closes #44

bastibl commented 2 years ago

Looks like this needs more updates or does the build work for you with just this change? For me, it seems to point to an outdated iso.

stkrzysiak commented 2 years ago

I was going to update the iso in a different branch, can do it in this one if you prefer? Even then I run into another issue I was going to tackle in another PR, just thought I'd get the contribution ball rolling.

bastibl commented 2 years ago

At the moment, I have no way to test. It would be great, if you could update to a point that it builds again. Everything else can be handled is separate/follow-up PR.

stkrzysiak commented 2 years ago

This still errors on a curtin cmd-in-target step, seems to be something introduced in the later versions of 20.04.1+. There seem to be efforts to fix this but upgrading to 21.04 breaks the install automation so not sure how valid the fix is. Will circle back.

https://stackoverflow.com/questions/67457987/ubuntu-server-installation-stops-at-curtin-command-in-target https://bugs.launchpad.net/ubuntu/+source/subiquity/+bug/1875560

bastibl commented 2 years ago

Looks like we have a working build again. Let me know if there are further issues.

stkrzysiak commented 2 years ago

Roger that @bastibl