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

OOBE Initial #36

Merged
merged 44 commits into from
Sep 28, 2018
Merged

OOBE Initial #36

merged 44 commits into from
Sep 28, 2018

Conversation

sirredbeard
Copy link

This implements the basics of an OOBE experience, building on the previous.

It drops some packages from the default install and moves them into optional packages.

It also incorporates some experimental package scripts like fzf.

The bash logic is still funky and needs work, if you could help me clean it up that would be great.

@sirredbeard sirredbeard added the To Implement This is something we should implement. label Sep 25, 2018
@sirredbeard sirredbeard added this to the 1.1.20 milestone Sep 25, 2018
@sirredbeard sirredbeard self-assigned this Sep 25, 2018
@@ -13,7 +13,7 @@ DIST="stable"
cd $TMPDIR

# bootstrap image
sudo cdebootstrap -a $ARCH --include=sudo,locales,git,ssh,apt-transport-https,wget,ca-certificates,man,less,xclip,gnome-themes-standard,gtk2-engines-murrine,dbus-x11,zsh,neovim,curl $DIST $DIST http://deb.debian.org/debian
sudo cdebootstrap -a $ARCH --include=sudo,locales,git,ssh,apt-transport-https,wget,ca-certificates,man,less,xclip,curl $DIST $DIST http://deb.debian.org/debian

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe move xclip to guiinstall? since xclip is actually a xserver application

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed.

@@ -0,0 +1,2 @@
#!/bin/bash
explorer.exe https://github.com/WhitewaterFoundry/WLinux

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe use wslview instead of explorer.exe? explorer.exe do not support https protocol when I tested it

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed.

@sirredbeard
Copy link
Author

I really need help on this with the logic and handling of the interactive prompts and making sure the command arg is passed and properly handled when script is re-run as an update.

In the long run we will probably move the individual package scripts into their own folder and files, but for now we will keep it monolithic.

@patrick330602
Copy link

I agree. I am currently trying to make something similar to yaourt, that let users to install with ease and let developers to build debian packages with ease. it is still in early stage, but I think in the future this would be a better idea to install some applications without deb packages.

@sirredbeard
Copy link
Author

I like the idea of adopting a consistent non-deb package system for packages not in an apt or deb, as opposed to our own scripts.

We should look more into that after we get bugs out of base monolithic OOBE and in place that at least duplicates our current package sets.

This was referenced Sep 25, 2018
@patrick330602
Copy link

I don't think international welcome should be added. Windows Console still have issue displaying CJK characters or characters from non-European countries, even it is using UTF-8 code page.
image

@sirredbeard
Copy link
Author

sirredbeard commented Sep 26, 2018

I really like the shell menu.

The y/n logic in setup is still funky, sometimes I say y it goes n, vice-versa. Can we nail that down?

I added powershell and filled in some other packages.

I also added pulseaudio support, for audio on WSL, thanks to CygWin32 and some voodoo.

I apologize if the code is messy, I just had to get it out of my head before bed.

If you can help clean it up I would really appreciate it.


# wget zip

wget https://brkp5a.dm.files.1drv.com/y4mzzXYR0K2wcJJhmeps-X7Crdn8qFBFaLBMBt4Ked1R9FBgKug7DUIiLsCgC0DumFtzos0a5RsikTLhzdcTyQ870sZojPNElVCVQueheYgH4YFbDTGOlkfU0iTyf8-ArW8UTTavCNuSrRoHc1Eu5Pcms8vM1T5knJn3rE_UREukPEDmHbcgRRop97gOjunaOyk/PulseAudio26sept18.zip

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I can put the file on my server if you want, it would be more easy to manage

@sirredbeard sirredbeard merged commit da093d9 into master Sep 28, 2018
@sirredbeard sirredbeard deleted the oobe-staging branch September 28, 2018 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
To Implement This is something we should implement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants