Can't compile Scratch with vala 0.16

According to http://www.valadoc.org/#!api=libpeas-1.0/Peas.ExtensionSetForeachFunc the on_extension_added function should have three parameters, so one more then currently defined.

This only turns up with vala-0.16 since that checks more carefully than previous versions... so this is really a bug and not just compiler incompatibility.

  status triaged
  importance high

If I recall correctly the broke the vapi between vala-0.14 and
vala-0.16. The easier way would be to use the -0.16 and to make a cast
in order to support the -0.14 version, I think it should work (well, we
have to check that it doesn't crash, I'm not 100% sure we don't need
these parameters). Or there is also some # if VALA_0_16 things.

Le 28/03/2012 21:06, Pim Vullers a écrit :
> This only turns up with vala-0.16 since that checks more carefully than
> previous versions... so this is really a bug and not just compiler
> incompatibility.
>

I managed to port scratch to 0.16 but I am not sure if it works since I have some granite problems...please test it

Same here, when I try to compile scratch I get the following:

[ 2%] Scanning dependencies of target i18n
Generating main.c, SearchManager.c, SourceView.c, Settings.c, PluginManager.c, TemplateManager.c, scratchcore.vapi, scratchcore.h, scratchcore_internal.h
[ 5%] messages.”
[ 5%] Built target i18n
granite.vapi:229.4-229.31: error: overriding method `Granite.Widgets.TimePicker.input' is incompatible with base method `Gtk.SpinButton.input': incompatible type of parameter 1.
   protected override int input (out double new_value);
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
granite.vapi:229.4-229.31: error: Granite.Widgets.TimePicker.input: no suitable method found to override
   protected override int input (out double new_value);
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:76.26-76.110: warning: unhandled error `GLib.Error'
        var enumerator = origin.enumerate_children ("standard::type,standard::name,standard::content-type", 0);
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:78.25-78.47: warning: unhandled error `GLib.Error'
        var file_info = enumerator.next_file ();
                        ^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:86.25-86.47: warning: unhandled error `GLib.Error'
            file_info = enumerator.next_file ();
                        ^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:44.9-44.37: warning: unhandled error `GLib.Error'
        destination.make_directory ();
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:52.17-52.112: warning: unhandled error `GLib.FileError'
                FileUtils.get_contents (Path.build_filename (origin.get_path (), file.get_name ()), out content);
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:58.17-58.113: warning: unhandled error `GLib.FileError'
                FileUtils.set_contents (Path.build_filename (destination.get_path (), file.get_name ()), content);
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/TemplateManager.vala:63.17-63.35: warning: unhandled error `GLib.Error'
                orig.copy (dest, 0);
                ^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/SearchManager.vala:448.9-448.72: warning: unhandled error `GLib.Error'
        css.load_from_data("* { padding-left:0; padding-right:0; }", -1);
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
/var/tmp/portage/app-editors/scratch-9999/work/scratch-9999/scratchcore/SearchManager.vala:458.5-458.25: w...

I fixed the bug but I suggest to compile with valac-0.14 because with 0.16 scratch can't load the plugin at launch...I am already investigating on this

Now everything works with 0.16 which is the required valac version

Thank you Mario!