[pacman-dev] [PATCH] Compress hard linked man pages

Aaron Griffin aaronmgriffin at gmail.com
Tue Dec 4 19:06:17 EST 2007


On Dec 4, 2007 5:54 PM, Allan McRae <mcrae_allan at hotmail.com> wrote:
> Aaron Griffin wrote:
> > Besides what Roman mentioned, I noticed you used the backtick inline
> > execution syntax - we like to frown on that around here. Could you
> > please use $() instead?
> >
> Fixed.  I didn't know about that syntax - how is it better?

Well, first off, backtick expansions is officially deprecated if I
remember correctly, but at the rate these standards move it will be
some time before it dies. In addition to hat, $() is much cleaner when
it comes to parsing and nesting. It's easier to match a pair of
characters than two of the same characters. Take this for example:

   `foo -c `bar -x``
   $(foo -c $(bar -x))

The first one might not even parse right in the first place.

> New patch attached.

Danke! It looks good to me. We don't have a testsuite for makepkg like
we do for pacman, so do you happen to know any packages, offhand, that
suffer from the hardlink issue, so that I can test this?




More information about the pacman-dev mailing list