On Tue, Nov 3, 2009 at 7:26 AM, Cedric Staniewski <[email protected]> wrote:
> Eric Bélanger wrote:
>> On Tue, Nov 3, 2009 at 12:59 AM, Allan McRae <[email protected]> wrote:
>>> Dan McGee wrote:
>>>> On Mon, Nov 2, 2009 at 11:00 PM, Eric Bélanger <[email protected]>
>>>> wrote:
>>>>
>>>>> When DESTDIR is used, symlinks to the packages will be put in the build
>>>>> directory. This combines the convenience of a global package cache with
>>>>> the
>>>>> ease of having a package (i.e. a symlink) in the build directory for
>>>>> testing
>>>>> and installation purpose.
>>>>>
> I really like your idea of having a symlink to the package. Regarding the
> patch, I think we should stick with the "new" coding style and also merge the
> two if statements; could look like this:
>
> if (( ! ret )) && [[ $PKGDEST != $startdir ]]; then
> ln -sf "${pkg_file/$EXT/$PKGEXT}" "$pkglinks_target"
> ret=$?
> fi
>
I was just following the current style of the function I was working
in. Allan, Dan: is the above coding style OK with you?
