Message ID | CADj25HP5NjZ5bHmmwx3aVRUNcHe+L9Q9r-Dch+mdnxVUQpko1Q@mail.gmail.com |
---|---|
State | New |
Headers | show |
Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html The patch is to fix dependency issue of libgcc Makefile.in by adding 'rm libgcc_tm.stamp' in the clean rule. That was fixed on main trunk (r187838) but not on gcc-4_7-branch. Since gcc-4_7-branch is still open, is it OK to backport r187838 into gcc-4_7-branch? Best regards, jasonwucj
On Fri, May 17, 2013 at 7:29 PM, Chung-Ju Wu <jasonwucj@gmail.com> wrote: > Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html > > The patch is to fix dependency issue of libgcc Makefile.in > by adding 'rm libgcc_tm.stamp' in the clean rule. > > That was fixed on main trunk (r187838) but not on gcc-4_7-branch. > Since gcc-4_7-branch is still open, is it OK to backport r187838 > into gcc-4_7-branch? It's OK with me but in general I think the release managers need to approve backports. Ian
2013/5/19 Ian Lance Taylor <iant@google.com>: > On Fri, May 17, 2013 at 7:29 PM, Chung-Ju Wu <jasonwucj@gmail.com> wrote: >> Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html >> >> The patch is to fix dependency issue of libgcc Makefile.in >> by adding 'rm libgcc_tm.stamp' in the clean rule. >> >> That was fixed on main trunk (r187838) but not on gcc-4_7-branch. >> Since gcc-4_7-branch is still open, is it OK to backport r187838 >> into gcc-4_7-branch? > > It's OK with me but in general I think the release managers need to > approve backports. > > Ian Thanks Ian's approval for the libgcc part. Hi, Richard, here is the mail for my patch: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html The patch doesn't add any new feature to gcc-4_7-branch. It just fixes the dependency issue in libgcc Makefile.in. Is it OK to do backport? Best regards, jasonwucj
Il 18/05/2013 04:29, Chung-Ju Wu ha scritto: > Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html > > The patch is to fix dependency issue of libgcc Makefile.in > by adding 'rm libgcc_tm.stamp' in the clean rule. > > That was fixed on main trunk (r187838) but not on gcc-4_7-branch. > Since gcc-4_7-branch is still open, is it OK to backport r187838 > into gcc-4_7-branch? It's trivial enough that you can go ahead and backport it. Thanks! Paolo
2013/5/19 Paolo Bonzini <bonzini@gnu.org>: > Il 18/05/2013 04:29, Chung-Ju Wu ha scritto: >> Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html >> >> The patch is to fix dependency issue of libgcc Makefile.in >> by adding 'rm libgcc_tm.stamp' in the clean rule. >> >> That was fixed on main trunk (r187838) but not on gcc-4_7-branch. >> Since gcc-4_7-branch is still open, is it OK to backport r187838 >> into gcc-4_7-branch? > > It's trivial enough that you can go ahead and backport it. > > Thanks! > > Paolo Thanks for both Ian and Paolo's approval. :) Committed into gcc-4_7-branch as Rev.199091. Best regards, jasonwucj
On Sun, 19 May 2013, Chung-Ju Wu wrote: > 2013/5/19 Ian Lance Taylor <iant@google.com>: > > On Fri, May 17, 2013 at 7:29 PM, Chung-Ju Wu <jasonwucj@gmail.com> wrote: > >> Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html > >> > >> The patch is to fix dependency issue of libgcc Makefile.in > >> by adding 'rm libgcc_tm.stamp' in the clean rule. > >> > >> That was fixed on main trunk (r187838) but not on gcc-4_7-branch. > >> Since gcc-4_7-branch is still open, is it OK to backport r187838 > >> into gcc-4_7-branch? > > > > It's OK with me but in general I think the release managers need to > > approve backports. > > > > Ian > > Thanks Ian's approval for the libgcc part. > > Hi, Richard, here is the mail for my patch: > http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html > > The patch doesn't add any new feature to gcc-4_7-branch. > It just fixes the dependency issue in libgcc Makefile.in. > > Is it OK to do backport? Ok. Thanks, Richard.
Index: libgcc/Makefile.in =================================================================== --- libgcc/Makefile.in (revision 198729) +++ libgcc/Makefile.in (working copy) @@ -121,7 +121,8 @@ .PHONY: all clean clean: - -rm -f config.h libgcc_tm.h stamp-h stmp-ldirs libgcc.map + -rm -f config.h libgcc_tm.h libgcc.map + -rm -f libgcc_tm.stamp stamp-h stmp-ldirs -rm -f *$(objext) -rm -f *.dep -rm -f *.a