Add documentation for brew linkapps

[ELLIOTTCABLE]

Although it's in the contribs/ folder, it seems like a rather important command; so I've added documentation for it to the manpage.

Mind you: it's currently documented with the assumption that #22378 and #22379 are accepted as-is, for the most part. I'll be happy to refactor this documentation to reflect whatever decisions are made therein.

[adamv]

Since this command is already referenced in certain caveats, we might as well make it official (and not contrib); which means we can also rename it (once!) at the same time if anyone has a better name for this.

[ELLIOTTCABLE]

Any argument against just brew apps?

What about moving it into ln, as brew link --apps? Since that's exactly what it's doing, linking the build-products into a central, generally-accessible directory.

[adamv]

I see the appeal in adding an option to brew link, but don't really want to overload it that way. But would like to hear other maintainers opinions on that.

[adamv]

Also apps works for me so far.

Will also want to add this to the (bash) autocomplete.

[MikeMcQuaid]

brew apps works for me.

[jacknagel]

Putting my end-user hat on, I wouldn't really expect a command named "apps" to "do stuff" since "apps" is not a verb; it's closer to "uses", "deps", or "outdated". What's wrong with keeping "linkapps" (then nobody has to update scripts or aliases or whatever)?

[ELLIOTTCABLE]

I think @adamv is making the point that, if anybody wants to rename it, now is the time. What with it being moved around the source-tree anyway.

My vote is for removing it entirely, in favour of a flag; but that said, I think the sanest route is probably Jack's. That is, not to move it at all.

—

Sent from Mailbox for iPad

On Mon, Sep 9, 2013 at 5:07 PM, Jack Nagel notifications@github.com
wrote:

Putting my end-user hat on, I wouldn't really expect a command named "apps" to "do stuff" since "apps" is not a verb; it's closer to "uses", "deps", or "outdated". What's wrong with keeping "linkapps" (then nobody has to update scripts or aliases or whatever)?

Reply to this email directly or view it on GitHub:
#22380 (comment)

[adamv]

Keeping the name is fine with me, just throwing the option out there.

[MikeMcQuaid]

Good points @jacknagel; linkapps is certainly more explicitly obvious.

[adamv]

Merge this documentation change into #22378, since it documents the new (as yet unpulled) behavior.

[adamv]

Closing in favor of #22378.