Tiny Core Base > TCB Bugs
mnttool: notice on failed umount, sync w/ cli umount?
dentonlt:
This is probably a curaga type question:
Continuing on the mnttool edits ...
Currently there is no warning/notice when mnttool fails to umount. mnttool just continues to display green (mounted). I'd be interested in a brief popup that either says "I can't umount this" or even better "foo and faw have this mount point open." I'm happy to add that code - it should be straightforward to handle the failed system("umount foo") call.
Second, mnttool seems to get out of sync with /mnt/ if the user calls umount or mount via the CLI. Some mount points are displayed green/red when they're not ... or just altogether don't exit. I'm happy to investigate and put in a patch myself.
Main concern is whether the team considers these adding cruft! Else I'll get to it.
Rich:
Hi dentonlt
--- Quote ---Currently there is no warning/notice when mnttool fails to umount. mnttool just continues to display green (mounted).
--- End quote ---
The fact that it stays green is your warning.
--- Quote ---Second, mnttool seems to get out of sync with /mnt/ if the user calls umount or mount via the CLI.
--- End quote ---
Well sure, you're changing mounted items behind mnttools back. Clicking the Refresh button should bring it back in sync.
curaga:
Yeah, changing behind its back is not supposed to work really. A popup for failure would be welcome.
nitram:
Haven't looked at mnttool and not an experienced coder, just my thoughts. For me popups not appreciated. What about incorporating an idle time out every 10-15 seconds that auto-refreshes mounts when mnttool open. If mismatch between colour indicator and mount results, indicator flashes red/green [edit] or turns yellow [/edit] to notify user of a problem. Cherry on top would be to check if ntfs-3g is loaded prior to mounting NTFS partitions, then mount them with ntfs-3g command so they are read/write not just read-only. Thanks for your work dentonlt.
dentonlt:
--- Quote ---The fact that it stays green is your warning.
--- End quote ---
Ha! I'm surprised I hadn't thought of it that way ... You're right, of course.
--- Quote ---Yeah, changing behind its back is not supposed to work really.
--- End quote ---
Maybe that's the main problem I should address. Or at least first.
--- Quote ---Cherry on top would be to check if ntfs-3g is loaded prior to mounting NTFS partitions, then mount them with ntfs-3g command so they are read/write not just read-only.
--- End quote ---
That's a novel idea. I guess the main question is whether that's suitable to include that sort of code inside of stock/base TC? I don't know if we have other tools/areas that add/extend features dependent upon the presence of optional binaries ... ? At face value, that sounds good to me.
Navigation
[0] Message Index
[#] Next page
Go to full version