Re: [Pingus-Devel] Pingus 0.7.6 - Xmas 2011

[Jonas Bähr]

Am 29.12.2011 um 03:45 schrieb Josh Dye:
> Thx, I'll admit that I don't always write clean code.  I applied all  
> your patches
>
> 0001-Fixed-the-language-selection-of-Pingus.app.patch
> 0002-Added-a-link-to-the-MacOS_X_Compiling-wiki-page.patch
> 0003-Implemented-language-detection-for-Mac-OS-X.patch
> 0004-Removed-the-wrapper-script-from-the-app-bundle.patch
> data-dir-check.patch
>
> They all work, so I committed them to GIT

Thanks. One little wish for the future though: please use "git am" to  
apply such patches (like written in the initial mail about the  
language detection patch), since this keeps commit messages,  
authorship and the logical grouping of the changes. Now the history  
looks like this:
------8<------8<-----
commit 093d3eb273a01ed27f45d96f9c252be1046b8849
Author: Josh Dye <address@hidden>
Date:   Wed Dec 28 18:42:46 2011 -0800

    Alot of code cleanups relating to OSX by Jonas
------8<------8<-----

While it could have been:
------8<------8<-----
commit 8f75c9ad1ce56bdf0201788088cad5287537a2f3
Author: jonas <address@hidden>
Date:   Thu Dec 29 02:35:26 2011 +0100

    Removed the wrapper script from the app bundle

    Since the pingus executable is now by itself capable of finding the
    data in the bundle as well as to detect the preferred language  
there
    is no need for the wrapper anymore.

commit 311149eea07cc303e4dc5207b5da69d60416549c
Author: jonas <address@hidden>
Date:   Thu Dec 29 02:02:48 2011 +0100

    Implemented language detection for Mac OS X

    If an application is started via Finder the LC_MESSAGES is not set
    (at least not here in Leopard) so we fetch the first entry of the
    preferred languages from the global preferences.

commit fde6540558f110fdd74400fbf67d030c2822fd9a
Author: jonas <address@hidden>
Date:   Wed Dec 28 23:15:51 2011 +0100

    Added a link to the MacOS_X_Compiling wiki page

commit 2e6c2843fb85dbd082f960eb4f87079199feda78
Author: jonas <address@hidden>
Date:   Tue Dec 27 21:47:04 2011 +0100

    Fixed the language selection of Pingus.app

    Due to missing escape chars the language got already selected at  
compile time.
    Now the current language is selected at run time of the lancher  
script.
------8<-----8<-----

From my experience a clean history with informative comments can help  
a lot when trying to understand foreign code, review changes and fix  
bugs.

bye,
Jonas

