Guru Swarupa
Guru Swarupa
> > why would you move grub theme setup to application ? and now renaming it to grub? > > grub was already in applications-setup, i never moved it Someone...
> @guruswarupa ... It seemed to work when those lines which you changed were echo. Later while changing it to printf, it broke. I did test on my system before...
> @guruswarupa Too many scripts are being added here; you should move that ssh script to a new PR so this PR is easier to review. @nnyyxxxx only 4 :)...
@ChrisTitusTech @nnyyxxxx i've fixed those conflicts.
> This method does not work with all desktop environments. Add a check for the current desktop environment yes it works on kde plasma, gnome, xfce. doesn't work for window...
> This method does not work with all desktop environments. Add a check for the current desktop environment could you name the desktop environment in which it doesn't work?
> After taking a look at all of the scripts in monitor-control it seems like there are a lot more bashisms than we anticipated, I'm going to make a PR...
> > > After taking a look at all of the scripts in monitor-control it seems like there are a lot more bashisms than we anticipated, I'm going to make...
at first when i pushed, it was in bash. Later they changed it cause it was not working in ubuntu to use sh instead of bash :)
@jeevithakannan2 squash those commits to 1 as its just 1 feature your working on.