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

Get image to build; alez command on booted image still has zsh permis… #70

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

Conversation

pmarreck
Copy link

@pmarreck pmarreck commented Oct 10, 2021

…sions error

Description

I got the image to build by adding a proper profile definition for mkarchiso, and passing it in as an argument in the Dockerfile.
Unfortunately, after booting the resulting image, running alez returns a zsh permissions error, even as root. That's as far as I got so far.

Testing

  • Tested UEFI install to successful boot
  • Tested BIOS install to successful boot

Code

Note: Shellcheck returned the following, but not from my changes:

$ shellcheck *.sh

In alez.sh line 399:
            --menu "$3" $HEIGHT $WIDTH $COUNT "${PARAMS[@]}" \
                                       ^----^ SC2086: Double quote to prevent globbing and word splitting.

Did you mean: 
            --menu "$3" $HEIGHT $WIDTH "$COUNT" "${PARAMS[@]}" \

For more information:
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
  • When necessary, comments have been added in hard-to-understand areas
  • The changes generate no shellcheck warnings or errors.

…runs the resulting ISO using qemu (new deps are listed in the test file)
…ional output when debugging or testing. Added missing 'dialog' dependency. Installed archlinux-keyring dependency as part of key initialization
…sh strict mode switches. Fixed bugs. alez still fails on partition selection when testing this way, though... possibly idiosyncratic disk naming in qemu that the script doesn't recognize, still investigating...
…ssatisfied with the over-engineered options already available
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.

None yet

1 participant