Bug 68346 – [GTK] Fix distcheck build

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Carlos Garcia Campos]

make distcheck doesn't currently work.

[Carlos Garcia Campos]

Created attachment 107826 [details]
Patch

[Philippe Normand]

Comment on attachment 107826 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=107826&action=review

Looks good overall, just a doubt of the mime-type handling test.

> Source/WebKit/gtk/tests/testmimehandling.c:-79
> -    } else if (g_str_equal(path, "/ogg")) {

Hum, why remove this part of the test? If it fails we'd better fix it.

[Carlos Garcia Campos]

(In reply to comment #2)
> (From update of attachment 107826 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=107826&action=review
> 
> Looks good overall, just a doubt of the mime-type handling test.
> 
> > Source/WebKit/gtk/tests/testmimehandling.c:-79
> > -    } else if (g_str_equal(path, "/ogg")) {
> 
> Hum, why remove this part of the test? If it fails we'd better fix it.

Soup behaviour has changed, the "fix" would depend on soup version, so we should bump the soup requirements to make sure the test won't fail. See:

https://bugzilla.gnome.org/show_bug.cgi?id=648846#c8

So, for now we have decided to remove that part of the test.

[Philippe Normand]

Oh, OK then :) I've missed that link you also mentionned in the ChangeLog, sorry.

[WebKit Review Bot]

Comment on attachment 107826 [details]
Patch

Clearing flags on attachment: 107826

Committed r95417: <http://trac.webkit.org/changeset/95417>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Martin Robinson]

In the future, I think it would be better to update the source lists and fix the tests in two separate patches. That would allow an automatic rollout when the soup version is updated. Source list updates do not need review either, so it isn't much harder.

[Carlos Garcia Campos]

(In reply to comment #7)
> In the future, I think it would be better to update the source lists and fix the tests in two separate patches. That would allow an automatic rollout when the soup version is updated. Source list updates do not need review either, so it isn't much harder.

Ok, sorry, in this case just updating libsoup wound't work, we need to rethink the test itself, but still, you are right.