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?
_______________________________________________
gobolinux-devel mailing list
[email protected]
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel
