From: Lane S. <dow...@gm...> - 2007-07-24 15:46:40
|
I believe that all references to the deprecated MRJ api should be gone now. I have deleted the MRJ stub file, and I updated build.xml to use the new AppleJavaExtensions.jar. Make sure you also update your build environment (Eclipse or other) to refer to the new AppleJavaExtensions.jar file instead of MRJToolkitStub.zip. I also deleted games.strategy.triplea.ui.MacWrapper. TripleAFrame now refers to games.strategy.common.ui.MacWrapper. I also updated MacLobbyWrapper to use the new API. If you could test to verify that these changes didn't break anything on Windows and Linux, that would be great. Thanks, Lane On 7/23/07, Sean Bridges <sbr...@ya...> wrote: > Sorry, I should have looked for the new jar when it > wouldn't compile on windows. > > Can you remove the code that uses the mrj code and the > MRJToolkitStub.zip. All the code should be in the > MacWrapper classes. > > Also, you will need to update the full.path property > in build.xml. If you don't have access to a windows > machine email me when your done, and I'll make sure it > works. > > Sean > > --- Lane Schwartz <dow...@gm...> wrote: > > > Hmm... no, those really aren't good changes. > > > > The changes in revisions 1862,1861,1860 negate the > > Mac integration > > work that I had done. Mac users expect the About > > menu to be under the > > TripleA menu (which Mac OS X automatically creates). > > My changes made > > it so that on Mac OS X, we don't create an About > > menu under the Help > > menu, but instead we make use of the About menu item > > that is > > automatically created by Mac OS X. The above > > revisions return the > > About menu to the Help menu, and mean that if a user > > selects the > > OS-created About menu item under the TripleA menu, > > all they see is a > > blank dialog box that says "java". > > > > Additionally, the old code is based on the MRJ API. > > MRJ is deprecated. > > One goal in adding the new Mac code was to be able > > to completely get > > rid of all MRJ code, and MRJToolkitStub.zip. > > > > AppleJavaExtensions.jar is the new stub file that > > replaces > > MRJToolkitStub.zip. It should have all of the stubs > > required to > > compile the new code on non-Mac platforms. > > > > When you were trying to compile the new code, did > > you make sure that > > the new AppleJavaExtensions.jar file was in your > > classpath? > > > > Lane > > > > > > On 7/21/07, Sean Bridges <sbr...@ya...> > > wrote: > > > Can you look at the changes I made in revisions > > > 1862,1861,1860. I needed to remove that code to > > get > > > it to compile on windows, since the windows jdk > > does > > > not contain the apple classes. > > > > > > Some of the mac code we have in stub form in > > > MRJToolkitStub.zip. Having it in stub form allows > > us > > > to compile on windows. > > > > > > Thanks, > > > > > > Sean > > > > > ------------------------------------------------------------------------- > > This SF.net email is sponsored by: Splunk Inc. > > Still grepping through log files to find problems? > > Stop. > > Now Search log events and configuration files using > > AJAX and a browser. > > Download your FREE copy of Splunk now >> > > http://get.splunk.com/ > > _______________________________________________ > > Triplea-developers mailing list > > Tri...@li... > > > https://lists.sourceforge.net/lists/listinfo/triplea-developers > > > > > > ____________________________________________________________________________________ > Luggage? GPS? Comic books? > Check out fitting gifts for grads at Yahoo! Search > http://search.yahoo.com/search?fr=oni_on_mail&p=graduation+gifts&cs=bz > -- When a place gets crowded enough to require ID's, social collapse is not far away. It is time to go elsewhere. The best thing about space travel is that it made it possible to go elsewhere. -- R.A. Heinlein, "Time Enough For Love" |