#22 Text Message to be echoed in bashrc to make the Installer more accesible.
Merged by x3mboy. Opened by nasirhm.
i3-sig/ nasirhm/Fedora-i3-Spin text_message_bashrc  into  master

Download 22.patch

Added an echo statement in .bashrc for Liveuser to make it easier and accessible for people using the SIG to open the installer.

It'll be executed everytime, the user creates an interactive terminal session with bash (default) on Terminal

Currently, The message displays:
Please type liveinst and press Enter to start the installer.

Metadata Update from @jflory7:
- Pull-request tagged with: type - kickstart

The content of the change itself looks good to me. +1!

I have one question though. Is there a way we can define this just once in the kickstarts? Or is there a way that one kickstart can "inherit" the other as a base? My only thought would be that is nice if we can avoid repetition of similar instructions across multiple files. This increases the likelihood of making a change to one file and forgetting to change it in another.

I am not a kickstart expert though, so not sure if there is a clean way of doing this, or if we should revisit in a future PR.

The content of the change itself looks good to me. +1!

Thank you :)

I have one question though. Is there a way we can define this just once in the kickstarts? Or is there a way that one kickstart can "inherit" the other as a base? My only thought would be that is nice if we can avoid repetition of similar instructions across multiple files. This increases the likelihood of making a change to one file and forgetting to change it in another.

We're modified it only on one Kickstart (fedora-live-i3.ks), the other one is actually the flattened one.

I am not a kickstart expert though, so not sure if there is a clean way of doing this, or if we should revisit in a future PR.

There would be other ways to do it, I tried doing them with modifying the defaults. If there are other clean ways of doing it, we can revisit it in a next future PR though.

Pull-Request has been merged by x3mboy

Metadata