Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2015-01-02 Thread Tomasz Pala
On Wed, Dec 10, 2014 at 14:38:20 +0100, Jacek Konieczny wrote: if [ -r /etc/env.d/GREP_OPTIONS ]; then [...] >>> Why? >> >> For backward compat with what we had so far. > > Such a shell script for any interactive shell started, just because one > or two PLD users have uncommented the /etc/e

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2015-01-02 Thread Tomasz Pala
On Thu, Dec 11, 2014 at 12:27:12 +0200, Elan Ruusamäe wrote: >> $ grep alias/etc/shrc.d/*.sh >> [...] >> /etc/shrc.d/which.sh:alias which='alias | /usr/bin/which --tty-only >> --read-alias --show-dot --show-tilde' >> [...] > > "which" needs to be alias to handle shell builtins (--read-alias opti

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2015-01-02 Thread Tomasz Pala
On Wed, Dec 10, 2014 at 13:50:37 +0100, Jacek Konieczny wrote: > Implementing any such non-standard behaviour in the distribution is a > very bad idea. Not only we make the shell load ages to initialize > features most users don't even know about, but we may find great 1. Having zsh one can preco

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-11 Thread Elan Ruusamäe
On 10.12.2014 22:10, Adam Osuchowski wrote: Ok, so why there are still other aliases defined in /etc/shrc.d? For example: $ grep alias/etc/shrc.d/*.sh [...] /etc/shrc.d/which.sh:alias which='alias | /usr/bin/which --tty-only --read-alias --show-dot --show-tilde' [...] "which" needs to be alia

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-11 Thread Elan Ruusamäe
On 10.12.2014 21:38, Adam Osuchowski wrote: Elan Ruusamäe wrote: i totally dislike such alias being enabled by default. i have my own, and this one will ovewrite mine (because it's after entry in shrc.d) Really? Since when ~/.bashrc, ~/.zshrc, etc. is interpreted _before_ /etc/shrc.d? If you do

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Adam Osuchowski
Jacek Konieczny wrote: > Such a shell script for any interactive shell started, just because one > or two PLD users have uncommented the /etc/env.d/GREP_OPTIONS we might > have wrongly introduced long time ago? There are no difference between putting grep's options into alias and into $GREP_OPTION

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Adam Osuchowski
Elan Ruusamäe wrote: > i totally dislike such alias being enabled by default. i have my own, and > this one will ovewrite mine (because it's after entry in shrc.d) Really? Since when ~/.bashrc, ~/.zshrc, etc. is interpreted _before_ /etc/shrc.d? If you don't like system aliases unalias them or re

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Jacek Konieczny
On 10/12/14 14:14, Arkadiusz Miśkiewicz wrote: > On Wednesday 10 of December 2014, Jacek Konieczny wrote: >> On 10/12/14 12:46, Arkadiusz Miśkiewicz wrote: >>> Maybe something like that would be better? >>> >>> # cat /etc/shrc.d/grep.sh >>> if [ -r /etc/env.d/GREP_OPTIONS ]; then >>> >>> un

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Arkadiusz Miśkiewicz
On Wednesday 10 of December 2014, Jacek Konieczny wrote: > On 10/12/14 12:46, Arkadiusz Miśkiewicz wrote: > > Maybe something like that would be better? > > > > # cat /etc/shrc.d/grep.sh > > if [ -r /etc/env.d/GREP_OPTIONS ]; then > > > > unset GREP_OPTIONS > > . /etc/env.d/GREP_O

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Elan Ruusamäe
On 10.12.2014 14:50, Jacek Konieczny wrote: If anybody needs custom options to any shell command he can prepare an alias that suits him and his shell most. so as original changeset author is not making any movements (adwol), let's remove the aliases completely. i totally dislike such alias bei

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Jacek Konieczny
On 10/12/14 12:46, Arkadiusz Miśkiewicz wrote: > > Maybe something like that would be better? > > # cat /etc/shrc.d/grep.sh > if [ -r /etc/env.d/GREP_OPTIONS ]; then > unset GREP_OPTIONS > . /etc/env.d/GREP_OPTIONS > if [ -n "$GREP_OPTIONS" ]; then > alias

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-10 Thread Arkadiusz Miśkiewicz
On Monday 08 of December 2014, Elan Ruusamäe wrote: > On 08.12.2014 22:44, adwol wrote: > > commit 274761483d541b779ab9135532aecf8aadc550e8 > > Author: Adam Osuchowski > > Date: Mon Dec 8 21:41:18 2014 +0100 > > > > - move from GREP_OPTIONS environmental variable to alias due to its > >

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-08 Thread Jakub Bogusz
On Mon, Dec 08, 2014 at 10:30:47PM +0100, Adam Osuchowski wrote: > Elan Ruusamäe wrote: [...] > > this definately is not the same thing as it was before! restore previous > > behaviour by *not* enforcing such options! > > Any evidences, examples? It is very close to GREP_OPTIONS. The alternative

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-08 Thread Adam Osuchowski
Elan Ruusamäe wrote: > a reference to such "obsolescence" ? Did you try to run grep 2.21 with GREP_OPTIONS set even once? Please try it and search sources for message you will see, before asking such a question. > this definately is not the same thing as it was before! restore previous > behavio

Re: [packages/grep] - move from GREP_OPTIONS environmental variable to alias due to its obsolescence

2014-12-08 Thread Elan Ruusamäe
On 08.12.2014 22:44, adwol wrote: commit 274761483d541b779ab9135532aecf8aadc550e8 Author: Adam Osuchowski Date: Mon Dec 8 21:41:18 2014 +0100 - move from GREP_OPTIONS environmental variable to alias due to its obsolescence a reference to such "obsolescence" ? -cat << EOF >$RPM