Skip to content

Add *HELP, *TYPE, *CLS, *MOUNT Commands#60

Closed
leighbb wants to merge 9 commits intobreakintoprogram:mainfrom
leighbb:commands
Closed

Add *HELP, *TYPE, *CLS, *MOUNT Commands#60
leighbb wants to merge 9 commits intobreakintoprogram:mainfrom
leighbb:commands

Conversation

@leighbb
Copy link
Copy Markdown
Contributor

@leighbb leighbb commented May 24, 2023

In order to make a more comfortable MOS environment, I have added the following commands:
*HELP - Display help on commands
*TYPE - Like the equivalent DOS command, displays a text file on the console
*CLS - shortcut to VDU 12 (which not all users may be familiar with)
*MOUNT - Re-(mount) the MicroSD card

Happy for any feedback.

Leigh Brown and others added 5 commits May 24, 2023 12:26
Move fs variable from main.c to src/mos.c
Create new mos_mount() function in src/mos.c
Call mos_mount() from main() instead of directly calling f_mount
Add mos_cmdMOUNT() that calls mos_mount() in src/mos.c
Add additional commands to list of commands in HELP text.
@leighbb
Copy link
Copy Markdown
Contributor Author

leighbb commented May 24, 2023

Example showing use of MOUNT after MicroSD card removed and re-inserted:
image

@leighbb
Copy link
Copy Markdown
Contributor Author

leighbb commented May 24, 2023

Example showing *HELP ALL:
image

@leighbb
Copy link
Copy Markdown
Contributor Author

leighbb commented May 24, 2023

Finally, using the TYPE command to display a text file:
image

@breakintoprogram breakintoprogram added the enhancement New feature or request label Jun 27, 2023
@leventp
Copy link
Copy Markdown

leventp commented Jul 2, 2023

This looks good. I think the '*' char should not be displayed in the help text. After all , it is the MOS prompt and not needed to be typed in, and making the text a bit confusing.

@leighbb
Copy link
Copy Markdown
Contributor Author

leighbb commented Jul 2, 2023

This looks good. I think the '*' char should not be displayed in the help text. After all , it is the MOS prompt and not needed to be typed in, and making the text a bit confusing.

Fair enough - updated patch to remove the *'s

@breakintoprogram
Copy link
Copy Markdown
Owner

Hi, this has been merged as part of pull request #93 so am closing as it is now done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Won't Do

Development

Successfully merging this pull request may close these issues.

3 participants