[MPlayer-cvslog] r26352 - in trunk: Makefileosdep osdep/Makefile

[Diego Biurrun]

On Sun, Apr 13, 2008 at 09:01:26AM +0800, Zuxy wrote:
> 
> "Diego Biurrun" <diego at biurrun.de> 写入消息 news:20080413004049.GI2075 at biurrun.de...
> > On Sun, Apr 13, 2008 at 08:32:24AM +0800, Zuxy wrote:
> >> 
> >> "Diego Biurrun" <diego at biurrun.de> ???????? 
> >> news:20080412233847.GF2075 at biurrun.de...
> >> > On Sun, Apr 13, 2008 at 03:15:45AM +0800, Zuxy Meng wrote:
> >> >>
> >> >> "diego" <subversion at mplayerhq.hu> ????????????:20080408182530.5AD2F3B3E2 at natsuki.mplayerhq.hu...
> >> >> >
> >> >> > Log:
> >> >> > Merge osdep/Makefile into the top-level Makefile, thus getting rid
> >> >> > of recursive make in the osdep/ subdirectory.
> >> >> >
> >> >> > --- trunk/Makefile (original)
> >> >> > +++ trunk/Makefile Tue Apr  8 20:25:30 2008
> >> >> > @@ -96,9 +107,6 @@ LIBS_MPLAYER-$(GUI)               += gui
> >> >> >
> >> >> > -# Having this in libosdep.a is not enough.
> >> >> > -OBJS_MPLAYER-$(TARGET_WIN32) += osdep/mplayer-rc.o
> >> >> > -
> >> >>
> >> >> Why did u remove this line? The little logo is gone after this commit and
> >> >> r26388....
> >> >
> >> > Umm, I bungled this one.  I cannot really test on Windows.  What "little
> >> > logo" does this thing add?  I thought it filled in some of the info that
> >> > you can get to by looking at the properties from the file context menu
> >> > in the Windows explorer...
> >> 
> >> It's the MPlayer icon as in etc/mplayer.ico that looks similar to 
> >> mplayer.xpm.
> > 
> > Ah, OK.  Does this work on both MinGW and Cygwin?
> 
> I guess so since Cygwin produces PE executables too.

Fixed.

Diego