bugGNU Octave - Bugs: bug #39477, mxe-octave: qt fails compiling...

 
 

bug #39477: mxe-octave: qt fails compiling dbus support in native mingw build

Submitter:  None
Submitted:  Sat 13 Jul 2013 11:32:47 AM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  None
Originator Name:  John Donoghue Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * Microsoft Windows Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 18 Jul 2013 08:23:26 PM UTC, comment #11: 
Michael Goffioul <goffioul>
Thu 18 Jul 2013 06:51:31 PM UTC, comment #10: 

I've the patch ready, it's pending submission. I just want to complete at least one successful octave build before submitting. But I've been plagued by other unrelated minor problems since then. I will submit it soon.

Michael Goffioul <goffioul>
Thu 18 Jul 2013 06:47:41 PM UTC, comment #9: 

Any change for msvc for this issue?

John Donoghue <lostbard>
Group Member
Mon 15 Jul 2013 12:39:23 AM UTC, comment #8: 

That worked for me

Ive reattached the dbus patch, patch file renamed to src/qt-2-dbus.patch

Do you want to push the change set when you also update for msvc?

(file #28581)

John Donoghue <lostbard>
Group Member
Sun 14 Jul 2013 01:02:35 PM UTC, comment #7: 

Looks good, but I'd rather have a solution for all systems. So my suggestion would be to move your Qt patch to qt-2-dbus.patch, so it's applied for all systems, then use this additional MXE patch. I've tested it with cross-compile and it seems to work fine. Could you try native MinGW?

I'll adapt my MSVC patch afterwards.

(file #28566)

Michael Goffioul <goffioul>
Sun 14 Jul 2013 07:30:54 AM UTC, comment #6: 

how about this then - new patch

(file #28565)

John Donoghue <lostbard>
Group Member
Sat 13 Jul 2013 04:00:52 PM UTC, comment #5: 

I solved it a different way to MSVC. See:
http://hg.octave.org/mxe-octave/file/d24828818d36/src/msvc-qt-1.patch#l186

Maybe something similar could be done for MinGW as well.

Michael Goffioul <goffioul>
Sat 13 Jul 2013 03:53:04 PM UTC, comment #4: 

A new patch included that adds dbus-.0 to the include paths instead of removing dbus - trying it now... it will take an hour or so to compile on my machine :)

(file #28561)

John Donoghue <lostbard>
Group Member
Sat 13 Jul 2013 03:31:26 PM UTC, comment #3: 

how about remove as a requirement? :)

I was going to look at the path of pulling the dbus include files - but got called in to work so will have to take a look at doing that later.

John Donoghue <lostbard>
Group Member
Sat 13 Jul 2013 02:42:28 PM UTC, comment #2: 

To be honest, I don't think it's the right move. D-Bus is marked as a requirement for qt and it is compiled in with cross-compiling, so you would end up with different features when native compiling and cross compiling.

Michael Goffioul <goffioul>
Sat 13 Jul 2013 12:55:11 PM UTC, comment #1: 

Patch included that removes --dbus-linked from the native mingw build.

(file #28559)

John Donoghue <lostbard>
Group Member
Sat 13 Jul 2013 11:32:47 AM UTC, original submission:  

With the change to add dbus in native mingw build (--dbus-linked), mingw no longer compiles, failing at trying to find dbus-1.0/dbus/dbus.h include files.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #28581:  add_qt-2-dbus.patch added by lostbard (959B - application/octet-stream)
file #28566:  qt-dbus.patch added by goffioul (2KiB - text/x-patch)
file #28565:  patched_qt2.patch added by lostbard (962B - application/octet-stream)
file #28561:  cpp.diff added by lostbard (665B - application/octet-stream)
file #28559:  qt_dbus.patch added by lostbard (2KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lostbard (Updated the item)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-07-18 goffioul StatusNone Fixed
        Open/ClosedOpen Closed
    2013-07-15 lostbard Attached File- Added add_qt-2-dbus.patch, #28581
    2013-07-14 goffioul Attached File- Added qt-dbus.patch, #28566
    2013-07-14 lostbard Attached File- Added patched_qt2.patch, #28565
    2013-07-13 lostbard Attached File- Added cpp.diff, #28561
    2013-07-13 lostbard Attached File- Added qt_dbus.patch, #28559

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code