new Korn Shell based installer. #99
Draft
+2,131
−277
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
klausz65
force-pushed
the
new-installer
branch
from
8161281 to
94092a7
Compare
mtelka
suggested changes
Dec 2, 2025
| # Copyright (c) 2008, 2010, Oracle and/or its affiliates. All rights reserved. | ||
| # | ||
| """boot_archive_archive - Release the boot arhive mount and archive the | ||
| """boot_archive_archive - Release the boot archive mount and archive the |
There was a problem hiding this comment.
This is not related to korn shell. Please create this as a separate PR.
|
|
||
| # ============================================================================= | ||
| # ============================================================================= | ||
| # sparc_usb_confiure - Configure a populated boot archive area including usr |
Author
There was a problem hiding this comment.
Thanks, has been left over from early devlopment - changed.
| # into /mnt/misc which will cause the installer's attempt to | ||
| # create a user account to fail | ||
| rm ${PKG_IMG_PATH}/etc/.pwd.lock | ||
| [ -f ${PKG_IMG_PATH}/etc/.pwd.lock ] && rm ${PKG_IMG_PATH}/etc/.pwd.lock |
Author
There was a problem hiding this comment.
The test is only there, because during development sometimes the file has been created, and sometimes not. If the underlying processes didn't it created it, there was always the error message: no such file or directory. I just wanted to not complain about the missing file if it hasn't been there in the first place.
There was a problem hiding this comment.
... and rm -f should handle that properly and silently.
|
|
||
| # now after the lofi filesystems are active, get rid of | ||
| # the ones in the actual root filesystem and hope it also | ||
| # works on old Ultra1 and or Ultra10 systems. |
klausz65
force-pushed
the
new-installer
branch
4 times, most recently
from
2c8f8d8 to
52267fc
Compare
klausz65
force-pushed
the
new-installer
branch
4 times, most recently
from
d17c2c8 to
8704d50
Compare
klausz65
force-pushed
the
new-installer
branch
3 times, most recently
from
df809e8 to
5028d38
Compare
klausz65
force-pushed
the
new-installer
branch
from
5028d38 to
464002e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For discussion about the new installer - replacing the need for the Python text-based installer, Note nothing from the python installer has been removed in this PR, just as a proposal for discussion about future install possibility.