Daniele Maccari wrote:
> Lucas C. Villa Real wrote:
>
>> On Thu, May 8, 2008 at 2:32 PM, Daniele Maccari <[EMAIL PROTECTED]> wrote:
>>
>>
>>> Hello everybody,
>>>
>>>
>> Hey!
>>
>>
>>
>>> I was trying to create a recipe for OpenOffice 2.4.0, so I simply
>>> newversioned the preexistent recipe. However, after having manually
>>> fixed the recipe a little I found a problem in the installation phase,
>>> namely it seems that Compile doesn't correctly recognize some
>>> directories as such. The problem seems to be related to these lines
>>> (starting at line 1445 of Compile)
>>>
>>> if [ -d "$src" -a -d "$fulldst" ]
>>> then $sudo_exec cp -R "$src"/* "$fulldst"
>>> else $sudo_exec cp -R $src "$fulldst"
>>> fi
>>>
>>> In fact, when trying to copy files from /usr/share and
>>> /opt/OpenOffice2.4/share to Shared the if check seems to not work as
>>> expected, thus treating share as a simple file to copy under Shared,
>>> leading to /P/OpenOffice/2.4.0_bin/Shared/share, which prevent the
>>> pre_link function to correctly perform its task of symlinking desktop
>>> entries.
>>>
>>> I don't know whether this is a real Compile issue or it depends on the
>>> OpenOffice package, but instead of modifying the recipe to copy every
>>> single directory under share, I'd prefer some advices to come from you
>>> other.
>>>
>>>
>> Could you please try to run Compile again, this time with the echoes
>> shown in the patch below? The recipe for 2.3.1_bin looks fine, so I
>> can't understand what's going wrong down in the path. I was about to
>> test this change against 2.3.1_bin, but the download is slow here atm.
>>
>>
>> --- bin/Compile (revision 3254)
>> +++ bin/Compile (working copy)
>> @@ -1438,6 +1438,8 @@
>> dst=`echo $i | cut -d: -f2-`
>> fulldst="$installprefix/$dst"
>> $sudo_exec mkdir -p $(dirname "$fulldst")
>> + [ -d "$src" ] || echo "--> src $src is not a directory"
>> + [ -d "$fulldest" ] || echo "--> fulldest $fulldest is not a
>> directory"
>> if [ -d "$src" -a -d "$fulldst" ]
>> then $sudo_exec cp -R "$src"/* "$fulldst"
>> else $sudo_exec cp -R $src "$fulldst"
>>
>>
>>
> Neither of them seems to be a directory, but checking for them with sudo
> seems to work. So I added these lines:
>
> {
> $sudo_exec [ -d "$src" -a -d "$fulldst" ] && $sudo_exec cp -R
> "$src"/* "$
> } || $sudo_exec cp -R $src "$fulldst"
>
> in place of the if. Though now I get other errors:
>
> cp: cannot stat `opt/openoffice.org2.4/share/*': No such file or directory
> cp: cannot stat `opt/openoffice.org2.4/licenses/*': No such file or
> directory
> cp: cannot stat `opt/openoffice.org2.4/readmes/*': No such file or directory
> cp: cannot stat `usr/share/*': No such file or directory
>
> which seems a bit strange to me since cp already has $sudo_exec before
> it. Maybe I'm doing something wrong?
>
>
Of course the code should be
{
$sudo_exec [ -d "$src" -a -d "$fulldst" ] && $sudo_exec cp -R "$src"/*
"$fulldst"
} || $sudo_exec cp -R $src "$fulldst"
_______________________________________________
gobolinux-devel mailing list
[email protected]
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel
