List of all per-Cask todo comments

[iggyvolz]

Below is a list of all todos inside of Homebrew Cask, sorted by todo name and file. For those curious, here's how I generated it: https://gist.github.com/iggyvolz/93f17a54ad2e22077fc0

"Improve this machine-generated value" has thousands of uses so it is simply the number that is listed.
bittorrent-sync.rb

• "response was not XML" in bittorrent-sync.rb
• "glob/expand needed here" in clamxav.rb
• "the URL indicates there is an appcast" in colorpicker-propicker.rb
• "the nested_container hack must be revised when container :nested stops being an alias around 0.50.0" in decitime.rb
• "should this be" in dwarf-fortress.rb
• "verify that this does not contain user-generate content" in filezilla.rb
• "is this user-created content?" in fontexplorer-x-pro.rb
• "This is a horrible hack […]. The backend code should be fixed" in git-annex.rb
• "remove all ENV variables" in gpgtools.rb
• "an uninstall stanza should be provided, and this message removed" in hear.rb
• "expand/glob for '~/Library//HipChat/'," in hipchat.rb
• "what is the reason for this?" in jabber-video.rb
• "(none provided)" in ksdiff.rb
• "unsure if these contain user-created content" in launchbar.rb
• "is "uninstal" below (one "l") a typo" in mcs783x.rb
• "verify that this does not contain user-created content" in ngrok.rb
• "these will not work because the glob will not be expanded" in programmer-dvorak.rb
• "verify that this does not contain user-generated content" in soulver.rb
• "glob/expand support" in soulver.rb
• "mistaken capital A in .app name?" in time-sink.rb

[Amorymeltzer]

I can tell you right off the bat that that last one does seem to be a mistake. Downloading it direct from http://manytricks.com/download/timesink is proper lowercase

[iggyvolz]

@Amorymeltzer See #8204.

[iggyvolz]

@Amorymeltzer Totally misread that post - scratch that issue above.

[iggyvolz]

@Amorymeltzer #8205 should be correct now. Don't know what I was thinking!

[vitorgalvao]

Thank you for this, @iggyvolz. Titles are a bit overwhelming, though, and make it a bit hard to actually skim the issues. Could you please inline the todo message with the checkbox and the cask/line number, or do something else to make it easier to see?

[iggyvolz]

When I initially started writing the code, I did it on the assumption that there were going to be tons of duplicate messages and I did that to save space. Done.

[Amorymeltzer]

Just to continue cleaning the list, improve this machine-generated value was recently added (see #7933 and #7972) to all casks with license :unknown; every single instance of that todo is in that context. Not that those aren't important (I clearly think so) but for the sake of readability they could easily be consolidated into one line (or removed since that's a lot of work for this sort of scope).

[vitorgalvao]

@Amorymeltzer is correct. The improve this machine-generated value todo should be removed, as it has a different context than the others.

[iggyvolz]

Done!

[Amorymeltzer]

Re: the globbing issues, there's discussion in #4759 that was trying to fix the programmer-dvorak issue, and appears to include code that would be informative for fixing the others (contrary to the docs?)

[iggyvolz]

Regarding jabber-video.rb, the appropriate commit is 36ee5bb.

"Remove ForcedConfig.plist after install - Required to set the server config manually."

[rolandwalker]

@iggyvolz thanks! I linkified the Casks for you, edited the subject line, and tagged this as a roadmap.

But I'd advise you to steer discussions away from this tracking bug. GitHub has no threading, there multiple subjects at hand, and people just don't tend to read all the way down a long issue.

[claui]

@iggyvolz Nice and useful roadmap! I just added one more item to the list:

"This is a horrible hack […]. The backend code should be fixed" in git-annex.rb

[podlech]

@iggyvolz , regarding Jabber Video, I've added the rm command in order cleanup the configuration. The reason behind this is that this version of Jabber is locked to http://collaboration.iu.edu students and this is the only way we can unlock it. I'm open to suggestions. Sorry for delay on my response.

[adidalal]

Re: ngrok - ngrok now creates a ~/.ngrok2 directory with an ngrok.yml inside. This is strictly a configuration file (examples here), and should be removed with a zap, but not necessarily an uninstall.

I can put in a pull request to update the formula if deemed appropriate.

[vitorgalvao]

Please do, @adidalal.

[vitorgalvao]

This was a great effort, but now most of these are the same (glob related), and some have been fixed and others added without touching the list, so closing as we have open issues for the big pictures.