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"
>
>   
Sure, I'll try that asap. Indeed I didn't think about checking them 
separately during compilation, my fault :)

Bye
_______________________________________________
gobolinux-devel mailing list
gobolinux-devel@lists.gobolinux.org
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel

Reply via email to