aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* | restyle fallback module to match project styleGravatar Kurtis Rader2016-04-30
| | | | | | | | | | | | | | | | | | | | Reduces lint errors from 36 to 33 (-8%). Line count from 1910 to 1476 (-23%). Another step in resolving issue #2902. This also fixes a stupid mistake from an earlier commit where I didn't realize that osx/config.h was meant to be included as a semi-static file in the repository.
* | Remove use of __environGravatar ridiculousfish2016-04-30
| | | | | | | | It has apparently never worked. Fixes #2988
| * don't use colors when writing the ^C indicatorGravatar Kurtis Rader2016-04-30
| | | | | | | | | | | | There was an extended discussion in https://github.com/fish-shell/fish-shell/issues/2904 about using a bright yellow background to make the cancelled command indicator, ^C, standout. The upshot was that standout (i.e., reversing fg/bg colors) mode should be used until themes are agumented with proper support for background colors and special characters. (cherry picked from commit a897ef002521a3991adc4342189f0e30a0d048b0)
* | don't use colors when writing the ^C indicatorGravatar Kurtis Rader2016-04-30
| | | | | | | | There was an extended discussion in https://github.com/fish-shell/fish-shell/issues/2904 about using a bright yellow background to make the cancelled command indicator, ^C, standout. The upshot was that standout (i.e., reversing fg/bg colors) mode should be used until themes are agumented with proper support for background colors and special characters.
* | Remove using_command from netctl completionsGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | This allows `; and netctl` to work. First step towards #2705.
| * Remove using_command from netctl completionsGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | This allows `; and netctl` to work. First step towards #2705.
| * git completion: Allow optional "+" for pushGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | | | | | | | | | This signifies a force-push. To avoid cluttering, only complete branches if a + is already given. Fixes #2879. (cherry picked from commit fde26d404956f843860f2bf5bb00dd3e1636632a)
| * Add repository/refspec completion to gitGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | | | | | | | A few commands (fetch, pull and push at least) take a "repository" (aka "remote") and then a "refspec" (we currently do branches here). Fixes #2525 (seems that man is still alive) (cherry picked from commit b32bf2261617d30019d25d04ce451a29e7297c14)
* | git completion: Allow optional "+" for pushGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | | | | | This signifies a force-push. To avoid cluttering, only complete branches if a + is already given. Fixes #2879.
* | Add repository/refspec completion to gitGravatar Fabian Homborg2016-04-30
| | | | | | | | | | | | | | A few commands (fetch, pull and push at least) take a "repository" (aka "remote") and then a "refspec" (we currently do branches here). Fixes #2525 (seems that man is still alive)
| * document new `fish_realpath` builtinGravatar Kurtis Rader2016-04-29
| |
| * provide a realpath implementationGravatar Kurtis Rader2016-04-29
| | | | | | | | | | | | | | | | | | Not all distros have a `realpath` command. Provide a function that uses the real command if available else use the fish builtin. Fixes #2932 (cherry picked from commit 6c329e8a839c1b5eeaf2e545b4f4084c3a8830f7)
| * fix bind unit testsGravatar Kurtis Rader2016-04-29
| | | | | | | | | | | | | | | | In my rush to get the fix for the wrong default Vi mode escape delay merged (commit 3e24ae80b3739ded7d4066349fc87d13f2b70727) I neglected to update the unit test. This change corrects that oversight. Cherry-picked from 5092904ea3a95c6ba78350c3b3761511f613fde8
| * clarify fish_vi_mode deprecation warningGravatar Kurtis Rader2016-04-29
| | | | | | | | | | | | | | | | | | Also, correct the Vi mode default escape timeout. I intended it to be 100 ms in my previous change but it ended up 10 ms which is far too short. A 10 ms delay will continue to cause problems for people running fish inside `screen`, `tmux`, or over high latency connections. Cherry-picked from 3e24ae80b3739ded7d4066349fc87d13f2b70727
* | Erase the autosuggestion in fish_cancel_commandline by clearing to EOLGravatar ridiculousfish2016-04-29
| |
* | Remove an errant newline in the fish_cancel_commandline outputGravatar ridiculousfish2016-04-29
| | | | | | | | | | Now the next line appears immediately after the cancelled line, without an intervening newline
* | Remove some unused macros from builtin_set_color.cppGravatar ridiculousfish2016-04-29
| |
* | Add some more files missing from Xcode compile stepGravatar ridiculousfish2016-04-29
| | | | | | | | | | Previously the .cpp files were #included by common.cpp Now they get compiled separately
| * Issue a \r in fish_title, except when executing it for the promptGravatar ridiculousfish2016-04-29
| | | | | | | | | | | | | | | | | | | | | | fish_title currently outputs some escaped text, which can confuse the line driver (#2453). Issue a carriage return so the line driver knows we are at the beginning of the line, unless we are writing the title as part of the prompt. In that case, we may have text from the previous command still on the line and we don't want to move the cursor. Fixes #2453
* | Issue a \r in fish_title, except when executing it for the promptGravatar ridiculousfish2016-04-29
| | | | | | | | | | | | | | | | | | | | | | fish_title currently outputs some escaped text, which can confuse the line driver (#2453). Issue a carriage return so the line driver knows we are at the beginning of the line, unless we are writing the title as part of the prompt. In that case, we may have text from the previous command still on the line and we don't want to move the cursor. Fixes #2453
* | Fix the Xcode buildGravatar ridiculousfish2016-04-29
| | | | | | | | Add missing files to link phase
| * git takes --help even when it needs a command (#2984)Gravatar Sanne Wouda2016-04-29
| | | | | | `git --help` is a valid command and fish should complete it as such
| * git completion: Only show unmerged branches for cherry-pickGravatar Fabian Homborg2016-04-29
| |
* | git takes --help even when it needs a command (#2984)Gravatar Sanne Wouda2016-04-29
| | | | | | `git --help` is a valid command and fish should complete it as such
* | fix bind unit testsGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | In my rush to get the fix for the wrong default Vi mode escape delay merged (commit 3e24ae80b3739ded7d4066349fc87d13f2b70727) I neglected to update the unit test. This change corrects that oversight.
* | clarify fish_vi_mode deprecation warningGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | | | Also, correct the Vi mode default escape timeout. I intended it to be 100 ms in my previous change but it ended up 10 ms which is far too short. A 10 ms delay will continue to cause problems for people running fish inside `screen`, `tmux`, or over high latency connections.
* | restyle expand module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Reduces lint errors from 183 to 126 (-31%). Line count from 2231 to 1787 (-20%). Another step in resolving issue #2902.
* | restyle exec module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Reduces lint errors from 121 to 59 (-51%). Line count from 1578 to 1290 (-18%). Another step in resolving issue #2902.
* | retyle event module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Reduces lint errors from 39 to 30 (-23%). Line count from 915 to 670 (-27%). Another step in resolving issue #2902.
* | restyle env_universal module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Reduces lint errors from 121 to 52 (-57%). Line count from 1916 to 1671 (-13%). Another step in resolving issue #2902.
* | restyle env module to match project styleGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Reduces lint errors from 90 to 72 (-20%). Line count from 1719 to 1298 (-24%). Another step in resolving issue #2902.
* | document how to config editors for this projectGravatar Kurtis Rader2016-04-28
| |
* | provide a realpath implementationGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | | | Not all distros have a `realpath` command. Provide a function that uses the real command if available else use the fish builtin. Fixes #2932
* | git completion: Only show unmerged branches for cherry-pickGravatar Fabian Homborg2016-04-28
| |
* | switch to newer flag --set-upstream-to for git (#2982)Gravatar Cody Scott2016-04-28
| | | | | | | | | | | | | | set-upstream was deprecated in git 1.8.0 as stated in [1] in favor of set-upstream-to. this patch replaces the old flag in fish completions [1]: https://raw.githubusercontent.com/git/git/master/Documentation/RelNotes/1.8.0.txt
| * add way to comment/uncomment a commandGravatar Kurtis Rader2016-04-28
| | | | | | | | | | | | Fixes #2375 (cherry picked from commit 2f8d0e9aba3662fcb1032ffe33bc6faf2e04eb8f)
| * fish.spec: hostname dependency for openSUSEGravatar David Adam2016-04-28
| | | | | | | | | | | | | | As discussed in https://github.com/fish-shell/fish-shell/commit/ed4b78918a944fe3b3a1e05ac553e89d5f1fa411 [ci skip]
* | restyle common module to match project styleGravatar Kurtis Rader2016-04-27
| | | | | | | | | | | | Reduces lint errors from 194 to 142 (-27%). Line count from 3352 to 2645 (-21%). Another step in resolving issue #2902.
* | restyle color module to match project styleGravatar Kurtis Rader2016-04-27
| | | | | | | | | | | | Reduces lint errors from 16 to 14 (-13%). Line count from 558 to 463 (-17%). Another step in resolving issue #2902.
* | compile fallback.cpp and util.cpp seperatelyGravatar Kurtis Rader2016-04-27
| | | | | | | | | | | | | | Don't `#include "*.cpp"` modules in other cpp modules. I already took care of all the builtin_*.cpp modules in my previous change where I restyled the builtin code. This change fixes the two remaining instances of this anti-pattern.
| * Update usage of pcre2_substitute() for pcre2-10.21Gravatar Michael Steed2016-04-27
| | | | | | | | | | | | | | | | | | - Set PCRE2_SUBSTITUTE_OVERFLOW_LENGTH to get the required buffer length from pcre2 instead of guessing - Set PCRE2_SUBSTITUTE_EXTENDED to enable extra goodies in the replacement string (cherry picked from commit c2f9d60eb1f04a3aa0b01d785c1e133cac8ecf1d)
| * configure: require at least pcre2-10.21Gravatar Michael Steed2016-04-27
| | | | | | | | (cherry picked from commit c4c7983497e7be798c3bd1c97df38e8d1d1a58b8)
* | env_universal_common.cpp: pass correct flag to fcntlGravatar David Adam2016-04-27
| | | | | | | | | | | | Closes #2955. (cherry picked from commit d5797c543982a0fbc8201ae21db5009e9447f75d)
| * env_universal_common.cpp: pass correct flag to fcntlGravatar David Adam2016-04-27
| | | | | | | | Closes #2955.
* | Make busctl completions usefulGravatar Fabian Homborg2016-04-27
| | | | | | | | | | | | | | | | | | - More accurate - Fast enough to be usable (previously, this would sometimes take a few seconds) - A bit smaller
| * Make busctl completions usefulGravatar Fabian Homborg2016-04-27
| | | | | | | | | | | | | | | | | | - More accurate - Fast enough to be usable (previously, this would sometimes take a few seconds) - A bit smaller
* | restyle builtin modules to match project styleGravatar Kurtis Rader2016-04-26
| | | | | | | | | | | | | | | | | | | | | | Now that the IWYU cleanup has been merged compile all, not just a couple, of the builtin modules independent of builtin.cpp. That is, no longer `#include builtin_NAME.cpp` in builtin.cpp. This is more consistent, more in line with what developers expect, and is likely to reduce mistakes. Reduces lint errors from 384 to 336 (-13%). Line count from 6307 to 4988 (-21%). Another step in resolving issue #2902.
* | Update usage of pcre2_substitute() for pcre2-10.21Gravatar Michael Steed2016-04-27
| | | | | | | | | | | | | | - Set PCRE2_SUBSTITUTE_OVERFLOW_LENGTH to get the required buffer length from pcre2 instead of guessing - Set PCRE2_SUBSTITUTE_EXTENDED to enable extra goodies in the replacement string
* | configure: require at least pcre2-10.21Gravatar Michael Steed2016-04-27
| |
* | add better support for IWYU and fix thingsGravatar Kurtis Rader2016-04-26
| | | | | | | | | | | | | | | | | | Remove the "make iwyu" build target. Move the functionality into the recently introduced lint.fish script. Fix a lot, but not all, of the include-what-you-use errors. Specifically, it fixes all of the IWYU errors on my OS X server but only removes some of them on my Ubuntu 14.04 server. Fixes #2957