protected grep from alias, join some sed commands and don't load profile if not needed#68
Closed
glhez wants to merge 1 commit intojuven:masterfrom
Closed
protected grep from alias, join some sed commands and don't load profile if not needed#68glhez wants to merge 1 commit intojuven:masterfrom
glhez wants to merge 1 commit intojuven:masterfrom
Conversation
Collaborator
|
@glhez did you test this on Linux and Mac? |
Author
|
Yep. Tested on:
Don't have a Mac. I've added two more commit:
Now it display (here under linux): Also, I think the |
88b795c to
5ca30e1
Compare
Author
|
I mainly replaced old commit due to wrong mail being referenced. It is also rebase against master for all that matter. |
…(rare) case of command being aliased, minimize shell invocation, don't read profile/maven file needlessly, ...)
Author
|
I'm closing my PR because of the newly drastic change I've made ... and the fact they were never considered since 2015... You may still read those:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello,
I've pushed 3 commits:
alias grep='grep --color -Hn', then the$common_pluginslocal will contains line starting with(standard input):<line number>:.-Por--active-profilesis used.Note: I used a trick of sed, which can remove the
\ndelimiter. eg:sed -e s/\n/\|/gdoes the same astr '\n' '|'. This works on Git for Windows, did not test on a Linux bash.