Merging the android plugin to Qt Creator
BogDan
bog_dan_ro at yahoo.com
Tue Jul 26 15:09:56 CEST 2011
[..]
>> >0005-Simplfy-template-code.patch
>>
>> Not pushed, without setBaseUrl thing, qml engine will add file:// in front
>> of the file and it will not work with current android assets
>> implementation which need a file patch to start with a "/" or
> directly
>> with its name.
> Ah, that's reason. then I'd suggest moving the setBaseUrl to the
> private, that
> is applying the patch + readding the setBaseUrl in the private class.
>
Sorry, you lost me :) ... a little more help will be very appreciated :)
>
>> >j) bin/necessitas and bin/necessitas.bat
>> >The setting of ANDROID_NDK_HOST seemed to be a left over from before
>> >setting it in androidtoolchain.cpp, so I removed it, see attached
> patch.
>> >We used to have a similar LD_LIBRARY_PATH setting script on linux but
>> >after some internal discussions we removed it. On windows
>>
>> Can you share with use your secret ?
> Actually I don't understand the topic anymore and the expert on that topic
> is
> on vacation.
>
I'm just curios .... :), anyway I removed them from MR branch, I'll update the MR soon !
>> >p) debugger plugin
>> >The debbuger plugin has quite a few changes, some of them seem to be
>> >hacks. Also I'm not that well versed in the debugger plugins code,
> though
>> >I discussed a earlier diff with the debugger maintainer (Andre).
>> >I'm somewhat confussed whether you are using the python dumpers, or
> not.
>>
>> We are using python dumpers !
> Okay, good so the changes to dumper.cpp are obsolete? See the attached patch.
>
I think so .. I'll apply your patch.
>
>> >So, the changes:
>> >Adding "arm-linux-androideabi" to gnuTargets: fine
>> >
>> >soLibSearchPath: in general looks fine, andre mentioned some details he
>> >wanted to see fixed
>>
>> Which one ?
> We'll have to wait for him to be back.
>
Ok.
>
>> >u) Settings SystemScope to UserScope
>> >Quite a few settings (if not all) are using the UserScope instead of
> the
>> >SystemScope. Why is that needed?
>>
>> Because SystemScope is not user writable (if you'll install the SDK
> using
>> "sudo" that file can't be written by a normal user), so we
> choose to
>> move everything to user's home, if is not correct we'll revert that
>> change.
> I've discussed that with a colleague and I think I understand it now.
>
> We used to have a wrong setttings setup, which you commented out in your code.
> We've fixed that, though that change wasn't merged, and now the
> SystemScope is
> in a directory writeable by a normal user. Also the Core::settings() method
> already defaults to returning UserScope, so passing in UserScope is not
> necessary.
> Can you test the attached patch and commit it, if it works?
>
Ok, I'll provide feedback soon !
> daniel
>
Thanks.
More information about the Necessitas-devel
mailing list