Skip to content
This repository has been archived by the owner on Feb 8, 2021. It is now read-only.

All in one PR #6

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

All in one PR #6

wants to merge 5 commits into from

Conversation

imdjh
Copy link
Contributor

@imdjh imdjh commented Oct 8, 2015

As the commit comment says.
Fix #4

imdjh added 4 commits October 8, 2015 14:44
Renamed this function to check_initsystem for better explanation via its own name.
Remove useless logic.
I have notice that `show_message error` has always comes with an exit, so I merge the exit func into show_message.
You can use it with `show_message error ${ERR_TWO_ELEMENT_ARRAY[@]}`

I have noticed that the original implement on display_message always comesafter an error message. Since it's not the case user can easily solve themself, why not implement a fatal message and it would give the (almost) same result.
You can use it with `show_message fatal ${ERR_EXIT_CODE}`

Also fixed some typo and missleading message in this commit.
Install from remote -> Start fetching hyper package
Installing          -> Installing hyper and hyperd

Since __done__ is a reserve keyword in bash, I personally recommand remove it(as I did).
@imdjh
Copy link
Contributor Author

imdjh commented Oct 8, 2015

BTW: I'd like to have a T-shirt or something of your team mentioned at kernelpanic: podcast ;)

@carmark
Copy link
Contributor

carmark commented Oct 9, 2015

@imdjh

Thanks for your contribution, could you please send your address and size to me?
My email is [email protected]

@imdjh
Copy link
Contributor Author

imdjh commented Oct 9, 2015

@carmark

I have sent my requesting from the email which is used to committing. Let me know if you received that.

@imdjh
Copy link
Contributor Author

imdjh commented Oct 9, 2015

I have Introduced a better solution for #4 with new commit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants