search

canonical / core18

The core18 base snap
14 stars 26 forks source link

run-snapd-from-snap: fix quoting for firstboot #22

Closed mvo5 closed 6 years ago

mvo5 commented 6 years ago

The shell quoting is too strict so the current script will not find the seeded snapd.

sil2100 commented 6 years ago

Actually, let me think about it for a moment - will there ever be such a 'buggy' situation where there will be two or more snapd snaps in the /var/lib/snapd/seed/snaps/ directory? The following code won't work if there's more than one file found by 'find'. Should we worry about that?

mvo5 commented 6 years ago

@sil2100 If there are multiple snapd in the seed our seeding code is buggy. We could add code here though that gives a clear error if it finds multiple ones. Happy to do this.

sil2100 commented 6 years ago

I guess it's not necessary - we'll know instantly when that happens. Thanks!