> On Dec 28, 2011, at 6:08 PM, Jonas Bähr wrote:
>
> > Am 29.12.2011 um 02:10 schrieb Josh Dye:
> > > I'll look at that right now, I didn't even think about that.  Just  
> > > found a fix, will test and commit
> > > <osx.patch>
> >
> > I've got some minor enhancements to the patch (cleaned patch  
> > attached):
> > *) stick to the current coding-style
> > *) Check the bundle path only when needed
> > *) make sure the CFURLRef is always freed when used
> > *) give the helper method a comprehensive name and make it a static  
> > bool
> >
> > Thanks for your work,
> > bye,
> > Jonas
> > <data-dir-check.patch>
> >
> > > On Dec 28, 2011, at 3:12 PM, Jonas Bähr wrote:
> > >
> > > > Am 28.12.2011 um 23:22 schrieb Jonas Bähr:
> > > > > Am 28.12.2011 um 20:19 schrieb Josh Dye:
> > > > > > It has major GFX problems, at least on my compi=uter, the build  
> > > > > > that I am doing has the same problems- I want to make sure it's  
> > > > > > not just my computer--  http://www.mediafire.com/?gl4gzlnt1faj565
> > > > > >
> > > > > > PS-- I'm not sure if your build system will work anymore
> > > > > > http://code.google.com/p/pingus/wiki/MacOS_X_Compiling
> > > > >
> > > > > It is still working, I just checked.
> > > >
> > > > There is something else that now stopped working though:  
> > > > launching pingus from the source dir (without specifying the -- 
> > > > datadir)
> > > > While this is not critical, it's annoying. Responsible for that  
> > > > are the changes 54d9e1e9 and 6ee7570b. What do you think about a  
> > > > check, whether the application was executed from a bundle or not?  
> > > > Or checking if "data" exists in either location before setting it?
> > > > Don't get me wrong, the ability to use the bundle's resource dir  
> > > > out of the box is nice (removes the need for a wrapper script)  
> > > > but loosing the option to simply start pingus during development  
> > > > is not so nice.
> > > >
> > > > bye,
> > > > Jonas
> > > >
> > > > > also referenced the wiki page in my INSTALL.macosx. I'm sure  
> > > > > it'll be helpfull for those who want to do everything by hand.  
> > > > > See the attached patches. It also fixes the compile-time/run- 
> > > > > time language detection issue Andy ran into.
> > > > >
> > > > > bye,
> > > > > Jonas
> > > > > <macosx-build-fixes.tar.bz2>
> > > > >
> > > > > > My x86_64 build (new build system) http://www.mediafire.com/?tf1kv01f6qzqaec
> > > > > >
> > > > > >
> > > > > > On Dec 27, 2011, at 11:32 AM, Jonas Bähr wrote:
> > > > > >
> > > > > > > Hi,
> > > > > > >
> > > > > > > Am 27.12.2011 um 19:55 schrieb Andy Milne:
> > > > > > >
> > > > > > > > Thanks for the Mac port. But, sorry to report it has a couple  
> > > > > > > > of problems:
> > > > > > > > 1. It crashes as soon as I click on the play checkbox of any  
> > > > > > > > level that I select.
> > > > > > >
> > > > > > > Please try to start pingus in a terminal, maybe it reports  
> > > > > > > some errors there.
> > > > > > > To do so, cd to the folder containing Pingus.app (e.g. "cd / 
> > > > > > > Applications/Games/"), then execute "Pingus.app/Contents/MacOS/ 
> > > > > > > pingus"
> > > > > > > Here I get the following output:
> > > > > > > --------8<--------8<--------
> > > > > > > $ Pingus.app/Contents/MacOS/pingus
> > > > > > > Welcome to Pingus 0.7.6!
> > > > > > > ========================
> > > > > > > userdir:                 /Users/jonas/.pingus/
> > > > > > > datadir:                 Pingus.app/Contents//data
> > > > > > > language:                German (de)
> > > > > > > sound support:           enabled
> > > > > > > music support:           enabled
> > > > > > > fullscreen:              disabled
> > > > > > >
> > > > > > > 2011-12-27 20:30:10.229 pingus[328:613] Warning once: This  
> > > > > > > application, or a library it uses, is using NSQuickDrawView,  
> > > > > > > which has been deprecated. Apps should cease use of QuickDraw  
> > > > > > > and move to Quartz.
> > > > > > > --------8<--------8<--------
> > > > > > >
> > > > > > > > 2. It defaults to German, which is ok, but the --language  
> > > > > > > > option does not work to change it.
> > > > > > >
> > > > > > > Sorry, that's my fault. I just saw that the language was  
> > > > > > > selected at compile time instead of run time. To fix it, open  
> > > > > > > "Pingus.app/Contents/MacOS/pingus" using a text editor and  
> > > > > > > change the second line to "lang=<what you need>".
> > > > > > > Currently the line reads "lang=de" but it should be:
> > > > > > > ------8<--------8<------
> > > > > > > lang=`defaults read .GlobalPreferences AppleLanguages | tr -d  
> > > > > > > [:space:] | cut -c2-3`
> > > > > > > ------8<--------8<------
> > > > > > > Apparently I missed to escape the back ticks, so this was  
> > > > > > > executed while I build pingus already.
> > > > > > > Another problem with this language detection is, that it only  
> > > > > > > works with two-letter codes, so something like "pt_BR" won't  
> > > > > > > be detect correctly.
> > > > > > >
> > > > > > > > Which version of OSX should it work with? I'm running 64bit  
> > > > > > > > 10.7.2
> > > > > > >
> > > > > > > I build it on 32 bit intel, using 10.5 "Leopard". Here I can  
> > > > > > > play levels but some graphics are missing.
> > > > > > >
> > > > > > > bye,
> > > > > > > Jonas
> > > > > > >
> > > > > > > > On Tue, Dec 27, 2011 at 5:33 AM, Ingo Ruhnke  
> > > > > > > > <address@hidden> wrote:
> > > > > > > > On 24 December 2011 23:45, Ingo Ruhnke <address@hidden>  
> > > > > > > > wrote:
> > > > > > > > > Pingus 0.7.6 is out, not much changes to the core game, but  
> > > > > > > > > 10 brand
> > > > > > > > > new Xmas themed levels.
> > > > > > > > >
> > > > > > > > > * added new SVG icons for Pingus
> > > > > > > > > * added 10 new xmas themed levels
> > > > > > > > > * added ability to sort levelsets by priority
> > > > > > > > >
> > > > > > > > > Download: http://pingus.seul.org
> > > > > > > >
> > > > > > > > Jonas Bähr <address@hidden> build a MacOSX port of it,  
> > > > > > > > available at:
> > > > > > > >
> > > > > > > > http://pingus.googlecode.com/files/Pingus-0.7.6.dmg
> > > > > > > >
> > > > > > > > --
> > > > > > > > Blog:     http://grumbel.blogspot.com/
> > > > > > > > JabberID: xmpp:address@hidden
> > > > > > > > ICQ:      59461927
> > > > > > > >
> > > > > > > > _______________________________________________
> > > > > > > > Pingus-Devel mailing list
> > > > > > > > address@hidden
> > > > > > > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Pingus-Devel mailing list
> > > > > > > address@hidden
> > > > > > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> > > > > >
> > > > > >
> > > > > > _______________________________________________
> > > > > > Pingus-Devel mailing list
> > > > > > address@hidden
> > > > > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> > > > >
> > > > > _______________________________________________
> > > > > Pingus-Devel mailing list
> > > > > address@hidden
> > > > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> > > >
> > > >
> > > > _______________________________________________
> > > > Pingus-Devel mailing list
> > > > address@hidden
> > > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> > >
> > > _______________________________________________
> > > Pingus-Devel mailing list
> > > address@hidden
> > > https://lists.nongnu.org/mailman/listinfo/pingus-devel
> >
> > _______________________________________________
> > Pingus-Devel mailing list
> > address@hidden
> > https://lists.nongnu.org/mailman/listinfo/pingus-devel
>
>
> _______________________________________________
> Pingus-Devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/pingus-devel