Skip to content
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

Quality-of-life and typo fixes in several files. #307

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

oom-is
Copy link

@oom-is oom-is commented Oct 15, 2019

This commit/pull request fixes small typos in several files and makes small changes to behavior
that this developer found useful.

Colorized text (Red for failure, green for unlocked) when LinuxPBA runs
modified: LinuxPBA/UnlockSEDs.cpp
Added "sleep 5" after LinuxPBA runs, so that users can view unlock success or failure
(particularly useful with multiple disks being unlocked)
modified: images/buildroot/32bit/overlay/etc/init.d/S99PBA.sh
modified: images/buildroot/64bit/overlay/etc/init.d/S99PBA.sh

Typo fixes as well as changes to move "clone buildroot" into the getresources script
modified: images/buildpbaroot
modified: images/buildrescue
modified: images/getresources

Typo fixes, including all the changes from Pull Request 254 (jessehui)
modified: linux/PSIDRevert_LINUX.txt
modified: windows/PSIDRevert_WINDOWS.txt

oom-is and others added 3 commits October 15, 2019 12:28
This commit/pull request fixes small typos in several files and makes small changes to behavior
that this developer found useful.

Colorized text (Red for failure, green for unlocked) when LinuxPBA runs
	modified:   LinuxPBA/UnlockSEDs.cpp
Added "sleep 5" after LinuxPBA runs, so that users can view unlock success or failure
	modified:   images/buildroot/32bit/overlay/etc/init.d/S99PBA.sh
	modified:   images/buildroot/64bit/overlay/etc/init.d/S99PBA.sh

Typo fixes as well as changes to move "clone buildroot" into the getresources script
	modified:   images/buildpbaroot
	modified:   images/buildrescue
	modified:   images/getresources

Typo fixes, including all the changes from Pull Request 254 (jessehui)
	modified:   linux/PSIDRevert_LINUX.txt
	modified:   windows/PSIDRevert_WINDOWS.txt
Typo fixes (parameter) in DtaDevOpal.cpp and DtaOptions.h
QOL/nit bug fixes in DtaOptions.cpp:
  - Command line option --enableUser to SEDutil had incorrect number of parameters (was 2 now 3)
  - Command line option --enableUser wasn't documented as part of the "usage" printout
…scue

Several SED vendors, specifically including Seagate and Micron, provide Linux CLI-based utilities
to manage their SEDs.  This update to buildrescue provides an easy way to insert those utilities
into the Rescue{32,64} images that are produced, so they're available when Rescue becomes a need.

Create a subdirectory sedutil/images/vendor and copy any CLI utilities there, and they'll be
available under /usr/vendor on the Rescue image.

Additional notes are embedded as comments in the buildrescue script.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant