Default branch problems in Fedora 24

Bug #1598005 reported by tduell
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Hugin
Fix Released
Undecided
Unassigned

Bug Description

The default branch (hugin-2016.3.0.1e53df29f670) builds without error in fedora 24 x86_64 but there are display problems.
Image 'Selection_006.jpg' shows the window when it first opens.
Image 'Selection_007.jpg' shows the window after a slight drag from bottom RH corner.
Image 'Selection_008.jpg' shows that the menus exist, but the menu names are not displayed.
Image 'About Hugin_009.jpg' shows the build details.

Revision history for this message
tduell (tduell-iinet) wrote :
Revision history for this message
tduell (tduell-iinet) wrote :
Revision history for this message
tduell (tduell-iinet) wrote :
Revision history for this message
tduell (tduell-iinet) wrote :
Revision history for this message
tduell (tduell-iinet) wrote :

Sorry, I should have zipped all the screenshots into a single file.

Revision history for this message
tmodes (tmodes) wrote :

Hi Terry,

I can't reproduce this.

On Fedora 23 in a virtual machine all works fine.

The only difference I see is an older GTK version on my side (from the about screen):

wxWidgets: wxWidgets 3.0.2
wxWidgets Library (wxGTK port)
Version 3.0.2 (Unicode: wchar_t, debug level: 1),
compiled at May 6 2016 00:17:42

Runtime version of toolkit used is 3.18.
Compile-time GTK+ version is 3.18.9.

Revision history for this message
tduell (tduell-iinet) wrote :

Hello Thomas,
I also have no problem here with Fedora 23, the problem is with Fedora 24 and suspect the problem is the changes that it brings.

Cheers,
Terry

Revision history for this message
tmodes (tmodes) wrote :

Hi Terry,

then it would be interesting what changed in Fedora 24 and if this needs fixed upstreams in wxWidgets or if we need changes to Hugin source code.

Revision history for this message
tduell (tduell-iinet) wrote :

Hello Thomas,
It looks like runtime toolkit is now 3.20 and compile time is 3.20.2 but don't know if this brought any change that would cause what I'm seeing. Fedora 24 is also using gcc-6.1.1, could this have an influence?
It would be good if someone who has Fedora 24 x86_64 could test build and see if the issue is reproduced.
I use the Cinnamon desktop and pondered if there might be an issue there, so installed gnome, but the result is the same.

Cheers,
Terry

Revision history for this message
tduell (tduell-iinet) wrote :

There was a bug reported here <https://bugzilla.redhat.com/show_bug.cgi?id=1345924> that had an issue with wxGTK3, and noted that the problem was fixed by building against wxGTK, but we have passed the point where I could try that.
It may not be related.

Cheers,
Terry

Revision history for this message
tmodes (tmodes) wrote :

The problem with the missing menubar seems to be a combination of wxWidgets and GTK3. see ticker http://trac.wxwidgets.org/ticket/17539
Best would be to update the wxWidgets version with this fix.
A workaround could be to remove the wxTB_DOCKABLE style from the toolbar, as the ticket mentions that it only happens with this style.
Edit src/hugin1/hugin/xrc/main_tool.xrc, line 4. Change from
    <style>wxTB_FLAT|wxTB_DOCKABLE</style>
to
    <style>wxTB_FLAT</style>

But the problem with the sizers remains.

Revision history for this message
tduell (tduell-iinet) wrote :

Hello Thomas,
I've tied the workaround described in your post #11, and we get the icons back but still no menu names displayed.

Cheers,
Terry

Revision history for this message
tmodes (tmodes) wrote :

Hi Terry,

for the menu we need to remove the dockable style also from the menus.
I committed a fix to the default branch, which removes all dockable styles. This should fix the issue with the toolbars and menubars.
If you have tested it and it works then the following questions arise: should this patch also go into 2016.2 branch?

For the sizer issue: there is a bug report in the wxWidgets tracker http://trac.wxwidgets.org/ticket/17585 Maybe it gets fixed upstreams.

Revision history for this message
tduell (tduell-iinet) wrote :

Hello Thomas,
I have just tested a build of the default branch and all is OK except for the sizer issue.
I assume you have tested these changes in Fedora 23, and if OK then would guess it is OK for the patch to go into the 2016.2 branch.
Any builds of 2016.2 that use the later tools may still have the sizer problem (if it takes a while to be fixed upstream) but that is easily overcome by the user.

Cheers,
Terry

Revision history for this message
tmodes (tmodes) wrote :
Changed in hugin:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.