New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nonsdk_container_setup: correct sources line changing #78
base: master
Are you sure you want to change the base?
Conversation
New version of Ubuntu LXD images seem to have a slightly different sources line. Account for those by making the trailing slash optional. Fixes: #71
Upon some further investigation, this PR seems to generate a ports.list file that makes sense, but does not fix the fact that |
Could you please share the content of your |
|
Sorry for the delay. Could you please test the new version? It's weird, because the code seems to work with this content, but it seems like it's overwritten after the fact. My guess is that it's cloud-init, so I also inject our changes into cloud-init's template. Let's see if it works. |
Sorry for the (even longer) delay! Got very busy with school and other projects. Still doesn't work for some reason. I have not been able to figure out exactly why it doesn't work either; seems very, very weird. The sources list file:
The ports.list file:
|
eb3126b
to
8ac3b4e
Compare
Ok. What happens is the cloud-init template has changed its location, so the anti-race code doesn't work. I've pushed new version. Please try again. (Note: this PR doesn't include changes for Noble yet) |
With this PR (and also PR 79, which I dont think should have changed anything for this, but mentioning incase it is relevant), I can now build things like |
Seems like cloud-init can update sources.list after us. Make sure it will include our changes by injecting our changes into cloud-init too.
8ac3b4e
to
a7e876e
Compare
New version of Ubuntu LXD images seem to have a slightly different sources line. Account for those by making the trailing slash optional.
Fixes: #71