Possible compiler bug in 2.7 ignoring a <set-property>

[Jan Thewes]

Hey guys,

I've found what I think is a bug in GWT2.7.

We're trying to compile the attached file 'MyMegaModule.gwt.xml'

The last inherit in this file is BaseBrowsers.gwt.xml - I've attached that too.

In the BaseBrowsers.gwt.xml you can see that we only want gecko and safari compilations but when compiling we get an ie10 permutation too.

There might be other inherits that are setting the user.agent property to something including the ie10 but because of our BaseBrowsers.gwt.xml being inherited as the last module we should only get gecko and safari, am I right?

I've no idea how to fix this.

Thx for any ideas!

Cheers,

Jan

[Colin Alworth]

I don't recall if inherited modules are loaded more than once - in other words, if one of the higher up modules (like BaseGWT) already inherited BaseBrowsers, then a later module added ie10 via extend-property, it might not be possible to re-inherit BaseBrowsers and have it re-set user.agent down to just those two.

I don't know if this is actually the case or not, but it might make sense, since otherwise you could run in circles inheriting modules that inherit each other. Instead, I always encourage developers to list their inherits first, then their own specific overrides. Don't think of a module file like a function call, since that could lead to recursion and other fun issues, think of them more as dependencies.

--
You received this message because you are subscribed to the Google Groups "GWT Contributors" group.
To unsubscribe from this group and stop receiving emails from it, send an email to google-web-toolkit-co...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/google-web-toolkit-contributors/ac5c7e23-fd85-46d6-883b-8f8600c496cb%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Jan Thewes]

Well I've to check if this is the case. But I think this might really be the problem. I'll check this tomorrow and see if I can fix it. Thanks for that hint.

Cheers,
Jan