On Sat, Sep 01, 2012 at 07:32:50AM -0700, Alexander Hansen wrote:
> On 8/31/12 5:36 PM, Jack Howarth wrote:
> > Why exactly are we building the entirety of gettext-tools at -O0
> > in 10.7 branch when only a single source file, xgettext.c, is problematic on
> > older clang? The previous patch that applied -O4 to the compilation of
> > xgettext.c,
> > should work fine using -O0 instead for just that file.
> >
> >
> > --- gettext-0.18.1.1/gettext-tools/src/Makefile.in.orig 2012-08-31
> > 20:21:15.000000000 -0400
> > +++ gettext-0.18.1.1/gettext-tools/src/Makefile.in 2012-08-31
> > 20:22:03.000000000 -0400
> > @@ -2357,11 +2357,11 @@
> >
> > xgettext-xgettext.o: xgettext.c
> > $(AM_V_CC) @AM_BACKSLASH@
> > - $(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) $(xgettext_CPPFLAGS)
> > $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) -c -o xgettext-xgettext.o `test -f
> > 'xgettext.c' || echo '$(srcdir)/'`xgettext.c
> > + $(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) $(xgettext_CPPFLAGS)
> > $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) -O0 -c -o xgettext-xgettext.o `test -f
> > 'xgettext.c' || echo '$(srcdir)/'`xgettext.c
> >
> > xgettext-xgettext.obj: xgettext.c
> > $(AM_V_CC) @AM_BACKSLASH@
> > - $(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) $(xgettext_CPPFLAGS)
> > $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) -c -o xgettext-xgettext.obj `if test -f
> > 'xgettext.c'; then $(CYGPATH_W) 'xgettext.c'; else $(CYGPATH_W)
> > '$(srcdir)/xgettext.c'; fi`
> > + $(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) $(xgettext_CPPFLAGS)
> > $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) -O0 -c -o xgettext-xgettext.obj `if test
> > -f 'xgettext.c'; then $(CYGPATH_W) 'xgettext.c'; else $(CYGPATH_W)
> > '$(srcdir)/xgettext.c'; fi`
> >
> > xgettext-x-c.o: x-c.c
> > $(AM_V_CC) @AM_BACKSLASH@
> >
> > Building everything at -O0 seems like massive overkill.
> > Jack
> >
> >
> >
>
> Don't cc stuff over multiple lists. There's no reason to do that and it
> makes it easy to lose the discussion.
>
> Pretty much everybody on -core reads -devel, too
Alex,
I normally email to fink-devel and cc the maintainer, which in the case of
gettext-tools has the
fink-core mailing list as the maintainer email.
Jack
>
> And speaking of overkill, why raise the Revision?
> --
> Alexander Hansen, Ph.D.
> Fink User Liaison
> My package updates: http://finkakh.wordpress.com/
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Fink-devel mailing list
[email protected]
List archive:
http://news.gmane.org/gmane.os.apple.fink.devel
Subscription management:
https://lists.sourceforge.net/lists/listinfo/fink-devel
