Re: [Tracker] ANNOUNCE: tracker 0.8.17 released (stable)

[Martyn Russell <martyn lanedo com>]

On 15/09/10 10:52, Mildred Ki'Lya wrote:
> Hi,
>
> I'm having compilation problems:
>
>
> make[4]: Entering directory `/.../tracker-0.8.17/src/tracker-extract'
>    CC     tracker-extract-mplayer.lo
> tracker-extract-mplayer.c: In function âcopy_hash_table_entryâ:
> tracker-extract-mplayer.c:93:26: warning: initialization makes pointer
> from integer without a cast
> tracker-extract-mplayer.c: In function âextract_mplayerâ:
> tracker-extract-mplayer.c:132:2: error: too many arguments to function
> âtracker_spawnâ
> ../../src/libtracker-common/tracker-os-dependant.h:34:10: note: declared
> here
> make[4]: *** [tracker-extract-mplayer.lo] Error 1
>    CC     tracker-extract-totem.lo
> tracker-extract-totem.c: In function âextract_totemâ:
> tracker-extract-totem.c:88:2: error: too many arguments to function
> âtracker_spawnâ
> ../../src/libtracker-common/tracker-os-dependant.h:34:10: note: declared
> here
> make[4]: *** [tracker-extract-totem.lo] Error 1
> make[4]: Target `all-am' not remade because of errors.
> make[4]: Leaving directory `/.../tracker-0.8.17/src/tracker-extract'
>
>
> The warning isn't so important but tracker_spawn doesn't seem to have
> the correct signature.
>
> The declaration looks like:
>
> gboolean tracker_spawn           (gchar       **argv,
>                                    gint          timeout,
>                                    gchar       **tmp_stdout,
>                                    gint         *exit_status);
>
> But the invocation for mplayer looks like:
>
> tracker_spawn (argv, 10,&mplayer, NULL, NULL)
>
> And for totem:
>
> tracker_spawn (argv, 10,&totem, NULL, NULL)
>
> Am I missing something or is there an extra argument?

Hmm, looks like I cherry picked the fix for master which I think has a 
slightly different function signature. I will fix this before the next 
release. Thanks.

To keep track of this, could you report this as a bug for me please? :)

--
Regards,
Martyn