-
Notifications
You must be signed in to change notification settings - Fork 93
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
Slot type "file" gets unexpectedly suffixed with ".img" #316
Comments
Yes, since random files normally have a file ending that is not recognized as an image type, I found this the most convenient way. However, you are not the first one stumbling across this.
Yes, the expectation for bootloader images was that they are more likely to be supported by RAUC (and being
Yes, we have introduced the
For partitions that only have the purpose of holding the unpacked application directory, Artifact Updates may be the better solution in the future.
The Out of curiosity, how do you generate the application archive? Or is it just an integrated artifact deployed by a single recipe?
|
But this only applies to binaries. Text-based files e.g. may have various endings, which with
Ah yes, I meant
Yes, we currently do this via a single recipe generating an artifact that gets deployed to the |
Is there a specific reason why regular "file" slots have their filename get a
.img
appended?meta-rauc/classes-recipe/bundle.bbclass
Line 269 in 1242c33
This seems counter-intuitive to me, as I would expect for a slot with imgtype "file" to be left unmodified. This is the case for the imgtype "boot", but that does not fit e.g. file slots used for a user application. I could workaround this by specifying the
rename
slotflag, but this also looks weird:Without this workaround, my bundle has the following slot, which obviously fails to install:
As a proper fix I would suggest simply assigning the
imgsource
to the resultingimgname
:As an addition, I think we could add another imgtype called "image", which gets
.img
appended to the filename.What do you think about this? I think we had a similar discussion about this elsewhere but couldn't find any issue describing the reasoning behind it.
The text was updated successfully, but these errors were encountered: