Add many, many plugins and settings #102

thegranddesign · 2017-01-08T19:08:13Z

This is a fairly massive PR, which isn't exactly good form, but:

I put in a ton of commits so it should be fairly straightforward to follow
Many of the changes required changes on every plugin, which would have been extremely difficult to PR
I wanted to get this done

My main reason for wanting to get these into m-cli is simply because I wanted them out in the community so that fixes could be done in one place rather than in 90,000 dotfiles strewn across Github.

These changes focus on defaults style commands. I've spend dozens (maybe hundreds) of hours collecting these over the years such that I can now boot up a fresh OS install, run a script, reboot and it's set up exactly as I'd like. Hopefully now that these are all out in the community, we can stop doing these in our own silos and instead combine our resources.

There are things about this PR that could be improved (such as the way the "current" value of some of these commands are output) however (not counting the untold hours I spent actually finding which plist files changed when a setting changed), I've already spend three full days converting my old scripts to m-cli and I can't devote any more resources to this.

I definitely think it's mergable. I cause no (that I know of) breaking changes so it should be a minor Semver release.

And so that it always wipes out all items.

The previous `show_delay` was incorrect. It was changing the speed of the transition, not the delay before the transition started. I left `show_delay` in there and mapped it to the (incorrect) `auto_hide_speed` for backwards compatibility but removed reference to it from the README.

There's quite a bit of duplication among many of these functions. I'm going to add a few helper functions to clean some of it up. It will make adding new plugins easier as well as make the user experience more consistent. Not to mention cutting down on possible bugs.

This was much too prone to manual error. For now I just went through the commands that were predominantly 'defaults write' commands as that what I'm currently the most focused on. I also removed the duplication of the restarting of processes and made it happen after any command (which has been given a value) has been run.

thegranddesign · 2017-01-09T17:24:13Z

@rgcr thanks. :) Unfortunately I've spent way too much time on this already. I ran most of the commands against a fresh OS install of Sierra and it was about a 90% success rate. Which may mean that some of the settings need version checks (100% of them worked in 10.10).

I wish I could contribute further, but someone else is going to need to run it over the finish line.

thegranddesign · 2017-01-09T17:26:30Z

Additionally, some review will be needed to decide which of these commands need which process to be killed. I run this as a one time thing after a fresh OS install and so all I do is restart. I've never taken the time to try to make it work after each individual command is run.

rgcr · 2017-01-09T18:35:30Z

Don't worry, I'll work on it as soon as I can

shpoont · 2017-09-12T16:43:56Z

This PR is 9 months old, time to give birth :)

rgcr · 2017-09-13T22:44:34Z

I'm going to work on this, but I think we should keep it as simple as we can, I mean every 'plugin' should be independent of each other, so I will implement all plugins of this PR but in a simplest way, I think there are a lot of helper functions that are not necessary.

shpoont · 2017-09-16T20:31:11Z

@rgcr I could help you with separating those into PRs if you can provide a base those plugins should use, e.g. necessary helpers etc.

rgcr · 2017-09-18T23:39:48Z

@shpoont, That would be awesome and very helpful, I would like to remove just the dependency of lib/defaults.sh and lib/converters.sh on all plugins, add the new ones and add the new functionalities in the old ones.

shpoont · 2017-09-25T12:24:52Z

@rgcr sounds good, let me know when its ready? I'll then pick one of the plugins and make a test PR

thegranddesign · 2017-10-02T20:51:21Z

@rgcr I really don't understand why you would want to remove the "dependency" of those files. You have (in the current codebase) many multiples of places where those exact things are happening. Which means you have many places where you have the possibility of a bug. In fact, when I went through and refactored, I found many bugs where code had been copy/pasted wrong or a bug had been fixed in one place, but not fixed in the other half dozen places where that "same" code was being used.

Using those lib files, if there's a bug, you fix it in one place and it fixes it everywhere. Having those helper functions is just good DRY methodology.

Not to mention, when you remove all the parsing complexity, it makes the actual plugins themselves a lot easier to read and understand.

thegranddesign · 2017-10-02T20:58:36Z

Just as an example:

Before

m-cli/plugins/finder

Lines 27 to 57 in ba3af98

    
           hidden_files(){ 
        
               case $1 in 
        
                   [yY][eE][sS]) 
        
                       echo "Show hidden files: YES" 
        
                       defaults write com.apple.finder AppleShowAllFiles -bool true 
        
                       ;; 
        
                   [nN][oO]) 
        
                       echo "Show hidden files: NO" 
        
                       defaults write com.apple.finder AppleShowAllFiles -bool false 
        
                       ;; 
        
                   *) 
        
                       HIDDEN_FILE_STATUS=$(defaults read com.apple.finder AppleShowAllFiles 2>/dev/null) 
        
                       case $HIDDEN_FILE_STATUS in 
        
                           0|[nN][oO]|[fF][aA][lL][sS][eE]) 
        
                               HIDDEN_FILE_STATUS="NO" 
        
                               ;; 
        
                           1|[yY][eE][sS]|[tT][rU][eE]) 
        
                               HIDDEN_FILE_STATUS="YES" 
        
                               ;; 
        
                           *) 
        
                               echo "We can't read AppleShowAllFiles property" && exit 1 
        
                               ;; 
        
                       esac 
        
                       echo "Show hidden files: $HIDDEN_FILE_STATUS" 
        
                       exit 0 
        
                       ;; 
        
               esac 
        
               killall Finder 
        
           }

After

hidden_files(){
    value="$(_mcli_defaults_yes_no_to_boolean "com.apple.finder" \
                                              "AppleShowAllFiles" \
                                              "$1")"

    echo "${command} ${subcommand}: ${value}"
}

josh1703658784 · 2019-03-17T00:04:36Z

@rgcr This is really awesome. What kind of work would you like to see on it to get it into a merge-able state?

bensleveritt · 2019-05-08T09:02:24Z

I'd like to pitch in with getting this stuff in. I agree with @thegranddesign that this would remove a lot of my need to refer to various dotfiles. Just need a little guidance on direction from @rgcr.

rgcr · 2019-05-15T15:09:32Z

@bensleveritt go ahead, at this moment I don't have enough time work on it, just resolve the conflicts and be sure everything works as expected even the completions.

bensleveritt · 2019-05-15T19:57:57Z

Looking at all this, I'm tempted to break this into lots of little PRs.

If you're happy to accept the utilities lib as the first PR @rgcr, it'll enable me to go through each plugin and supply any changes that @thegranddesign did. This'll make reviewing a lot easier, and technically safer.

Are you happy with that approach?

My biggest reservation is losing credit for @thegranddesign for all the work they did.

rgcr · 2019-05-15T20:08:06Z

@bensleveritt, Yes I think that is the best approach. And don't worry I'll update the README to give credit to him and you. 😉

thegranddesign · 2019-06-11T23:54:49Z

@bensleveritt trying to break this up into multiple PRs is going to be a nightmare. There a ton of extraction to create the helper functions. The commits are already in "story order". I don't see how creating a bunch of PRs makes it any more manageable. Just look at the commits in order and approve them.

bensleveritt · 2019-06-12T07:00:18Z

@thegranddesign Haa.. You're not wrong about it being a lot of work, but the fact this PR has been stuck for over two years is a good indication that it's asking too much of folks to review. You yourself recognize it's bad form, and yet still expect others to do the work.

Since this PR, there have been other submissions for the same or similar functionality, and there are conceptual conflicts, as well as the obvious technical conflicts.

My intention is to reduce all of your changes into plugin-specific PRs, so if any parts aren't accepted, the others still can be. It's important that discussion happens in the context of the changes.

To be clear, the work you've committed is laudable, I'm just breaking it into more acceptable chunks.

thegranddesign · 2019-06-12T23:55:49Z

@bensleveritt To be clear, I don't expect anyone else to do the work, I simply said I had already spent multiple weeks on the code as well as writing commits to make it easy to review. And that I wasn't going to take more time to split it up. This could have been merged as it was with no additional work on anyone's part (other than review) 2.5 years ago. This PR caused almost zero breaking API changes at the time. It could have easily been merged and then tweaked post-merge.

For my part I put it up here to give back, and it's been disappointing not seeing it get merged, but my fork works great for everything I use it for and that's good enough for me, and anyone else who wants to use that instead.

I definitely wish you good luck.

rgcr · 2019-06-13T16:37:18Z

@thegranddesign as you wrote, that was a lot of work, unfortunately as you I didn't have too much time to review the PR at that time , I appreciate your work and the time you spent on this.

@bensleveritt thank you for being doing the hard work here.

Once all changes are reviewed and merged I'll close this PR

willfaught · 2020-08-14T02:15:40Z

@rgcr Any update on this?

bensleveritt · 2020-08-22T08:24:03Z

@rgcr Any update on this?

Ah, it's been a while since I've used m. Last I recall working on was working out which modules could be imported which don't already exist. Then existing modules could be checked to see if there was anything to port across.

