Bug 236235 - Problems importing microemu 2.0.2
Summary: Problems importing microemu 2.0.2
Status: CLOSED FIXED
Alias: None
Product: MTJ (Archived)
Classification: Tools
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All Windows XP
: P3 normal (vote)
Target Milestone: 0.9   Edit
Assignee: Diego Madruga Sandin CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, core
Depends on:
Blocks: 241964
  Show dependency tree
 
Reported: 2008-06-09 09:28 EDT by Diego Madruga Sandin CLA
Modified: 2008-10-15 09:22 EDT (History)
2 users (show)

See Also:


Attachments
Exception Log (7.93 KB, text/plain)
2008-06-09 09:28 EDT, Diego Madruga Sandin CLA
no flags Details
Device storing Error (9.88 KB, image/png)
2008-06-09 09:28 EDT, Diego Madruga Sandin CLA
no flags Details
Sets up the classpath correctly (9.94 KB, patch)
2008-07-07 11:24 EDT, Dan Murphy CLA
wds057: iplog+
Details | Diff
Properties patch (971 bytes, patch)
2008-07-08 07:05 EDT, Dan Murphy CLA
no flags Details | Diff
Properties patch, to be used inconjunction with importer patch (971 bytes, patch)
2008-07-08 07:07 EDT, Dan Murphy CLA
wds057: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Madruga Sandin CLA 2008-06-09 09:28:05 EDT
Created attachment 104145 [details]
Exception Log

Multiple failures when importing the 2.0.2 ?Microemu release into MTJ.

Files "/emptyapi/cldcapi11.jar" and "/emptyapi/midpapi20.jar" can't be found leading to a NullPointerException when storing the devices that avoids the microemu to be imported correctly.
Comment 1 Diego Madruga Sandin CLA 2008-06-09 09:28:32 EDT
Created attachment 104146 [details]
Device storing Error
Comment 2 Dan Murphy CLA 2008-07-07 11:24:47 EDT
Created attachment 106727 [details]
Sets up the classpath correctly

Just a proposal...
Comment 3 Gustavo de Paula CLA 2008-07-07 17:17:07 EDT
hi dan,

thanks for the patch. it solves the import and the project creation, but we can't use it to compile the classes. the main problem are the cldc classes that are still missing. 

it seems that we also need to change the microemu.property file to include the cldc jars on classpath property. i will do the test later to see if it solves the problem.

:)
gep
Comment 4 Dan Murphy CLA 2008-07-08 07:05:56 EDT
Created attachment 106806 [details]
Properties patch

Sorry, forgot to add the modified .properties file.... here it is
Comment 5 Dan Murphy CLA 2008-07-08 07:07:55 EDT
Created attachment 106807 [details]
Properties patch, to be used inconjunction with importer patch

Sorry, forgot to add the modified .properties file.... here it is
Comment 6 Gustavo de Paula CLA 2008-07-10 12:44:05 EDT
hi dan,

it solve the import, but there are still problems with the preverification and the execution of the emulator. 

the main problem seems to be that the stubs that are provided with microemu are not preverified, so when we try to run the preverifier on the midlet classes it just fails (i tested it with a standard sun preverifier).

on the end of this month we wll have access to some stubs that we will be able to use directly on mtj. on that time we will come back to this bug to see if using those stubs fix the whole problem.

:)
gep 

(In reply to comment #5)
> Created an attachment (id=106807) [details]
> Properties patch, to be used inconjunction with importer patch
> 
> Sorry, forgot to add the modified .properties file.... here it is
> 

Comment 7 Dan Murphy CLA 2008-07-12 08:12:41 EDT
Hi Gustavo,
I applied the patch for progard preverification (https://bugs.eclipse.org/bugs/show_bug.cgi?id=234139) and this, for me at least, allowed me to run midlets on mpowerplay, microemu and my N95.

I have a slightly modified version of the patch that set's up the api names correctly for the jsr jars... would this be of any use to you.

All the best,
Dan
Comment 8 Gustavo de Paula CLA 2008-07-23 17:24:24 EDT
hi dan,

we are still evaluating if it make sense to add the proguard preverifier to mtj (at least it is already implemented :)). do you think it add value to mtj developers?

i'm not sure if you saw diego's email... diego is another committer on mtj project. we talked with microemu guys and they are going to do a 2.0.3 release that has a fix on microemu stubs. since that we want to commit your contribution to mtj and after that do some fixes on top of it (the information that diego requested is necessary by eclipse legal). once you send him that information he will commit the code. if you need some information, please let us know

thanks
:)
gustavo


(In reply to comment #7)
> Hi Gustavo,
> I applied the patch for progard preverification
> (https://bugs.eclipse.org/bugs/show_bug.cgi?id=234139) and this, for me at
> least, allowed me to run midlets on mpowerplay, microemu and my N95.
> 
> I have a slightly modified version of the patch that set's up the api names
> correctly for the jsr jars... would this be of any use to you.
> 
> All the best,
> Dan
> 

Comment 9 Dan Murphy CLA 2008-07-23 17:44:42 EDT
Hi,
As proguard can both obfuscate and the necessary preverification, I think it adds great value to MJT (and potential optimisation and shrinking capabillities)... Especially since I use OS X, so the only viable options seem to be proguard or Mpowerplayer SDK for preverification... so I'm more keen to use MicroEmu (better JSR coverage) with proguard since it seems to work :)
Just 2 cents worth :)
Comment 10 Diego Madruga Sandin CLA 2008-07-24 08:45:13 EDT
Hi Dan,

your patch was integrated and will be available in the N20080725 Nightly Build.

Thanks
Diego
Comment 11 Gustavo de Paula CLA 2008-10-15 09:22:58 EDT
all bugs we integrated and release on MTj 0.9