Last Comment Bug 770754 - Allow build Qt mobility without maemo define
: Allow build Qt mobility without maemo define
Status: RESOLVED FIXED
:
Product: Core Graveyard
Classification: Graveyard
Component: Widget: Qt (show other bugs)
: Trunk
: ARM Linux
: -- normal (vote)
: mozilla16
Assigned To: Oleg Romashin (:romaxa)
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-03 17:25 PDT by Oleg Romashin (:romaxa)
Modified: 2016-07-11 21:54 PDT (History)
2 users (show)
ryanvm: in‑testsuite-
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Fix build qt mobility without maemo define (1.12 KB, patch)
2012-07-03 17:25 PDT, Oleg Romashin (:romaxa)
no flags Details | Diff | Splinter Review
Fix build qt mobility without maemo define, + maemo5 provider fix (1.44 KB, patch)
2012-07-03 19:10 PDT, Oleg Romashin (:romaxa)
doug.turner: review+
Details | Diff | Splinter Review

Description Oleg Romashin (:romaxa) 2012-07-03 17:25:05 PDT
Created attachment 638922 [details] [diff] [review]
Fix build qt mobility without maemo define

Currently Qt build without maemo defined and with QtMobility libraries available is broken due to wrong define in dom/system/unix
Comment 1 Oleg Romashin (:romaxa) 2012-07-03 19:10:20 PDT
Created attachment 638941 [details] [diff] [review]
Fix build qt mobility without maemo define, + maemo5 provider fix

This actually works better for maemo5
Comment 2 Ryan VanderMeulen [:RyanVM] 2012-07-09 17:55:33 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6dc10b2f617
Comment 3 Ryan VanderMeulen [:RyanVM] 2012-07-10 15:48:38 PDT
https://hg.mozilla.org/mozilla-central/rev/e6dc10b2f617

Note You need to log in before you can comment on or make changes to this bug.