[prev in list] [next in list] [prev in thread] [next in thread] 

List:       util-linux-ng
Subject:    Re: [PATCH] look: use WORDLIST environment variable to find word list
From:       Karel Zak <kzak () redhat ! com>
Date:       2017-08-21 9:20:00
Message-ID: 20170821092000.q3vyxgosrcbqivwx () ws ! net ! home
[Download RAW message or body]

On Tue, Aug 15, 2017 at 07:13:34PM +0100, Sami Kerola wrote:
> On Mon, 14 Aug 2017, Karel Zak wrote:
> 
> > Would be better fallback to _PATH_WORDS if WORDLIST= points to
> > nowhere?
> > 
> >     file = _PATH_WORDS; /* default */
> >     if ((x = getenv("WORDLIST")) && access(file, R_OK) == 0)
> >         file = x;
> > 
> > seems like more backwardly compatible way.
> 
> No problems, change is updated much like proposed above.
> 
> https://github.com/kerolasa/lelux-utiliteetit/commit/dd8be07d84988b130286a1194210bdb26b223736
> 
> That is in branch: 2017wk32

Applied, thanks.

-- 
 Karel Zak  <kzak@redhat.com>
 http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic