Re: More patches

["G. Milde" <milde@xxxxxxxxxxxxxxxxxxxxx>]

On 25.05.08, J�rg Sommer wrote:
> "G. Milde" <milde@xxxxxxxxxxxxxxxxxxxxx> wrote:
> > On 20.05.08, J�rg Sommer wrote:

> > Then this issue should be solved. (There might be others.)

> No, it doesn't. I've forgot I'm using this patch for 50jed-extra.sl

> diff --git a/jed.d/50jed-extra.sl b/jed.d/50jed-extra.sl
> index b4e547b..e00a95d 100644
> --- a/jed.d/50jed-extra.sl
> +++ b/jed.d/50jed-extra.sl
> @@ -13,6 +13,8 @@ $2 = path_concat($1, "libdir.sl");
>  #stop
>  #endif

> +set_slang_load_path($1 + "drop-in:" + get_slang_load_path() + ":$1/utils:$1"$);
> +
>  % Add (and initialize) library directories
>  % ----------------------------------------

> Without it the jed-extra directories aren't in the load path.

The inserted line should not be necessary, if the 
  % Add (and initialize) library directories
code works.

This means that either

* the file /usr/share/jed/jed-extra/libdir.slc is missing, or

* the functions add_libdir() and append_libdir() from libdir.sl do *not*
  work as intended.
  
Could it be that your installed version of Jed still misses this:

> > * a patch in site.sl will alias (s|g)et_jed_library_path() to
> >   (s|g)et_slang_load_path().

?



> >> > Could you run the jedmodes unit tests with 0.99 and tell me about any
> >> > errors (or just mail a testreport)? There is a utility script in
> >> > jed-extra/trunk/utils/do-unittests.sh.

> All tests fail with the error message ‘view_mode is undefined.’ 

This is a missing autoload in unittest.sl, fixed in version 0.5.3

> I've added this patch and now the test exits with the return value 20.

Which means that there are still 20 errors in the test. Could you mail me
the generated testreport.txt?

Thanks,

Günter

--------------------------
To unsubscribe send email to <jed-users-request@xxxxxxxxxxx> with
the word "unsubscribe" in the message body.
Need help? Email <jed-users-owner@xxxxxxxxxxx>.