bensleveritt · 2020-08-22T08:45:23Z

To this end, I've got a board of progress started https://github.com/bensleveritt/m-cli/projects/1 which I'll update as I move the plugins across.

The Grand Design added 30 commits January 5, 2017 15:05

Add airdrop plugin

c00bbd5

Add airport plugin

006986b

Add debugmode plugin

1e0a7d3

Add lightsensor subcommand to display

febeb9d

Fix dock pruning so that it always works

ae21fa6

And so that it always wipes out all items.

Allow the dock to be completely disabled

7d0817f

Add magnificationsize to the dock

70759c9

Add iconsize to the dock

0b95915

Add bounce effects to the dock

06bcbab

Whitespace

a9d1b71

Add active indicators to dock

9d7f6cf

Add more helper functions to remove a bunch of duplication

5f6fdb9

Clear more duplication by creating more helpers

f9090f3

Add hidden app dimming to the dock

7e09ee0

Add only show running to the dock

c448b24

Add full screen delay to the dock

db9b798

Add iTunes notifications to the dock

e8c6271

Change helpers to converters to be more specific

0638d9f

Add keyboard plugin

724c018

Add tags to gitignore

883160a

Add spotlight plugin

07d85d3

Add tooltips plugin

f153ae8

Add password delay to screensaver

c7fab51

Add bannertime to notification

508218c

Add mission control

4259f41

Add hot corners plugin

0c8f545

Add filevault plugin

6823062

rgcr added the help wanted label Jan 9, 2017

rgcr added In progress enhancement and removed help wanted labels Jan 24, 2017

bensleveritt mentioned this pull request May 15, 2019

Adds utility functions in preparation for refactoring #138

Merged

This was referenced Aug 24, 2020

Add animations #156

Open

Add dialogs plugin #157

Open

huyz mentioned this pull request Sep 18, 2023

Project dead? #197

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add many, many plugins and settings #102

Add many, many plugins and settings #102

thegranddesign commented Jan 8, 2017

thegranddesign commented Jan 9, 2017

thegranddesign commented Jan 9, 2017

rgcr commented Jan 9, 2017

shpoont commented Sep 12, 2017 •

edited

rgcr commented Sep 13, 2017 •

edited

shpoont commented Sep 16, 2017

rgcr commented Sep 18, 2017 •

edited

shpoont commented Sep 25, 2017

thegranddesign commented Oct 2, 2017

thegranddesign commented Oct 2, 2017 •

edited

josh1703658784 commented Mar 17, 2019 •

edited

bensleveritt commented May 8, 2019

rgcr commented May 15, 2019

bensleveritt commented May 15, 2019 •

edited

rgcr commented May 15, 2019

thegranddesign commented Jun 11, 2019

bensleveritt commented Jun 12, 2019

thegranddesign commented Jun 12, 2019 •

edited

rgcr commented Jun 13, 2019

willfaught commented Aug 14, 2020

bensleveritt commented Aug 22, 2020

bensleveritt commented Aug 22, 2020

Add many, many plugins and settings #102

Are you sure you want to change the base?

Add many, many plugins and settings #102

Conversation

thegranddesign commented Jan 8, 2017

thegranddesign commented Jan 9, 2017

thegranddesign commented Jan 9, 2017

rgcr commented Jan 9, 2017

shpoont commented Sep 12, 2017 • edited

rgcr commented Sep 13, 2017 • edited

shpoont commented Sep 16, 2017

rgcr commented Sep 18, 2017 • edited

shpoont commented Sep 25, 2017

thegranddesign commented Oct 2, 2017

thegranddesign commented Oct 2, 2017 • edited

Before

After

josh1703658784 commented Mar 17, 2019 • edited

bensleveritt commented May 8, 2019

rgcr commented May 15, 2019

bensleveritt commented May 15, 2019 • edited

rgcr commented May 15, 2019

thegranddesign commented Jun 11, 2019

bensleveritt commented Jun 12, 2019

thegranddesign commented Jun 12, 2019 • edited

rgcr commented Jun 13, 2019

willfaught commented Aug 14, 2020

bensleveritt commented Aug 22, 2020

bensleveritt commented Aug 22, 2020

shpoont commented Sep 12, 2017 •

edited

rgcr commented Sep 13, 2017 •

edited

rgcr commented Sep 18, 2017 •

edited

thegranddesign commented Oct 2, 2017 •

edited

josh1703658784 commented Mar 17, 2019 •

edited

bensleveritt commented May 15, 2019 •

edited

thegranddesign commented Jun 12, 2019 •

edited