[pacman-dev] [PATCH] Compress hard linked man pages
Allan McRae
mcrae_allan at hotmail.com
Tue Dec 4 19:48:01 EST 2007
Dan McGee wrote:
> On Dec 4, 2007 6:06 PM, Aaron Griffin <aaronmgriffin at gmail.com> wrote:
>
>> 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?
>>
>
> I'm making a few changes to the patch locally, and I'll send it here
> if it works right.
>
> -Dan
>
My testing used tcl which is really bad (~600 uncompressed man pages).
tk is just about as bad from memory...
More information about the pacman-dev
mailing list