Re: [PATCH (topgit) 1/2] Implement setup_pager just like in git

From: Kirill Smelkov <kirr@landau.phys.spbu.ru>
Date: Jan 8, 2009, 09:23 UTC
Message-ID: <20090108092304.GA4835@roro3>
In-Reply-To: <20090108020650.GC7345@lapse.rw.madduck.net>

On Thu, Jan 08, 2009 at 03:06:50PM +1300, martin f krafft wrote:

Show 12 quoted lines

> also sprach Kirill Smelkov <kirr@landau.phys.spbu.ru> [2009.01.08.1100 +1300]:
> > > So I suppose you could use
> > > 
> > >   ${GIT_PAGER-${PAGER-less}}
> > > 
> > > or similar.
> > 
> > Good eyes, thanks!
> > 
> > I'll rework it.
> 
> I am not 100% on this, but I think nested {}'s are a bashism.

It seems to be ok:

kirr@roro3:~$ dash $ unset GIT_PAGER $ unset PAGER $ echo ${GIT_PAGER-${PAGER-less}} less $ PAGER=more $ echo ${GIT_PAGER-${PAGER-less}} more $ GIT_PAGER='' $ echo ${GIT_PAGER-${PAGER-less}}

$ GIT_PAGER=/bin/cat $ echo ${GIT_PAGER-${PAGER-less}} /bin/cat

Show 10 quoted lines

> > On Wed, Jan 07, 2009 at 03:24:02PM +0100, Bert Wesarg wrote:
> > > On Wed, Jan 7, 2009 at 12:27, Kirill Smelkov <kirr@landau.phys.spbu.ru> wrote:
> > > > Martin, thanks for your review.
> > > > +       # atexit(close(1); wait pager)
> > > > +       trap "exec >&-; rm "$_pager_fifo"; rmdir "$_pager_fifo_dir"; wait" EXIT
> > > I think you need to escape the double quotes.
> > 
> > Good eyes -- corrected and thanks!
> 
> You could also just use single quotes inside the double quotes.

Thanks for the tip - I'll keep it in mind. Or is it the preferred way?

Thanks, Kirill