Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

Non-translated strings #527

Open
lurch opened this issue Jan 9, 2019 · 5 comments
Open

Non-translated strings #527

lurch opened this issue Jan 9, 2019 · 5 comments

Comments

@lurch
Copy link
Collaborator

lurch commented Jan 9, 2019

The QMessageBox here doesn't use the tr stuff, and so will always be displayed in English, even if NOOBS is set to a different language.

@maxnet
Copy link
Collaborator

maxnet commented Jan 9, 2019

That's an internal error that should never occur though.

The fact that it sometimes does is probably a bug here: https://github.com/raspberrypi/noobs/blob/master/recovery/main.cpp#L85
Probably needs a && !devname.endsWith("0") to catch partitionless SD cards.

@procount
Copy link
Contributor

What about partitionless USB sticks? Will they need an extra condition?
(I imagine some users will be able to create such things!)

@lurch
Copy link
Collaborator Author

lurch commented Jan 10, 2019

Looks like the code already skips e.g. /dev/sda (because we'd want /dev/sda1) and I think the change @maxnet is suggesting might mean it would also skip /dev/mmcblk0 (because we'd want /dev/mmcblk0p1)

@maxnet
Copy link
Collaborator

maxnet commented Jan 10, 2019

Not skip, but include.
It should also look on mmcblk0

@lurch
Copy link
Collaborator Author

lurch commented Jan 10, 2019

Ah, I only skimmed the code quickly 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants