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

 
 

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

Submitted by:  None
Submitted on:  Sat 13 Jul 2013 11:32:47 AM UTC  
 
Category: Configuration and Build SystemSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Build Failure
Status: FixedAssigned to: None
Originator Name: John DonoghueOriginator Email: -unavailable-
Open/Closed: ClosedRelease: dev
Operating System: Microsoft Windows

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Thu 18 Jul 2013 08:23:26 PM UTC, comment #11:

http://hg.octave.org/mxe-octave/rev/fae248a5b181

Michael Goffioul <goffioul>
Project Member
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>
Project Member
Thu 18 Jul 2013 06:47:41 PM UTC, comment #9:

Any change for msvc for this issue?

John Donoghue <lostbard>
Project 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>
Project 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>
Project Member
Sun 14 Jul 2013 07:30:54 AM UTC, comment #6:

how about this then - new patch

(file #28565)

John Donoghue <lostbard>
Project 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>
Project Member
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>
Project 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>
Project 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>
Project Member
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>
Project 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 File(s):
   
   
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
  • -unavailable- added by goffioul (Posted a comment)
  • -unavailable- added by lostbard (Updated the item)
  • -unavailable- added by None (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 18 Jul 2013 08:23:26 PM UTCgoffioulStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Mon 15 Jul 2013 12:39:23 AM UTClostbardAttached File-=>Added add_qt-2-dbus.patch, #28581
    Sun 14 Jul 2013 01:02:35 PM UTCgoffioulAttached File-=>Added qt-dbus.patch, #28566
    Sun 14 Jul 2013 07:30:54 AM UTClostbardAttached File-=>Added patched_qt2.patch, #28565
    Sat 13 Jul 2013 03:53:04 PM UTClostbardAttached File-=>Added cpp.diff, #28561
    Sat 13 Jul 2013 12:55:11 PM UTClostbardAttached File-=>Added qt_dbus.patch, #28559

    Back to the top


    Powered by Savane 3.1-cleanup1