Comment 6 for bug 534311

Revision history for this message
Martin Pitt (pitti) wrote :

So for the previous failure, the GUPnPSimpleIgdThreadClass wrapper's _init() function apparently tries to set the context for the wrapped GUPnPSimpleIgdThread:

  g_object_set (self, "main-context", self->priv->context, NULL);

However, GUPnPSimpleIgdThread's property is declared as G_PARAM_CONSTRUCT_ONLY, thus you can't set it from outside.

I replaced G_PARAM_CONSTRUCT_ONLY with G_PARAM_CONSTRUCT, which fixes above problem, but is that actually the correct solution?

Also, this now fails later on in a different test:

/simpleigd/new: OK
/simpleigd/default_ctx: OK
/simpleigd/custom_ctx: OK
/simpleigd/thread: OK
/simpleigd/random/no_conflict: OK
/simpleigd/random/conflict: OK
/simpleigd/dispose_removes/regular: OK
/simpleigd/dispose_removes/thread: OK
/simpleigd/invalid_ip: OK
/simpleigd/empty_ip:
GLib-GObject-WARNING **: invalid uninstantiatable type `(null)' in cast to `GUPnPContextManager'
aborting...
/bin/bash: Zeile 5: 10679 Trace/breakpoint trap (Speicherabzug geschrieben) XML_PATH=. G_SLICE=debug-blocks LD_LIBRARY_PATH=../../libgupnp-igd/.libs ${dir}$tst
FAIL: gupnp-simple-igd

but this seems unrelated.